actor: Call show() on set parent conditionally

This commit adds a further conditional check for calling
clutter_actor_show() when adding a child to an actor. We cannot
unconditionally change the value of the show-on-set-parent property like
the original solution of commit 81b19a78f5e02ae8ffd9235054d80b172496415b
as that breaks the document invariant that show-on-set-parent will be
changed iff an actor is without a parent.

The new ADD_CHILD_SHOW_ON_SET_PARENT flag is part of the default and
legacy flags, thus retaining the default behaviour when adding a child;
the flag is not passed when reordering the list of children, which means
we ignore the state of the show-on-set-parent property.

The conformance test suite fully passes, including the newly added test
to verify that changing the paint order does not trigger visibility.

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

Revert "actor: Unconditionally set show_on_set_parent"

This reverts commit 81b19a78f5e02ae8ffd9235054d80b172496415b.

The commit breaks the conformance test unit for the invariants we
guarantee for the 1.x API:

ERROR:actor-invariants.c:307:actor_show_on_set_parent: assertion failed: (show_on_set_parent)

docs: Add the 1.12 symbols index to the API reference

Be resilient in case there is no device manager

It's possible to run Clutter with the 'null' input backend, which means
that clutter_device_manager_get_default() may return NULL. In the future
we may add a default dummy device manager, but right now it's safer to
just add a simple NULL check in the places where we ask for the device
manager.

Remove calls to cogl_disable_fog

I can't think of any reason why it would do this and there's no
comment explaining it so let's just remove it. The global fog state
has been removed in Cogl 2.0 so it will cause problems later.

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

clutter-stage-cogl: Use CoglPrimitive for the debug redraw code

CoglVertexBuffer is deprecated so here is a fairly simple replacement
to use the experimental CoglPrimitive API.

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

device-manager: select/unselect device events as device change

When a device is added or changes, select/unselect events from X
accordingly.

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

device-manager: don't select events for disabled devices

I.e. only select events for core devices or devices explicitly enabled
by clients.

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

conform: Make sure that raising/lowering children doesn't change state

Namely, visibility and show-on-set-parent.

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

actor: Unconditionally set show_on_set_parent

Otherwise, doing something like adjusting the child's position
on a hidden actor will re-show it, which is not what we want.

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

actor: Not-actors don't have metas

cally: Fix the pkg-config file

Drop a bunch of variables that are not meant to be used by Cally; also,
drop the wrong library name from the Libs key: Clutter has a single
shared library, now.

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

x11/device-manager-xi2: Flip deltas around

Up/down is Y.

Updated POTFILES.in

Deprecate ClutterMedia

The interface looked like a good idea around the time Clutter 0.2 was
out, but in reality we never had a proper, and supported implementation
outside of clutter-gst - thus, ClutterMedia was acting like a wrapper
around GStreamer, leading to hilarious issues of impedence mismatch
between API and all sorts of indirection issues typical of wrong
abstractions.

In theory, ClutterMedia should have been deprecated and removed before
we hit 1.0, but we kept flip-flopping on the issue.

For 2.0, it's time to take it out.

And shoot it in the face.

text: Bubble up a click and quit early for just static text

Static text shouldn't need a cursor position or anything like that,
and it always eats a button press event.

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

[l10n] Updated German translation

actor: Return a valid paint volume by default

It's been a year and change, and two stable releases, since we
introduced the paint volume mechanism to allow actors to paint outside
their allocation safely in environments that support clipped redraws.

The time has come to flip the switch, and return a valid paint volume,
matching the actor's allocation, by default - at least for Actor
instances from classes that do not override paint() and
get_paint_volume().

If an actor has a paint signal handler then it's the user responsability
not to paint outside the allocation - and to suffer the consequences of
doing so; in an ideal world, paint() would not be a signal in the first
place anyway. Plus, the idea that painting can happen at any time and
still have a valid surface greatly conflicts with the design goal of
making Clutter's rendering operations fully retained into a render tree.

We can still revert this commit before spinning 1.12, if need be.

interactive/snap-constaint: Modernize code

actor: Check for mapped children in the default get_paint_volume()

Unmapped children won't be painted so they should not contribute to the
default paint volume computation.

actor: Clear MetaGroups when empty

When removing the last Action, Constraint, or Effect, we should also be
clearing the corresponding MetaGroup: code inside ClutterActor relies on
NULL checks, and changing them all to check for NULL && n_items == 0
would not be fun.

Rename XINPUT_2_2 define to HAVE_XINPUT_2_2

configure.ac defines XINPUT_2_2 if XI 2.2 support was found. The code
expects XINPUT_2_2 in the device manager, but HAVE_XINPUT_2_2 in the x11
backend.

On newer X servers, the latter causes a BadValue when XIQueryDevice sends a
different major/minor than gdk's device manager (gnome-control-center).

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
https://bugzilla.gnome.org/show_bug.cgi?id=673961

