a11y: Redo cell update function

1) always emit signals
   Previously, newly constructed cells would be told to not emit events.
   However, we can ensure that nothing is connected to the signals, so
   they will not actually emit anything.
2) don't return anything
   The return value is unused anyway.

a11y: Remove unused function

a11y: remove now unused property lists

a11y: Don't copy data between renderers

Now that we don't use custom renderers anymore, we don't need to copy
data between them anymore.

a11y: Use cell renderer creation funcs unconditionally

Also delete the previous "new" functions and actually set the real cell
renderer here.

gtk: Add accessible types for cell renderers

a11y: Use the accessible type

cellrenderer: Add API to keep the accessible type

It's private for now, because we require a GType that isn't exported in
the API yet.

API: cellrenderer: Add a private struct

a11y: Add GtkRendererCellAcessible::renderer

a11y: Remove unused variable

treeview: Send changed signal directly to a11y code

No more signal handler is needed, therefore the code can also get rid of
tracking the treemodel. And we use a faster approach for iterating the
changed cellrenderers: We just iterate all columns instead of over all
cell accessibles, as that number is likely quite a bit smaller.

tests: Add test resetting the model

We first set a NULL model and then reset the old model, just to get the
effect of clearing and then resetting.

We reset the cursor and selection afterwards, so the reset doesn't
destroy all the work we did.

treeview: Emit cursor-changed signal when new model is set

a11y: Improve treeview's add/remove_selection()

a11y: Improve get_selected_rows()

a11y: Improve table_is_row_selected()

treeview: Signal selection changes to the accessible

And another signal gone.

a11y: Emit children-changed properly for treeviews

Fixes the patch reverted in b7e74ef95f1d9cd851fb81a124beca0ca11dad00
properly.

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

treeview: Remove unnecessary accessible_expanded()

Instead, just call remove_state().

treeview: Add add() function to accessible

... and use that to emit the insertion signals.

treeview: Set expandable flag from treeview

... instead of trying to update it manually.

a11y: Redo cell accessible action handling

Multiple changes:
- actions are now available unconditionally, but only work in the right
  state. This mirrors other actions.
- instead of adding actions manually, they invoke the action on the
  GtkCellAccessibleParent interface.

Tests have been updated to reflect the changes

a11y: Remove "toggle" action from cell renderers

This is identical to "edit" (even though that name is kinda confusing).

treeview: Set EXPANDED state directly on a11y elements

treeview: Deprecate gtk_tree_view_set_destroy_count_func()

The accessibility code doesn't need it anymore.

a11y: Remove destroy_count_func usage

It's not needed anymore.

a11y: Emit row-deleted signal when rows get deleted

Instead of in the old callbacks.

a11y: Revert 22a47c3dbd7875af10df58feedd81d7b661222b6

The fix is wrong on multiple levels. I'll add a proper fix soon.

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

treeview: Don't use signals in a11y row expansion

Instead, add a function and call it directly.

a11y: Add _gtk_tree_view_accessible_add_state()

... and _gtk_tree_view_accessible_remove_state(). They should be called
by gtktreeview.c to notify about changes of state for a row.

a11y: Split out function

I want to use that function when sending notifications. See next
commits.

a11y: Readd FOCUSABLE state

The state was lost in commit 08d86fabc4237092bf04a97795498f2c45575085.

Also order the states that are always set alphabetically.

a11y: Remove old notify about showing/visible

As cells are always visible and showing, there is no need to do complex
notification mechanisms anymore.

a11y: Add _gtk_cell_accessible_state_changed()

It's supposed to replace the old state_set calls.

a11y: Split out renderer<->atk state matching

We now keep a state map. This makes adding new states easier, and
doesn't introduce bugs with state change emissions.

a11y: Don't duplicate information anymore

We do provide the expandable and expanded information via
GtkCellRendererState now so there is no need to use extra flags.

a11y: Set new treeview flags

treeview: Set the new flags when rendering the cell

cellrenderer: Add EXPANDED and EXPANDABLE states

These make more sense as flags than they do as properties of the cell
renderer.

tests: Add a test for latest commit

widget: Don't change sensitive state with insensitve parent

This way child.set_sensitive(True) with an insensitive parent doesn't
cause child to become sensitive.

fontchooserwidget: Handle dpi < 0

Just use 96.0

