Add generation of private g-types for private headers

This starts with just the gtkcsstypesprivate.h header

reftests: Make sure border-image-gradient keeps working

.. even with more advanced themes.

tests: Add testtreechanging

It's just a treestore going crazy while the treeview has to keep up.

win32: Make all GDK_WINDOW_TEMP always on top

This makes menus, popups and tooltips show above the windows
task bar.

tests: Add prop editors to testtreeview

It tests treeviews, so we'd better be able to edit them :)

reftests: Mark windows as popups

Improves performance, gets rid of resize grip.

a11y: Implement ref_state_set using the new APIs

Various tiny semantic changes in here.

The big fix is that we consider every cell as visible and as showing if
the treeview is mapped.
This was also fixed in the tree dump test.

a11y: Add get_renderer_state() to cell accessibles

Accessibles can use this function to query the state that their row
would be rendered with.

a11y: Add _gtk_cell_accessible_set_cell_data()

See the function documentation for details.

Also included is the implementation for the treeview, but no users yet.

treeview: Notify a11y about reordered rows

That way, no more signal handler is needed in the a11y code. Plus, we
avoid needless signal emissions for rows we don't care about.

a11y: Don't create fake cell renderers

build: Add gmodule-2.0 dependency for gtk-query-immodules-3.0

win32-theme: Fix up radio and menus in various states

This makes all states in widget-factory look ok

win32-theme: Theme column headers

prop-editor: Implement property editing for GdkRGBA properties

Bug #664467.

Revert "tests: use GdkRGBA instead GdkColor"

This reverts commit 85747da97238dc101f92cd8241aa7aea44f35789.

Conflicts:

tests/prop-editor.c

Bug #664467.

color-button: Notify "color" and "alpha" properties when changing the rgba

Bug #664469.

GtkIconView: Fix a segfault

Keynav on an empty iconview was causing segfaults. This
was reported in https://bugzilla.gnome.org/show_bug.cgi?id=664456

bgo#662814 - Don't print a g_warning() when GtkRecentManager can't find a file

Also, *do* add the file to the recently-used list, even if the file does not
exist yet.  This is used from the Save dialog, so even shitty apps which don't
add the file to GtkRecentManager, will get the file added from the Save dialog.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

widgetpath: correctly print the siblings index

In gtk_widget_path_to_string() we were counting siblings from zero
instead of one, resulting in confusing output.

frame: don't access padding before it's initialized

Move padding initialization before its first access.

gtk-demo: Fix segfault in editable cells demo

The strings can be freed when edting the description column. This does
not work with constant strings.

rbtree: Rewrite to not lose node order

_gtk_rbtree_reorder() was moving the node's data while reordering. As we
use the node pointer in the a11y code as a hash key, this didn't work.

So this rewrite changes that. As a bonus, it is less code and faster.
Woohoo!

tests: Add test for _gtk_rbtree_reorder()

tests: Speed up rbtree test

We don't need so many tree consistency checks while creating when the
test is about something else.

Fix a typo

Revert "Clean up includes"

This reverts commit c5e4af761c9db702b9bbf0740404c204e5b086d2.

Move pangoft2 to Requires.private

We have pango in Requires anyway, and this is just so we
can call pangofc api ourselves.

Clean up includes

There's no sign of Pango in this header, so no need to include
pangocairo.h

rbtree: Remove needless tests

if the checks are run on the nil node, they will get the second test the
  way we expect it anyway, so it's unnecessary.

rbtree: Simplify rotation functions

- Make sure the rotated nodes aren't nil
- Use existing functions for complex computations
- Don't use NULL checks for variables guaranteed to not be NULL/nil

rbtree: Remove the nil member

rbtree: Introduce _gtk_rbtree_is_nil()

Makes for more readable code than "== tree->nil" and allows removing the
nil member later.

rbtree: Introduce _gtk_rbtree_first()

... and use it.

rbtree: Replace nil node allocation with a preallocated nil

This has one major caveat: The new value is const, so read-only memory.
Any attempt to write to it will cause a crash. Note that we are not ever
supposed to write to it, but bugs happen...

Bump version

3.3.4

More updates

