a11y: Notify ATK_STATE_SHOWING state change

https://bugzilla.gnome.org/show_bug.cgi?id=648633

clutter-stage-win32: Fix name cogl_win32_onscreen_set_foreign_window

Since commit 38b67e2884f of Cogl the naming scheme for winsys-specific
API has changed to be cogl_win32_onscreen_* instead of
cogl_onscreen_win32_* so it wouldn't build on Windows.

osx: Avoid leaks in Stage::realize

clutter_stage_osx_realize() can now be called multiple times without
leaking views and windows.

https://bugzilla.gnome.org/show_bug.cgi?id=655307

osx: Implement windowDidResignKey

Added isHiding field to _ClutterStageOSX to allow windowDidResignKey
delegate to not order full screen window back whilst the full screen
window was being hidden.  This caused other application windows to be
hidden.  Also added code to keep hidden stage windows from being listed
in the application's Windows menu.

https://bugzilla.gnome.org/show_bug.cgi?id=655311

osx: Add windowDidChangeScreen delegate

Added -windowDidChangeScreen: delegate to handle condition where moving
host window to a different screen would cause pick errors to be output.
The delegate just causes the stage to be redrawn which re-creates the
pick buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=655306

model: Remove an unused variable

introspection: add missing introspection data from Vala bindings

https://bugzilla.gnome.org/show_bug.cgi?id=655387

docs: Update with the newly added Animatable symbol

state: Use the Animatable interface

The Animatable interface allows object classes to provide and animate
properties outside of the usual GObject property introspection API.

This change allows ClutterState to defer to the animatable objects the
property introspection and animation, just like ClutterAnimation does.

actor: There is no need to implement animate_property()

It's a deprecated virtual function, and its replacement does the right
thing by default.

animation: Use Animatable.interpolate_value()

And stop using the deprecated animate_property() method.

animatable: Deprecate and replace animate_property()

The animate_property() method of the Animatable interface is far less
than optimal:

  • it has a direct reference to ClutterAnimation;
  • it has an interval decomposed as two values.

These issues tie the Animatable interface with the Animation object,
even though it's neither necessary nor future-proof.

Let's introduce a new method, interpolate_value(), which does not
reference ClutterAnimation and uses a ClutterInterval to express the
initial and final states.

model: Clean up ModelIter vfunc wrappers

All ClutterModelIter virtual functions have a default implementation,
and G_TYPE_INSTANCE_GET_CLASS cannot return NULL unless in case of a
catastrophic event in the type system - which will most likely blow up
any application code way before you could call a ModelIter method.

Thus, the idiom:

  klass = CLUTTER_MODEL_ITER_GET_CLASS (instance);
  if (klass && klass->vfunc)
    klass->vfunc (instance);

is utterly useless complication, and it can be perfectly replaced by:

  CLUTTER_MODEL_ITER_GET_CLASS (instance)->vfunc (instance);

without any loss of safety.

model: Make sure to emit ::row-changed

Currently, only clutter_model_iter_set_valist() is in charge of emitting
the ClutterModel::row-changed signal. Both the set() and the
set_valist() functions can be called with multiple columns, so we
coalesce the signal emission at the end of the set_valist(), to have a
single ::row-changed emission per change.

The clutter_model_iter_set_value() function is just a thin wrapper
around the set_value() virtual function, but since it's called
internally we cannot add the signal emission there as well, as we'd
break the signal coalescing.

For this reason, we need some code refactoring inside the various set()
variants of ClutterModelIter:

  - we only use the internal va_arg variant for both the set() and
    set_valist() public functions, to avoid multiple type checks;
  - the internal set_valist() calls an internal set_value() method
    which calls the virtual function from the iterator vtable;
  - a new internal emit_row_changed() method is needed to retrieve
    the ClutterModel from the iterator, and emit the signal;

Now, all three variants of the value setter will call an internal
ClutterModelIter::set_value() wrapper, and emit the ::row-changed
signal.

To check that the intended behaviour has been implemented, and it's not
going to be broken, the test suite has grown a new unit which populates
a model and changes a random row.

Updated Czech translation

Updated Spanish translation

test/cairo-clock: Remove a double source color set