Add new symbols

iconhelper: treat the empty string as a NULL icon name

Treat the empty string as if it was a NULL icon name instead of
rendering GTK_STOCK_MISSING_IMAGE.
This used to be the previous behavior of GtkCellRendererPixbuf and
applications might still rely on it.

cellrendererpix: save the icon size locally instead of in the helper

We need to save the icon size locally for GtkCellRendererPixbuf, since
its properties are set again every time it's drawn (resetting the icon
size on the helper by means of _gtk_icon_helper_clear() each time).

menuitem: cleanup arrow-spacing fetch code

Refactor get_arrow_size() to return the spacing as well, since basically
we always need both.

menuitem: share code between HFW and non-HFW height requests

The code is very similar; having two slightly different code paths is
bad and can lead to bugs. Refactor the code to use the same height
request function.

menuitem: properly respect padding when drawing the separators

Use the padding when calculating the draw coordinates of the separator.

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

menuitem: force odd height if we're not using wide separators

So that the 1px line can receive the same amount of space above and
below.

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

menuitem: don't request twice the padding for separator menu items

Padding was requested twice for separator menu items.

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

demo: Add test for hide-titlebar-when-maximized

Add an additional option to the appwindow demo to test the new
gtk-hide-titlebar-when-maximized setting.

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

window: Add hide-titlebar-when-maximized property

For maximized windows, titlebars cannot be used to reposition or
scale the window, so if an application does not use it to convey
useful information (other than the application name), the screen
space occupied by titlebars could be put to better use.
Add a new window property which requests from the window manager
to hide titlebars when windows are maximized to account for this.

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

scale: remove unused variable

widget: Synthesize crossing only if sensitivity changed while propagating state

Add has-origin property for GtkScale

If the scale has an origin (it will have one by default), GtkRange will
render the two sides before/after the current value with different style
classes, making it possible for themes to use different colors and
properties for the two areas.
This was possible in GTK 2 with style details, but got lost during the
road to 3.0.

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

wayland: Add dummy implementation of the wl_data_device functionality

Setup listener functions for the drag and drop events as well as the selection
events. Then create and save a data structure representing the data offer from
the other client.

wayland: Add an implementation for GdkAtom

This implementation is based on a hash table. The hard coded GtkSelection
atoms are preloaded into the hash table at the correct values. User generated
atoms start after the last preloaded atom.

wayland: Get the data device during input device setup

The data device is a associated with the input device - fetch the data device
from the data manager for the current input device.

wayland: Retrieve the data device manager object

The data device manager is a global object that provides the support
infrastructure around data devices. These data device objects are the basis
for handling drag and drop as well as selections in Wayland.

wayland: Don't use deprecated function: gdk_window_get_pointer

Replace it with gdk_window_get_device_position.

Updated Slovak translation

print dialog: Fix authentication logic

Reset state of CUPS requests correctly during authentization and
check CUPS requests for errors.
Don't initialize variables holding password and username with empty
string (#664640).

Fix nicks for the window class enum

As pointed out in bug 665999, these were just not right.
Before this commit, the nicks were 'output' and 'only'.
After this commit, they are 'input-output' and 'input-only'.

Fix set_focus_child handling during focus changes

In many cases we used to set focus_child to NULL all the way up
to the top and then to the right value, even if there was
a common ancestor, meaning these see a temporary NULL value for
focus_child. Only when the new focus widgets direct parent was
in the previous ancestor list did we stop early.

This fixes that by always stopping propagation when reaching
the common ancestor.

GtkFrame: Fix a casting in the compute_child_allocation() method

The casting used to calculate the child allocation is confusing MAX().
As a result, width and height end up with negative values, which makes no sense.

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

themingengine: split a GtkThemingBackground object

Allows easier modification and general sanity of the CSS background
rendering code.

image: take misc padding into account when calculating x/y draw coords

This fixes the misc-alignment.ui reftest.

image: add gtk_image_get_preferred_size()

Add a function to query the preferred size of the GtkImage, and use it
in the get_preferred_width/height implementations.

treeview: Ensure selection's changed signal

There was a corner case where the changed signal was not emitted.
If rows were built like this:
  1    (not selected)
  + 2  (selected)
  + 3  (not selected)
And row 1 was removed, no signal would be emitted.

I like it when writing tests actually finds bugs that have been around
since 2003 - introduced by 4a03ea2334bde7d2e15d0933c5a78f5a2338a854
actually. :)