tests: Add a test for the rbtree code.

rbtree: Don't write to nil node

The code used to set nil->parent, which could cause segfaults. Don't do
that. We also need to pass the parent explicitly to the fixup code,
because the node during fixup may be the nil node.

rbtree: Add a local variable

This will be needed in the next patch, and I wanted to split that patch
up.

rbtree: Move to an approach where we don't move contents

So instead of copying the children and height to the new node, we keep
the old node and copy all the old stuff to it.

This is necessary so the accessibility code can use the node as a key in
the hash table or store the node as a reference to the row instead of
GtkTreeRowReference. And because it already does that (oops), this fixes
a bunch of segfaults with a11y enabled.

rbtree: Use gtk_rbnode_adjust() even more

Use it for the modified node in gtk_rbnode_remove(). Cleans up the code
quite a lot.

rbtree: Use gtk_rbnode_adjust()

Make gtk_rbnode_remove() use it when unlinking the node.

rbtree: Use gtk_rbnode_adjust() more.

Make _gtk_rbtree_node_set_height() use it.

rbtree: Use gtk_rbnode_adjust() more

Make _gtk_rbtree_insert_before() use it.

rbtree: Use gtk_rbnode_adjust()

Make _gtk_rbtree_insert_after() use it.

rbtree: Split out a common function

gtk_rbtree_adjust() will adjust the summed values of a node and all its
parents in the tree. Currently only implemented by splitting out the
function from gtk_rbtree_free().

rbtree: Don't set variable twice

rbtree: Add assertions to clarify usage

rbtree: Mark all debug sections as debug

rbtree: Make debug functions private

They're not used elsewhere

notebook: use the current page allocation when computing the redraw area

Instead of taking the first page and trying to adjust the rect with
random padding values, take the current page, as it's always guaranteed
to be at least as tall as inactive tabs.
This fixes some annoying 1px drawing artifacts while switching tabs when
the theme disables notebook padding.

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

notebook: properly subtract the initial_gap width from the tab space

Instead of modifying the allocation.

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

Drop uses of @returns syntax

Prevent local filesystem influence in the a11y tests

By using the 'unix' gio volume monitor, we avoid variation
in the file chooser that otherwise breaks the 'pickers' test.

win32: Don't double free stuff when finalizing drag context

We were freeing stuff that was already freed in the parent class
finalizer.

win32: Don't dereference keyboard_grab if it is NULL

This fixes a crash in e.g. testdnd

win32: Make gdk_drag_context_new static

win32: get rid of GdkDragContextPrivateWin32 and related machinery.

This turns GdkWin32DragContext into a proper GdkDragContext subclass.
Because we now correctly initialize GdkWin32DragContext in
gdk_drag_context_new, we no longer crash immediatly when a DnD
operation is initialized (the find_window, drag_status, ... vfuncs
where all pointing to 0x0 instead of their proper win32 implementations).

We now try to consistently refer to GdkDragContext as "context",
GdkWin32DragContext as "win32_context" and the ole2 related
target_drag_context and source_drag_context as "ctx".

Members of GdkWin32DragContext only used by the ole2 DnD codepaths
are now explicitly marked with a ole2_dnd_ prefix.

Convert gtk-win32.css to unix line endings

Add gtk_symbolic_color_new_win32 to gtk.symbols

win32-theme: Fix up docs for gtk_symbolic_color_new_win32

Merge branch 'win32-theme2'

[icons] text-x-generic isn't a generated icon

Calling it one causes maintainer-clean to delete it.

gtk/gtkapplication.c: Fix typo

frame: make sure to allocate the CSS border width

Similar to GtkNotebook, GtkFrame was only allocating space for the
padding width, and not the border.

This could be seen by just running tests/testframe. With a theme that
renders frame borders, setting xthickness = 0 in the test draws the
button border over the frame border, which is wrong.

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

frame: add GTK_STYLE_CLASS_FRAME in _init()

Instead of adding it every time we use the GtkStyleContext, just add it
in _init().

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

notebook: make sure to allocate the CSS border width

Instead of taking only the CSS padding into account when allocating the
notebook children, also allocate the border width.

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