Do not call cairo_set_source_rgba() right after calling
clutter_cairo_set_source_color().

a11y: Use existing clutter_actor_get_paint_visibility

Cally was initially created with Clutter 0.6 in mind. To check
recursively the visibility of a actor a custom method was added.
Since 0.8.4 clutter_actor_get_pain_visibility provides
the same functionality.

Also removed a dummy method. Lets add methods that provide a real
functionality.

cairo-texture: Add the :auto-resize property

Keeping the backing Cairo surface of a CairoTexture canvas in sync with
the actor's allocation is tedious and prone to mistakes. We can
definitely do better by simply exposing a property that does the surface
resize and invalidation automagically on ::allocate.

tests: initial support for cally-text conform tests

Post-release version bump to 1.7.7

Release Clutter 1.7.6 (snapshot)

bin-layout: Fix a leak

https://bugzilla.gnome.org/show_bug.cgi?id=655342

docs: Fixes for building the API reference

cairo-texture: Deprecate create()/create_region()

The recommended way of drawing on a ClutterCairoTexture is the ::draw
signal.

cairo-texture: Use signal-based drawing

The current "create context/draw/destroy context" pattern presents
various problems. The first issue is that it defers memory management to
the caller of the create() or create_region() methods, which makes
bookkeeping of the cairo_t* harder for language bindings and third party
libraries. The second issue is that, while it's easier for
draw-and-forget texturs, this API is needlessly complicated for contents
that have to change programmatically - and it introduces constraints
like calling the drawing code explicitly after a surface resize (e.g.
inside an allocate() implementation).

By using a signal-based approach we can make the CairoTexture actor
behave like other actors, and like other libraries using Cairo as their
2D drawing API.

The semantics of the newly-introduced ::draw signal are the same as the
one used by GTK+:

  - the signal is emitted on invalidation;
  - the cairo_t* context is owned by the actor;
  - it is safe to have multiple callbacks attached to the same
    signal, to allow composition;
  - the cairo_t* is already clipped to the invalidated area, so
    that Cairo can discard geometry immediately before we upload
    the texture data.

There are possible future improvements, like coalescing multiple
invalidations inside regions, and performing clipped draws during
the paint cycle; we could even perform clipped redraws if we know the
extent of the invalidated area.

conform/cogl-materials: Fix a compiler warning

The function checking for the presence and use of the GLES2 support in
Cogl should be protected by #ifdef guards, to avoid a compiler warning.

[l10n]Added Catalan (Valencian) translation

[l10n]Added Catalan translation

Updated Czech translation

Updated Spanish translation

docs: Remove manual/

The programmers manual never materialized, and its role has been taken
by both the cookbook and the API reference.

Updated Slovenian translation

tests: Dynamically resolve GL symbols

Some of the tests are making direct GL calls. Eventually we want
Clutter not to link directly against any GL library so that it can
leave Cogl to load it dynamically. As a step towards getting this to
work this patch changes the tests to resolve the symbols using
cogl_get_proc_address instead of linking directly.

docs: Add fixed size and constraints to the size negotiation docs

actor: Clean up the allocate() checks

Move the conditions to their own boolean flags, to avoid making the
if() incomprehensible.

docs: Rename the invariants documentation

docs: Add a simple description of the size negotiation mechanism

Describe (with an example) the ordering and meaning of flags for the
size negotiation in ClutterActor.

texture: Clean up error reporting

ClutterTexture relies too much on GError, even for things that are
clearly programmer errors. Also, no error message passed to GError
is marked for translation as it should.

We should move the programmer errors, like passing the wrong bpp
value with regards to the presence of the alpha channel, to real
warnings; we should also try and harmonize all the error messages,
and not mention Cogl — especially in the ones marked for translation.

build: don't explicitly include and egl or gl headers

This avoids explicitly including gl or egl headers in
clutter-egl-headers.h. We were getting build failures when building
clutter against a libcogl that has runtime support for GL and GLES
because cogl-defines.h was including gl.h and then clutter-egl-headers.h
was later including GLES2/gl.h with typedef conflicts. Clutter relies on
Cogl to abstract GL and GLES and the winsys APIs like EGL and GLX so
Clutter should just rely on cogl.h to include the appropriate egl.h in
clutter-egl-headers.h.

Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
Reviewed-by: Neil Roberts <neil@linux.intel.com>