actor: Fix completed transitions logic

We need to remove the transition only if the current repeat is equal to
the number of repeats, and if the transition was marked as remove on
complete. Otherwise, the transition has to remain where it is.

build: Bump pango dependency to 1.30

Clutter now uses pango_layout_get_character_count() which was added in 1.30.

[l10n]Updated Catalan (Valencian) translation

[l10n] Fixes on Catalan translation

actor: Remove unnecessary relayout/redraw calls

They will be handled internally.

actor: Do not queue_redraw() in set_opacity()

The opacity internal setter will do it for us, and it will take into
consideration any eventual flatten effect applied to the actor.

This unbreaks the actor-offscreen-redirect conformance test.

x11/stage: Leave a comment about multi-head setup

The get_geometry() implementation is broken on multi-head systems; the
only solution is to use XRandR to get the size of the CRTC that contains
the stage.

stage: Unset the natural size when fullscreening

This ensures that constraints and signal handlers will go through the
allocation and not through the cache after ::fullscreen has been
emitted.

interactive/stage-sizing: Clean up

x11/device-manager-xi2: Add 'Abs Distance' axis

In XInput 2, the proximity events of XInput 1 have been replaced by an
axis on a valuator class. This means that, on devices that support
proximity information, for instance pens of a tablet, you will start
receiving events with the distance as an axis value - similarly to how
the pressure and tilt are presented in the API.

enums: Add CLUTTER_INPUT_AXIS_DISTANCE

ClutterText: Fix length passed to clutter_text_buffer_set_text()

clutter_text_buffer_set_text() expects a char count, not a byte
count, so pass -1 rather than using strlen.

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

x11/device-manager-xi2: Remove heuristics for touchscreen devices

Devices that have touch valuator classes will be touchscreens or
touchpads; anything else should be ignored.

osx: Fix build

build: Remove -Waggregate-returns

Updated Swedish translation

actor: Make _clutter_actor_foreach_child() safe again

We were using g_list_foreach() prior to the first Apocalypse, and that
function is resilient against changes to the list while iterating it;
since we are not using a GList any more, we need handle this case
ourselves.

interactive/bin-layout: Call save/restore easing state

Don't do unprotected calls to set_easing_mode() and
set_easing_duration().

actor: Make :content-gravity animatable

Let's start making some non-trivial property animatable, like the
content gravity.

canvas: avoid emission of "draw" signal when the size is zero

This also avoids the warning

  Cogl-WARNING **: ./cogl-buffer.c:215: GL error (1285): Out of memory

generated by cogl_buffer_map when the CoglBuffer has zero length.

actor: Ensure that we correctly remove transitions

The check on :remove-on-complete was inverted, but we also need to check
that the current repeat of the timeline is also the last.

actor: Minor optimization to avoid get_preferred_*

When the easing state has a duration of zero milliseconds we can skip
the entire create_transition() call inside set_width() and set_height(),
to avoid what may be a costly call to get_preferred_*.

actor: Ensure we use the current easing duration and mode

When updating a transition in flight, if the current easing state does
not match the transition's.

docs: Mention in-flight value changes

interactive/animation: Use implicit animations instead of animate()

Also, nest animations.

interactive/transitions: Jump to middle-click coordinates

Middle click will update an existing transition while in flight, and
skip it.

actor: Check easing duration when updating in-flight transitions

If we update a transition that is currently playing, we need to check
the current easing state, and look at the eventual duration, in case
the user wants to cancel the transition.

table-layout: Fix [xy]_expand

A copy and paste thinko.

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

actor: Invalidate the current state when popping easing states

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

actor: Simplify setters of animatable properties

Instead of checking the duration of the current easing state we should
check if there's a transition in progress, and update it
unconditionally.

If there is no easing state, or the easing state has a duration of zero
milliseconds, then create_transition() should bail out early and set the
requested final state.

This allows us to write:

  clutter_actor_save_easing_state (actor);
  clutter_actor_set_x (actor, 200);
  clutter_actor_restore_easing_state (actor);

  [...]

  clutter_actor_set_x (actor, 100);

and have the second set_x() update the easing in progress, instead of
being ignored.

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

interactive/transitions: Add missing easing state save/restore

The test hasn't been updated after commit 229241b8, and was trying to
change the easing state without creating one.

Remove unused fields from LayoutInfo

The x_expand and y_expand members would mess up the initialization of
the constant LayoutInfo default structure.

actor: Add debug annotation in add_transition()

actor: Add IN_DESTRUCTION checks

Commit 80626e75842a24c3d0a45068e241ba309f6ec138 removed an
IN_DESTRUCTION check from within the add_child_internal() method,
outlining an option for bringing it back. It was too late for the 1.10
cycle to do it, and eventually pick up the pieces, but now that we're
at the beginning of the 1.11 cycle we can restore it, and add checks
elsewhere to balance it.

Properly manage timeline refcounting in clutter_animation_set_timeline()

