actor: Fix add_child_at_index() for negative index

There is a typo in the check for a negative index: the index variable
should be index_, not index - unfortunately, the latter can still be
resolved to index(3), so compiler and linker are perfectly happy.

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

Updated Serbian translation

text: Avoid changing the contents when possible

An editable ClutterText will reset the selection and cursor whenever the
contents are changed — even if those contents are the same. As this may
confuse the user, we should check if we're setting the exact same string,
and bail out if necessary.

text: Fix annotations

Both set_text() and set_markup() have relaxed their preconditions on a
non-NULL string as their argument, so we need to update the annotations.

text: Style cleanups

Inline wrapper functions, and remove deep nested if's.

interactive/text-field: Modernize

text: Add the coords_to_position() method

The reverse of position_to_coords().

While providing documentation on how to implement it using the
PangoLayout API, I realized that the verbosity of it all, plus the usage
of the Pango API, was not worth it, and decided to expose the method we
are using internally.

docs: Fix up Since annotation

We only use stable releases for "since" tags.

Post-release version bump to 1.9.11

Release 1.9.10 (snapshot)

docs: Update release notes

Fixed clutter_state_remove_key_internal() to propagate the is_inert state

This was causing warnings when calling g_object_weak_unref on an
already finalized object.

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

Fix Makefile to properly include clutter-backend-eglnative.h in the disted tarball

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

model: Drop GValueArray usage

We can use a plain C array of GValues when deserializing a ClutterModel
implementation from a ClutterScript description.

list-model: Migrate from GValueArray

GValueArray is on its way to deprecation in GLib; as far as the
ListModel class is concerned, a plain C array of GValue is a perfectly
suitable replacement for the GValueArray usage. It actually is an
improvement, given that it's going to take less memory.

Clean up clutter-actor.h

It's time, we delayed far too long.

interactive/actor: Show event and animation API

docs: Clarify actor map and unmap methods

ClutterActor stopped requiring to override the map and unmap virtual
functions some time ago.

Now that ClutterActor implements the Container interface, overriding map
and unmap to control the MAPPED state of the children is pretty much
going to be a source of bugs and misunderstandings.

Plus, the ordering of the unmap, destroy, dispose, and finalize calls
should be be documented properly.

The documentation should clarify all that.

docs: Update the actor invariants

• Clear up what's deprecated.
• Remove mentions of set_parent/unparent, and use add_child/remove_child
  instead.
• Clarify that reparent may not touch the MAPPED state.

introspection: add some missing annotations

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

Clean up the clutter API reference Makefile.am

docs: Add an image for the XIncluded code

docs: Add private macro CLUTTER_PRIVATE_FIELD

docs: XInclude test-actor inside the Actor's description

docs: Add an example to the Actor description

actor: Add diagnostic message for "behaviours" in ClutterScript

Behaviours have long since been deprecated; we should notify the user
that still uses behaviours in ClutterScript definitions.

actor: Do not unmap/unrealize twice on destruction

When calling clutter_actor_destroy(), ClutterActor calls
update_map_state() on itself to unset the REALIZED and MAPPED states,
prior to running the dispose() implementation.

The default dispose() will call remove_child() (either directly or
through the Container implementation), which will check for the MAPPED
state and then run update_map_state() again. We use the previously set
MAPPED state to decide whether or not the parent should queue for a
relayout/redraw when removing a visible children.

If the MAPPED flag was cleared prior to remove_child(), though, it'll
always be unset by the time we get to remove_child(), and this will
cause missing redraws/relayouts; we were ignoring this prior the
post-First Apocalypse changes because we were doing:

  if (was_mapped)
    clutter_actor_queue_relayout (parent);

  clutter_actor_queue_redraw (parent);

which is obviously wrong. Once I removed that glaring brain damage from
the remove_child() implementation, bugs started appearing — bugs that
were probably the reason why we introduced that brain damage in the
first place, instead of checking the source of those bugs.

The obvious fix is to avoid clearing up the actor's state on destroy()
until we remove the actor from its parent. This also reduces the amount
of work we do, and the code paths that can potentially go wrong.

actor: Move ClutterShader-related code out of clutter-actor.c

Since the code dealing with ClutterShader is pretty self-contained, now,
we can safely move it outside of the main ClutterActor source file and
into its own. This will allow us to just drop a bunch of files when
branching for 2.0.

x11: Remove CLUTTER_DISABLE_DEPRECATED usage

Switch to CLUTTER_DEPRECATED and CLUTTER_DEPRECATED_FOR.

texture: Deprecate YUV setter

The YUV support depends on the driver support, and not only not many
drivers support YUV natively: the supported colorspaces are pretty much
useless.

The proper way to do YUV to RGB colorspace conversion on the GPU is to
use a fragment shader; for that, ClutterTexture and Cogl provide enough
API to achieve a good result - see the Clutter-GStreamer implementation,
for instance.

texture: It's bytes per pixel, not bits

Clarify the error message when checking the bpp argument.

Use ClutterActorIter inside layout managers

Whenever it is possible, or convenient.

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Updated Slovenian translation