Updated Norwegian bokmål translation

alpha: Don't use the GClosure to compute the alpha when possible

Setting up the call and calling the GClosure was showing up in profiles
and seemed an easy one to remove.

Instead of calling the closure, let's remember the alpha func and the
user_data when possible (ie set_mode() and set_func()) and use it in
get_alpha().

https://bugzilla.gnome.org/show_bug.cgi?id=654727

text: Fix the default value of "editable"

The "editable" property is documented to default to TRUE, but is
initialized to FALSE in the _init() function.

Third party code would be affected if we changed the default to be
TRUE, so we have to change the default value in the GParamSpec.

https://bugzilla.gnome.org/show_bug.cgi?id=654726

Do not use return with void functions

https://bugzilla.gnome.org/show_bug.cgi?id=654718

timeline: Cast elapsed time before emitting ::new-frame

When emitting a new-frame signal, priv->elapsed_time is passed as a
parameter. This is a gint64. The closure marshal uses an INT. On some
platforms, this is not received correctly by signal handlers (they
receive 0). One solution is to cast priv->elapsed_time to a gint when
emitting the signal.

We cannot change the signature of the signal without breaking ABI.

https://bugzilla.gnome.org/show_bug.cgi?id=654066

snap-constraint: Fix allocation for CLUTTER_SNAP_EDGE_RIGHT

Use source_with instead of source_height to calculate the the x1/x2
positions for CLUTTER_SNAP_EDGE_RIGHT

https://bugzilla.gnome.org/show_bug.cgi?id=654024

[l10n] Updated German translation

[l10n] Updated German translation

x11/event: Avoid compiler warnings with HAVE_XGE undefined

build: Escape AC_DEFINE arguments

actor: Plug a leak in an error path

If the meta for the animation property is not found, the name of the
property to look for is still from the token, and we need to free the
memory allocated for it.

x11: Improve XGenericEventCookie data allocation

https://bugzilla.gnome.org/show_bug.cgi?id=654656

Clutter may be used together with GTK+, which indirectly may use
XInput2 too, so the cookie data must persist when both are handling
events.

What happens now in a nutshell is, Clutter is only guaranteed to allocate
the cookie itself after XNextEvent(), and only frees the cookie if its
XGetEventData() call allocated the cookie data.

The X[Get|Free]EventData() calls happen now in clutter-event-x11.c as
hypothetically different event translators could also handle other set
of X Generic Events, or other libraries handling events for that matter.

pick: Use Cogl API not GL API to disable dithering

When picking we need to disable dithering to be sure that the hardware
will not modify the colors we use as actor identifiers. Clutter was
manually calling glEnable/Disable GL_DITHER to handle this, but that was
a layering violation since Cogl is intended to handle all interactions
with OpenGL. Since we are now striving for GL vs GLES to be a runtime
choice we need to remove this last direct usage of GL from Clutter so it
doesn't have to be linked with GL at build time.

Signed-off-by: Neil Roberts <neil@linux.intel.com>

Provide stable paint-boxes for fixed sized actors

This updates _clutter_paint_volume_get_stage_paint_box to try and
calculate more stable paint-box sizes for fixed sized paint-volumes by
not basing the size on the volume's sub-pixel position.

So the aim is that for a given rectangle defined with floating point
coordinates we want to determine a stable quantized size in pixels that
doesn't vary due to the original box's sub-pixel position.

The reason this is important is because effects will use this API to
determine the size of offscreen framebuffers and so for a fixed-size
object that may be animated across the screen we want to make sure that
the stage paint-box has an equally stable size so that effects aren't
made to continuously re-allocate a corresponding fbo.

The other thing we consider is that the calculation of this box is
subject to floating point precision issues that might be slightly
different to the precision issues involved with actually painting the
actor, which might result in painting slightly leaking outside the
user's calculated paint-volume. This patch now adds padding to consider
this too.

Signed-off-by: Neil Roberts <neil@linux.intel.com>
Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>