This patch fixes clutter to not crash when multiple animations share
the same timeline and the actors are explicitly destroyed before
the timeline completes (bug 672890)

Updated Belarusian translation.

Updated Latvian translation.

Uploaded Ukranian

Updated Danish translation

Don't use any GL types or defines in Clutter

Some of the Clutter code was using GL types for the primitive types
such as GLint and GLubyte and then passing these to Cogl. This doesn't
make much sense because the Cogl functions directly take native C
types. This patch just replaces them with either a native C type or a
glib type.

Some of the cogl conformance tests are trying to directly call GL for
example to test creating a foreign texture. These tests have been
changed to manually define the GL enum values instead of relying on a
GL header to define them.

This is necessary because Cogl may soon stop including a GL header
from its public headers.

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

configure: Fix the -o operator in a call to test

The disjunction operator was misspelt as -O which tests whether the
following file is owned by the calling user. This doesn't take enough
arguments so bash was showing an error and the test was always
failing. This meant that NEED_XKB_UTILS was always false which should
have broken the build but the Makefile was mistakenly including
clutter-xkb-utils.c again if SUPPORT_WAYLAND is defined.

See 1b77565e for reference.

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

build: Remove disabling Clutter's deprecation warnings while building it

Now that we have deprecation warnings and not full on symbol removal
from the headers, we can just skip Clutter's own, and use only GLib's.

hindi translation by Chandan Kumar

version: Add 1.12 version macros

osx: Fix compilation warnings

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

Fix fallback profile counter macro

[l10n] Updated German translation

Updated Russian translation

table-layout: Remove dead code

If a column/row is expanding, n_expand has to be greater than 1.

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

table-layout: Fix x/y-expand for spanning actors

Ported to ClutterTableLayout from MxTable:

https://github.com/clutter-project/mx/commit/239e53146a678ae753b1fb9a862f4030c66a1d9b

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

table-layout: Fix column visibility for spanning actors

Ported to ClutterTableLayout from MxTable:

https://github.com/clutter-project/mx/commit/6633f185fcfb7ce8a6ef85276d09e4ee92f584b8

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

table-layout: Remove unused 'shrink' field.

Ported to ClutterTableLayout from MxTable:

https://github.com/clutter-project/mx/commit/053a1b04290e6ac307f90033babf761179d1db89

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

actor: Freeze property change notifications on destroy

Updated Telugu Translations

docs: Clean up the animations sections of the Actor reference

interactive/scrolling: Use the implicit animation API

build: Remove superfluous ','

Updated Brazilian Portuguese translation

conform: Set harsher environment for running tests

Set the MALLOC_CHECK_ and MALLOC_PERTURB_ environment variables to catch
allocation issues.

paint-node: Initialize ClutterPaintOperation

Don't leave it to the compiler.

clutter-wayland-compositor.h: Adds missing header guards

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

wayland: Fix some missing includes

The commit 90e5088 added some extra compiler warning options that were
triggering warnings when enabling the wayland build due to missing
header includes. This adds those header includes in.

Reviewed-by: Robert Bragg <robert@linux.intel.com>
Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>

include standalone cogl-wayland-server/client headers

Because the wayland-server-protocol.h header includes symbols that
collide with wayland-client-protocol.h Cogl now provides top level
<cogl/cogl-wayland-server.h> and <cogl/cogl-wayland-client.h> headers so
that applications can ensure they only include one of the wayland
protocol headers in a particular compilation unit. This updates clutter
accordingly to include those headers.

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

wayland: Only try and resize the framebuffer if there is a valid framebuffer

docs: Reorganize the API reference index

docs: Fix typo

docs: Add a section on actor animations

Detail the implicit and explicit animation models used by ClutterActor.

property-transition: Remove animatable from the ctor

Should not have been there in the first place: the animatable will be
set either using ClutterTransition API, or when adding the transition
to a ClutterActor.

Use g_clear_object() instead of unref() + NULL

actor: Clarify and improve memory management of transitions

When adding a transition to a ClutterActor, the actor should hold a
reference on it, and release it only when we remove it. This makes
transitions just like other objects held by ClutterActor.

Post-branching version bump to 1.11.1

Update Visual C++ property sheets

-"Install" the newly-introduced headers
-Fix the situation when the Clutter sources are not unpacked in the root
 folder of a drive (ex: Clutter is unpacked in c:\blah or d:\blah, instead
 of c:\ or d:\)

Update test-interactive Visual C++ projects

Link to GDK-Pixbuf as well.

cookbook/examples: Use real implicit animations

Instead of clutter_actor_animate().

timeline: Increment current-repeat after ::completed

The ::completed signal emission is part of the current cycle; repeating,
like the automatic reverse of the timeline's direction, happens after
the ::completed chain of handlers has been called.

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Post-release version bump to 1.9.17

Release Clutter 1.9.16 (1.10 release candidate 1)