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.

win32-theme: Support -gtk-win32-size CSS value

Add initial cut at win32 theme support for CSS

We now support -gtk-win32-theme-part(class,part,state) in background
and border-image CSS properties. This renders the corresponding
theme part using DrawThemeBackground() and acts as a base for a
CSS based windows theme.

Note that we build the parsing code even on non-win32 so that
all themese will parse the same on all arches. We draw pink instead
of the actual theme parts on non-win32 though.

make _gtk_style_property_resolve copy the result to an output value

This is preparation for allowing it to return a newly created
value, rather than just copying one.

Actually pass in a StylePropertyContext when getting images

This is in preparation for getting size-dependent results back
with the win32 themes.

Remove _gtk_style_properties_peek_property

We want to move to a world where getting style properties can
allocate a new value, depending on the style property context.
In this world we can't expose "peeking" a property, as we neet
to return a ref to the property that is newly created.

So, we move the peek code into get_property and use get_property
from get_valist.

Add GtkStylePropertyContext and use it in when getting style properties

At the toplevel we have _gtk_theming_engine_get, which lets us pass
in a property context with (atm) the size to get the property for.
Then there is a lot of plumbing to push this down into the lower
layers of the style property code until finally hitting
the property resolvers.

I need this because I will be adding a property resolver for win32
theme parts, and they render differently depending on the size
(i.e. they don't scale linearly). The idea is that the code
to get the background properties will pass in the final size
and we will resolve the theme part specification to that particular
size.

If the old non-context calls are used we just hardcode a size
of 100x100.

Save a generic boxes source in GtkImageBorder

We used to special handle gradient, but we want to be able to store
other sources (that will eventually resolve to cairo patterns).
For instance, this is needed to handle win32 theme part sources.

win32: make fixup_event correctly ref all GdkWindow members

This fixes some crashes related to owner changes and selections.

ref, don't unref, requestor member when copying GdkEvents

This seems like a typo, we should obviously ref the member when
copying. Apparently nobody copied GDK_SELECTION_* events...

notebook: don't leak the action widgets

The notebook owns them, so it must destroy them in destroy().
(cherry picked from commit 87ebfb19fd9422de1d778396770c02e91e6481e8)

viewport: Get rid of deprecation warnings

viewport: Don't use deprecated functions

viewport: adjustments are never NULL

So there's no need to check for that in the getters.

viewport: Initialize the adjustment in the init function

viewport: Don't unset the adjustment on finalize

We unset it on destroy already. This makes it consistent with all other
scrollables.

treeview: fix the build

treeview: use gtk_tree_view_ensure_background() consistently

In gtk_tree_view_state_flags_changed() we were setting the background on
the bin_window without the necessary "view" style class, making the
treeview render with the wrong color in some circumstances (such as when
adding an empty treeview in a window, as spotted by Brian Smith).

treeview: factor out gtk_tree_view_ensure_background()

We'll use it shortly.

Maintain map/unmap invariants in GtkRecentChooserDialog

We used to explicitly map and unmap the child GtkRecentChooserWidget when
mapping and unmapping the dialog, respectively. Now that GtkContainer actually
unmaps child widgets (instead of avoiding that), we can assume that the
child GtkRecentChooserWidget will be unmapped when we want it to be.

This fixes a warning from gtk_widget_verify_invariants(), as we were mapping
our child widget before calling our parent class' ::map() handler. Bug #659257.

tests: Add tests for keypad keys parsing

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

gtk: Parse keypad numbers correctly

Try to find keycodes for keysyms at level-0 in the keymap, but don't
fail to parse if we can find those in level-1. This fixes the
parsing of numerical keys on the keypad, which require a shift level
through Num-Lock.

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

tests: Check "without keycode" code path too

And see whether the output of gtk_accelerator_parse()
matches that of gtk_accelerator_parse_with_keycode()

Revert "tests: Disable accel test until fixed"

The test now works correctly.

This reverts commit ba397a3431ee3850a72d8eab394b7ee62e65b737.

Update comment to new realities

a11y: Merge function into only caller

Also cleann up comments that are misleading now.

treeview: Call a11y functions for culmn changes directly

This way, the a11y code knows if a column was reordered, added or
removed and can do the right things instead of trying to guess and
getting it wrong.

Also, this patch finalizes the changes so that only visible columns
exist to the accessibility interface.

a11y: Unify column-by-index getting

We are only ever interested in visible columns. Invisible columns do not
exist for all a11y cares.

tests: Add an invisible column to the tree test

This one should be invisible if everything works as it should.

tests: Add checks for row/column index to accessibility-dump

tests: Don't dump stuff twice for tables

Captions and headers are printed as part of the children, so we can
avoid the duplication by just printing the name.

a11y: Use gtk_tree_view_get_n_columns()

In fact, invent our own function get_n_columns() which actually only
counts the visible functions, because that's the only ones we care
about.

The places where it's not used yet will be changed in the coming
patches.

API: treeview: Add gtk_tree_view_get_n_columns()

a11y: Fix get_column_index()

a11y: remove argument from get_column_number()

The argument was always FALSE.

treeview: Simplify a check

Use the same method as elsewhere.

treeview: Minor beautification

tests: Up the ante

Put 10000 rows into the tree-performance test, so the tests don't finish
in <1s. That's too little for profiling.

a11y: remove useless argument

The argument is unused these days, so don't confuse people with it.

a11y: Remove clean_rows function

It doesn't do anything anymore.

a11y: Don't crete cell infos for non-children

When we use a container inbetween, this ensures that the cell infos
don't replace each other in the hash table.

a11y: Make find by index lookup use the hash table again

This should give back the performance that was lost with the commit
"a11y: Make the cache lookup function inefficient".

a11y: Add a hash function for cell infos

Note that comparing the tree is not necessary as the nodes are already
unique per row.

a11y: Compute the cell index quicker

We have the node available, so we can compute it from there.

treeview: Add _gtk_rbtree_node_get_index()

.. as a replacement for _gtk_rbtree_node_find_parity(). Instead of 1 or
0, the function now returns the index of node in the complete tree
(counting from the root). And this is of course identical to the row
number.

a11y: Compute index the easy way

index is row_index * n_columns + column_index

We use it everywhere, why not here?

a11y: Redo indexing

We now index rows by actual expanded row number and don't count them
ourselves no more.

tests: Ensure the treeview is expanded

Otherwise the test code might ignore collapsed rows, which makes it go
rather fast.

treeview: Redo row tracking

Track the RBNode/RBTree instead of keeping a TreeRowReference. This is a
whole lot faster and less error-prone.

Also, notify the accessible of removal of rows before actually removing
them, so we have a chance to clean up.

widget: Add _gtk_widget_peek_accessible()

This function returns the accessible if it already exists. This way we
can call functions on the accessible from the widget itself instead of
having to rely on signals.

a11y: Remove needless checks

The cell_row_ref and cell_col_ref variables are always non-NULL, so
there's no need to check it.

a11y: Add a cell_info_get_path() function

Reads better and we only need to change it in one place.

treeview: Remove unnecessary variable

tmptree has the same value as tree everywhere, so just use tree instead.

a11y: Mark object as defunct in destroy notify

Instead of requiring a special functio to be called before just removing
from the hash table. This simplifies code a lot that used to remove from
the hash table.

a11y: Remove in_use member

After the latest changes, it is now always TRUE, so adjust the code
accordingly.

a11y: Don't garbage collect anymore

Instead, remove cell infos immediately from the hash table.

a11y: Improve cell_infos table

- Name it properly
- Make it just a set of cell_infos

Currently it's using direct hash, but as long as we don't actually
lookup stuff from it, who cares...

a11y: Implement find_cell_info() using qdata lookup

That should be orders of magnitude faster then iterating over a hash
table.

a11y: Store the cellinfo in the cell's qdata

a11y: Remove index from cellaccessible

That way we also get rid of the refresh_index function.

a11y: Don't use the cell's index anymore

We implement get_cell_index() now, so it's no longer necessary.
As a side effect, we need a different index for our hash table (which
now has a wrong name, but that will soon change).

a11y: Implement get_child_index in treeview

a11y: Implement get_child_index in treeviewaccessible

gtk: Fix compiler warnings from include fixes