actor: improve rounding in get_allocation_geometry

Instead of relying on C to round the floating point allocation to
integers by flooring the values we now use CLUTTER_NEARBYINT to round
the allocation's position and size to the nearest integers instead. Using
floor leads to rather unstable rounding for the width and height when
there may be tiny fluctuations in the floating point width/height.

Signed-off-by: Neil Roberts <neil@linux.intel.com>
Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>

Adds a CLUTTER_NEARBYINT macro for float rounding

This is a replacement for the nearbyint function which always rounds to
the nearest integer. nearbyint is a C99 function so it might not always
be available but also it seems in glibc it is defined as a function call
so this macro could end up faster anyway. We can't just add 0.5 because
it will break for negative numbers.

Signed-off-by: Neil Roberts <neil@linux.intel.com>
Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>

paint-volume: don't round paint-volume from allocation

The implementation of _clutter_actor_set_default_paint_volume which
simply uses the actor's allocation to determine a paint-volume was
needlessly using the allocation rounded to integers by internally using
clutter_actor_get_allocation_geometry instead of
clutter_actor_get_allocation_box. This was introducing a lot of
instability into the paint-volume due to the way rounding was done.

The code has now been updated to use clutter_actor_get_allocation_box
so we are dealing with the floating point allocation instead.

Signed-off-by: Neil Roberts <neil@linux.intel.com>
Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>

Make sure the gesture is cancelled even if we miss the release event

Remove unneeded class closure

osx: Code style cleanups

Add default closures for the signals in ClutterGestureAction

Also make sure we cancel the gesture as soon as we know we have to

Fixed typo of bidng to bind

https://bugzilla.gnome.org/show_bug.cgi?id=654103

build: Unescape conditionally defined defaults in AS_HELP_STRING

AS_HELP_STRING() escapes everything, so m4 doesn't have a chance to run
expansions.

build: Detect the appropriate default flavour

If we're building on/for Windows, set 'win32' as the default flavour; if
we're building on OS X, set 'osx' as the default flavour. For everything
else, use 'glx'.

Revert "build: Detect the appropriate default flavour"

Ouch, butterfingers and git commit -a.

This reverts commit 81e8c8dec902e1e3b6277ebad5804f2321535332.

build: The OSX backend is not experimental any more

The OSX backend is actively developed, and tested, so we can drop the
experimental flag.

build: Detect the appropriate default flavour

If we're building on/for Windows, set 'win32' as the default flavour; if
we're building on OS X, set 'osx' as the default flavour. For everything
else, use 'glx'.

clutter-stage: Add clutter_stage_get_redraw_clip_bounds

This adds a public function to get the bounds of the current clipped
redraw on a stage. This should only be called while the stage is being
painted. The function diverts to a virtual function on the
ClutterStageWindow implementation. If the function isn't implemented
or it returns FALSE then the entire stage is reported. The clip bounds
are in integer pixel coordinates in the stage's coordinate space.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2421

uprof: Add the units in the name of the "Per Frame" timer attribute

So it shows up in the profile report. Take the opportunity to write a
better long description for the attribute.

https://bugzilla.gnome.org/show_bug.cgi?id=654363

test-cogl-materials: Use glGetString to determine cogl driver

Rather than using the #ifdefs and assuming that only one Cogl driver
is compiled in (which is no longer true), the test now calls
glGetString to check the GL_VERSION. This is kind of a hack but the
test is already calling GL functions directly anyway.

test-cogl-materials: Remove a redundant comment

test-cogl-materials had a weird comment about glReadPixels using
inverted coordinates but the test now uses cogl_read_pixels instead of
glReadPixels so it is irrelevant.

build: update cogl_onscreen_x11 syms to cogl_x11_onscreen

Cogl recently renamed symbols with the form
cogl_onscreen_<platform>_blah to be consistent with other platform
specific APIs so they are now named like cogl_<platform>_onscreen_blah.
This makes the corresponding change to clutter.

backend-cogl: update _get_egl_context symbol name

Cogl changed has changed the name of cogl_context_egl_get_egl_context to
cogl_egl_context_get_egl_context to be consistent with other platform
specific symbols.