docs: Mention the DELEGATE_LAYOUT flag in set_allocation()

With code examples.

group: Set the NO_LAYOUT flag

Since FixedLayout won't do that for us any more, and we have Group users
and Group subclasses that may be relying on it.

fixed-layout: Remove the NO_LAYOUT flag set

ClutterFixedLayout is the default layout manager for ClutterActor.

Existing subclasses of ClutterActor will get a fixed layout manager
regardless of whether they are going to use it, but since it sets the
CLUTTER_ACTOR_NO_LAYOUT flag, it will introduce regressions on actors
that perform their own layout management.

The CLUTTER_ACTOR_NO_LAYOUT flag was a bit of a mistake in the first
place, as it was introduced as a last minute workaround in the 1.0
process to deal with broken stuff in Moblin. It's going to be a target
for deprecation towards a removal when we start the 2.0 process.

Fix spelling in Clutter Cookbook: ouest -> west

box: Use the ActorIter API

And remove a useless override of the pick() virtual function while we're
at it.

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

stage: Use the iterator API instead of the DOM one

Whenever we're iterating over the children of the Stage we can now use
the ClutterActorIter API.

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

actor: Add ClutterActorIter

Iterating over children and ancestors of an actor is a relatively common
operation. Currently, you only have one option: start a for() loop, get
the first child of the actor, and advance to the next sibling for the
list of children; or start a for() loop and advance to the parent of the
actor.

These operations can be easily done through the ClutterActor API, but
they all require going through the public API, and performing multiple
type checks on the arguments.

Along with the DOM API, it would be nice to have an ancillary, utility
API that uses an iterator structure to hold the state, and can be
advanced in a loop.

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

box: Restore the ::destroy handler

During the gutting of ClutterBox, the destroy and dispose implementation
were removed. The former, especially, destroyed all children - which
usually meant that the redraw queues for the childre was cleared as
well. The removal introduced crashes when a Box was destroyed while its
children were still queueing redraws.

conform: Avoid a deadlock

Something is causing a deadlock when using clutter_threads_* API inside
the offscreen redirect conformance test. The conformance tests are
pretty insane anyway, so for the time being, let's put g_timeout_add()
back in while we figure out the issue.

tests: Update to use the symbolic button names

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

docs: Update to show the symbolic button names

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

drag-action: Use the symbolic constants for the button

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

event: Add symbolic names for left, middle, and right buttons

Symbolic names are better than magic numbers, even if they are
well-established and won't likely change.

This maps to a commit in GTK+ that introduced the same names; it
was decided to go for PRIMARY, MIDDLE, and SECONDARY because of
the confusion that may arise when the button order gets flipped
in left-handed configurations - the "left" button (i.e. 1) becomes
the right-most button, and the "right" button (i.e. 3) becomes
the left-most button.

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

doc/cookbook: Remove dead link

Remove unused variable

wayland: Correctly support fullscreening before the stage is realized

Rather than just call into wl_shell_surface_set_fullscreen we must repeat all
the steps to correctly fullscreen the surface.

wayland: Force a redraw of the stage to get the new sized buffer attached

wayland: Use new stage state manipulation functions

wayland: Refine fullscreen/unfullscreen to emit the state change events

Also update the code to set the size of the stage to set it to the size of the
output. In future versions of the Wayland protocol we'll get a configure
message advising of us of the size we can be to achieve fullscreen.

wayland: Support programmatically resizing the stage

This will call into Cogl and ask it to resize the framebuffer which will then
update the underlying EGL surface.

wayland: Support setting fullscreen before the stage is realized

wayland: Implement set_fullscreen vfunc in ClutterStageWayland

wayland: Include the Wayland surface and shell surface in ClutterStageWayland

wayland: Save the output mode so that it can be used to fullscreen windows

x11: Unbreak the build

The stage wrapper is on the ClutterStageCogl instance.

Merge branch 'stage-state'

* stage-state:
  docs: Update ClutterStageState flags
  wayland: Use the Stage state tracking
  gdk: Use the Stage state tracking
  win32: Use the Stage state tracking
  x11: Use the Stage state tracking
  osx: Use the Stage state tracking
  stage: Add state tracking

docs: Update ClutterStageState flags

wayland: Use the Stage state tracking

gdk: Use the Stage state tracking

win32: Use the Stage state tracking

x11: Use the Stage state tracking

osx: Use the Stage state tracking

stage: Add state tracking

State changes on the Stage are currently deferred to the windowing
system backends, but the code is generally the same, and it should
be abstracted neatly inside the Stage class itself.

There's also the extra caveat for backends that state changes on a
Stage must also emit a ClutterEvent of type CLUTTER_STAGE_STATE, a
requirement that needlessly complicates the backend code.