Bug 653676 - Expand/Collapse doesn't respond after one click

Ignore leave-notify-event when mode is GDK_CROSSING_GTK_[UN]GRAB.
Just [un]grabbing shouldn't cause us to [un]prelight the current arrow
and node.

Don't call scroll to cell if the tree view is empty

Account for the case when gtk_drag_finish() didn't actually add a row
(can happen when dragging from an empty tree view to itself ...).

Bug 660554 - gtk_tree_view_drag_begin: assertion `path != NULL' failed

Turned assertion into silent return.

This assertion is only hit when dragging from an empty tree view.  In
this case, gtk_tree_view_begin_drag() is triggered from gtkdnd.c and not
from gtk_tree_view_maybe_begin_dragging_row().  We actually want to
cancel the drag at this point, but that is not possible with the GTK+
API as far as I can see.

The alternative is to not allowing the drag to start.  This could be
done by simply unsetting the tree view as drag source when it is empty
and setting it as drag source again when rows are added.  I didn't
choose to go with this for now, since this will likely break third party
code.

quartz: fix coding style in quartz-version of gtk_drag_begin_internal

quartz: do not call gdk_drag_begin() twice

This makes drag and drop work again on Quartz.

[Bug 664238] GTK apps crash when dragging something

Corrects a bad condition in a test in 085b98f4

Updated Spanish translation

Adapt iconview a11y test output

Improve AtkText implementations

There was some regressions from the recent display line fix;
while fixing it improve the test coverage and make GtkEntry
and GtkTextView return identical results.

Remove leftover debug spew

Add a missing static

Fix the build

Updates

Drop the Beagle search engine

It is not actively maintained.

Fix gdk_cairo_region_create_from_surface on big endian

gdk_cairo_region_create_from_surface doesn't work correctly on PPC.
This is most prominently seen with the GTK window resize grip, the
shape of which is mirrored every eight pixels horizontally.

At the same time, use an A1 surface for the resize grip shape to
eliminates an A8->A1 surface conversion.

win32: Default to gtk-win32 theme

Build in default win32 css as gtk-win32

Build gtkwin32css.h from gtk-win32.css

win32-theme: Add gtk-win32.css

gdk: remove the GET_EFFECTIVE_KEYMAP() stuff from gdkkeys-x11.c

We don't support passing a NULL keymap any longer. Also pull some
precondition checks into the parent class.

gdk_x11_keymap_get_modifier_mask(): always return a value

Turn the private #define for the group-shifting modifier into API

Add GDK_MODIFIER_INTENT_SHIFT_GROUP to enum GdkModifierIntent
and handle it in gdk_keymap_get_modifier_mask(). Add an X11
impl of the method and return keymap_x11->group_switch_mask.
Return 0 from the default impl because we don't know.

Bug 663856 - Make option-foo accelerators use the right symbol

If the keyboard group shifting modifier is *also* a normal
accelerator modifier, we need to special case it when calling
gdk_keymap_translate_keyboard_state(), so we get the right
key symbol for accelerators (for example we want Option-O,
not Option-Ø displayed in menu items). This patch should only
affect quartz where the Alt key both shifts the group and can
be used as accel modifier, and not X11 or Win32 where AltGr
is not used for accelerators.

- fix quartz' gdk_keymap_translate_keyboard_state() to return
  the right consumed_modifiers
- add _gtk_translate_keyboard_accel_state() which does the
  special casing
- use it everywhere instead of gdk_keymap_translate_keyboard_state()

Fix tiny docs typo.

win32-theme: Support mixing parts

We need this to do something about the non-existing inconsistent
radio button state in win32.

Add support for win32 theme colors as symbolic colors

Add pulse style class and use in progress bar and entry

This is used for indeterminate progress reporting.

Set top/bottom style classes on spinbutton buttons

Add top/left/bottom/right style classes to steppers

This is needed for e.g. win32 theming, but is also generally
useful.

win32-theme: Support multiple parts being combined

This is needed for e.g. scrollbar sliders

Render background image if set for checks and options

The default theme engine draws a fallback check/radio image, but
doesn't let you replace this. We now check if a background image
is set and if so render that instead of the default fallbacks.