treeview: Don't try to optimize away selection_changed signal

We get certain cases, in particular with SELECTION_MULTIPLE, where we
cannot figure out in advance of real_set_cursor() if the selection will
actually change.

tests: Add select/unselect calls

treeview: Add a CURSOR_INVALID flag to set_cursor_row()

This is necessary so the code doesn't attempt to do things with the
previous cursor, which currently cause crashes.

treeview: Replace lots of boolean args with flags

I want to add more and 3 boolean args is essentially unreadable code.

tests: Add sanity checks for selection

We want to know that the selection emits "changed" when it changes.

docs: Update code demo for newer glib

tests: Dump the operation that is performed to stderr

tests: Make treechanging test run as fast as possible

treeview: Handle the case where the cursor row gets deleted

Previously, the cursor would just become invalid, which used to
reselect the first row in the treeview later on (without a
cursor-changed signal). This leads to a crash now with the recent
refactorings.

The patch is longer than I'd like it to be, but the situation is kinda
complicated, because we want to make sure to move the cursor to a good
row. It works like this:
1) From the deleted row, search forward for the first row that is not
   going to be deleted and is not a separator.
2) If no such row exists, search backwards for a row that is not a
   separator.
3) If no such node exists, clear the cursor.

treeview: Replace cursor handling by keeping the node

Previously the code used a GtkTreeRowReference, which was (a) less
performant and more importantly (b) hiding errors.
The errors being hidden were the referenced row becoming invalid or
collapsed, because such rows would not be valid cursor rows and it would
be necesary to select a new row and emit a "cursor-changed" signal.

So if a crash bisects down to this commit, it is very likely that the
cursor has not correctly been updated and the cursor row is invalid.

tests: Put random number into local variable

This way, we can see what function had previously been called when the
checks fail.

treeview: Use _gtk_rbtree_contains() in a11y code

rbtree: Add _gtk_rbtree_contains()

It's useful in a bunch of places, so split it out

Fix move-resize emulation with XI2

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

wayland: Load cursors from wayland directory under XDG_DATA_DIRS

This means we don't have to have hardcoded "/usr/share/wayland" to find the
cursors.

This change also fixes up the warning messages for when loading fails.

Updated Breton translation

Updated Breton translation

Revert 3fd7dbc94074826755da7fc3237e439ba6e81ada

Calling gdk_keymap_add_virtual_modifiers causes _all_ virtual
modifiers to be added, which causes problem when they are co-located
on the same real modifier (as Super and Hyper often are). Effectively,
this made it impossible to enter key combinations involving Super,
since they all turn into Super+Hyper.

Test some virtual modifiers

This adds some accelerators involving Super and Hyper to testmerge.
This helps in testing our accel matching for those.

print to file: fixed lost filename in gui

Stop widget creating an event before updating both directory and
filename values.

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

overlay: Properly allocate the child upon addition

This ensures that after addition the child gets assigned the proper
allocation and the child window is properly shown/hidden.

overlay: Move code

Move code for child allocation into the child allocation function. Don't
keep it in the overlay allocation code.

See the next commit for why this is useful.

overlay: Create child window before setting parent

Otherwise, the window will not be used when set_parent() maps the child.

progressbar: Set state flags on draw

We were missing setting state flags on the style context before drawing.

widget: Unset window-unfocused in gtk_widget_unparent()

Widgets without a parent aren't inside a toplevel window so we must remove
window-unfocused as it doesn't make sense outside a toplevel.

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

treeview: Fix crash when scrolling to end

This broke in a4630d0e7b233479825b059e0df0e6d65b0e6734

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

window: copy back window surface data when the implicit paint is flushed

When an implicit paint is flushed during expose, e.g. because a
non-double buffered widget is painting, make sure to copy the existing
data from the window surface we rendered before flushing back to the
paint surface, instead of using an empty base.
Code was already handling that (and said so in the comment), but only
when no implicit paint was used at all, and not in the case when it's
flushed mid-expose.

window: trivial cleanup

window: remove unused arguments and variables

Trivial cleanup, no functional change.

modifierstyle: Fix compile errors

Introduced in 7bd2446801ffd0fc2ac5cc98fcd675ac4f3935d6