don't call cogl_set_default_context since it's been removed

cogl_set_default_context has been removed from Cogl so this updates
Clutter in-line with that change.

build: depend on cogl-1.7.3 development version

Since some experimental API in Cogl that Clutter uses has changed this
bumps our dependency up to 1.7.3 before landing the corresponding build
fixes for clutter to bring it in line with the Cogl changes.

uprof: Rename timers to remove the EGL specific bits

Cogl abstracts the winsys, Remove the egl prefix from the timer names.

https://bugzilla.gnome.org/show_bug.cgi?id=654361

Updated POTFILES.in

text: Make sure that the selection box is big enough

Do not use integers to store the result of the PANGO_UNITS scaling to
avoid rounding issues with the default C rules. We should use cielf()
instead.

osx: Fix a compiler warning

No declaration after statement.

conform/pick: Print debug spew only if verbosity is toggled

texture: Deprecate the new_from_actor() function

Now that we have proper offscreen and shader handling using the
ClutterEffect API, we can finally deprecate this hard to bind, easy
to break function.

Deprecate ClutterScore

The class is of dubious utility, now that we have a complex animation
API in ClutterAnimator and ClutterState, as opposed to a simple one
in ClutterBehaviour. The Score API also suffers from some naïve design
issues that made it far less useful than intended.

test-cogl-materials: Add a simple test for ref-counting copies

This adds a simple test which creates a material and a copy of
it. Both materials are then immediately unref'd. The test checks
whether the materials were actually freed by registering some user
data with a destroy callback. This should catch a bug that Cogl had
where it add an extra reference to the parent when a pipeline is
copied.

Signed-off-by: Robert Bragg <robert@linux.intel.com>

a11y: cally-text get_offset_at_point implementation

See http://bugzilla.clutter-project.org/show_bug.cgi?id=1733

a11y: cally-text get_character_extents implementation

See http://bugzilla.clutter-project.org/show_bug.cgi?id=1733

a11y: expose _cally_actor_get_top_level_origin in cally-actor-private

As this function is needed for atktext get_character_extents and
get_offset_at_point implementations.

a11y: get_default_attributes implementation on cally-text

See http://bugzilla.clutter-project.org/show_bug.cgi?id=1733

a11y: cally-atktext-example cleanups

Added an early return in test_atk_text().
Fixed a unichar conversion
Print the run_attributes too.

a11y: use actor color if no fgcolor found in pango attributes

a11y: factor some PangoAttributes->AtkTextAttributes mappings

That code can then be used by get_run_attributes and
get_default_attributes implementations.

[a11y] Exposing via ATK the proper toolkit name and version

clutter-actor: Don't apply shaders during picking

Commit 0ede622f5132 inadvertently made it so that shaders are applied
during picking. This was making test-shader fail to respond to clicks.

The commit also makes it so that culling is applied during
picking. Presumably this is also unintentional because the commit
message does not mention it. However I think it may make sense to do
culling during picking so it might as well stay that way.

https://bugzilla.gnome.org/show_bug.cgi?id=653959

Post-release version bump to 1.7.5

Release Clutter 1.7.4 (snapshot)

build: Fix up the tarball target used when releasing

build: Clean up the cally test Makefile.am

Use global target variables to avoid copy/pasting the same values in
every single binary target.

tests: Add performance tracking framework

This adds a performance tracking framework that can run a set of tests over
specified git revisions. The ruby script for generating the reports comes from
similar performance tracking in GEGL. The framework permits evaluating new
tests against older version of clutter.

The tests themselves go through a few hoops for disabling framerate limiting in
both mesa and clutter.

When running make check the tests will be run and lines of the form:

@ test-state: 40.51 fps

will be left in the output, a script can scrape these lines out of a build log
on a buildbot to in other ways track performance.

po: Fix the name of the translatable file

I will check before pushing.    I will check before pushing.
I will check before pushing.    I will check before pushing.
I will check before pushing.    I will check before pushing.
I will check before pushing.    I will check before pushing.
I will check before pushing.    I will check before pushing.

po: Fix the location of a translatable file

There is no glx/, only cogl/.

build: Fix the path to the tslib event source