tests/*: Use symbolic constants for sources and events

And make sure to use clutter_threads_add_* instead of the bare
g_*_add().

docs: Use symbolic constants for sources and events

x11/stage: Use symbolic constants for source function

And make sure to use the clutter_threads_add_timeout(), so that the
function is called under the Clutter lock.

Post-release version bump to 1.9.9

Release Clutter 1.9.8 (snapshot)

cookbook: Start migrating to the new API

Drop mentions of deprecated classes and API, and update the inline
example code.

Still some way to go, and the cookbook would probably benefit from
having a recipe on how to use ClutterActor to build a scene.

cookbook: Begin porting examples to the new API

Start dropping the usage of deprecated classes and API.

actor: use paint opacity to paint the background color

docs: Add missing function

docs: Inlined examples break enums gtk-doc

script: Add loading from a resource

GLib has gained support for compiling ancillary data files into the same
binary blob as a library or as an executable.

We should add this feature to ClutterScript, so that it's possible to
bundle UI definitions with an application.

actor: Remove unused function

Override Container inside ClutterStage

The only actor that results in a mix of the old Container API and the
new Actor API is ClutterStage. By inheritance, a Stage is a Group, but
we don't want it to behave like a Group - as it already overrides most
of the Actor API, and the reason why it was made as a Group in the
first place was convenience for adding/removing children.

Given that touching Group to make it aware of the new Actor API has
rapidly devolved into a struggle between a Demiurge that tries to
avoid breakage and a Chaos that finds new and interesting ways to
break ClutterGroup, let's declare API bankruptcy here and now.

ClutterStage should override ClutterContainer methods, and use the
layout management of ClutterFixedLayout as the proper class that it
was meant to be ages ago. Let ClutterGroup rot in pieces.

Allow mixing old and new API without falling apart

Now that we reinstated Group to its "former glory", we need to ensure
that applications using the deprecated containers with the new DOM API
in ClutterActor can actually work - or, at least, not break horribly.

This actually means making sure that ClutterStage and ClutterGroup can
cope with the DOM, while retaining their old implementations, as well as
their bizarre idiosyncrasies and their utter, utter brokenness.

interactive/actors: Update the venerable test-actors

Updated Galician translations

actor: NULL-ify sibling fields when removing

Let's try and avoid leaving around stale pointers.

wayland: Update to changes in the SHM api

The enum value for the SHM formats has changed to be more explicit about the
format of the data.

group: Restore previous implementation

Making Group just a proxy to Actor broke some behaviour that application
and toolkit code was relying on. Let's keep Group around to fight
another day.

This commit fixes gnome-shell as far as I can test it.

stage: Restore the chain up inside allocate()

Group does not have a custom allocate() any more, so it would end up
calling the default allocate() implementation provided by ClutterActor
anyway.

actor: Remove default layout-manager

It's creating more issues than what it's trying to solve. At least for
the time being, let's not set one.

group: Gut the implementation

A Group is a just a ClutterActor with the layout-manager property set at
instance initialization time. It doesn't need anything else from
ClutterActor's vtable, except the slightly custom show_all/hide_all
implementation, and a simplified get_paint_volume.

actor: Move underallocation warning to diagnostic mode

This should keep down the warning messages — at least for the time
being.

actor: Use internal add/remove child in reparent()

Do not use the public API; reparent() used to use unparent() and
set_parent(), so we need to maintain the old behaviour.

Post-release version bump to 1.9.7

Release Clutter 1.9.6 (snapshot)

build: Bump up the Cogl requirement

Clutter requires a new version of Cogl's API, but we never bumped the
requirement inside configure.ac.

stage: Use clutter_actor_set_allocation()

Instead of chaining up, given that we want to bypass chaining up and
just set the allocation. This also allows us to bail out of the
overridden allocate vfunc check, given that we want the default Actor
behaviour to apply - including eventual layout manager delegates.

actor: Fix the has_overridden_allocate check

docs: Add clutter_actor_set_allocation()

symbols: Add clutter_actor_set_allocation()

actor: Maintain behaviour of old allocate() implementations

The usual way to implement a container actor is to override the
allocate() virtual function, chain up, and then allocate the actor's
children.

Clutter now has the ability to delegate layout management to
ClutterLayoutManager directly; in the allocation, this is done by
checking whether the actor has children, and then call
clutter_layout_manager_allocate() from within the default implementation
of the ClutterActor::allocate() vfunc. The same vfunc that everyone, has
been chaining up to.

Whoopsie.

Well, we can check if there's a layout manager, and if it's NULL, we
bail out. Except that there's a default layout manager, and it's the
fixed layout manager, so that classes like Group and Stage work by
default.

Double whoopsie.

The fix for this scenario is a bit nasty; we have to check if the actor
class has overridden the allocate() vfunc or not, before actually
looking at the layout manager. This means that classes that override the
allocate() vfunc are expected to do everything that ClutterActor's
default implementation does - which I think it's a fair requirement to
have.

For newly written code, though, it would probably be best if we just
provided a function that does the right thing by default, and that
you're supposed to be calling from within the allocate() vfunc
implementation, if you ever chose to override it. This new function,
clutter_actor_set_allocation(), should come with a warning the size of
Texas, to avoid people thinking it's a way to override the whole "call
allocate() on each child" mechanism. Plus, it should check if we're
inside an allocation sequence, and bail out if not.

actor: Be more conservative with the default paint volume

Check the allocation, when one is being used to initialize the paint
volume.