[GtkPathBar] Remove unused error argument

It used to be that _gtk_path_bar_set_file() would return an error if
it wasn't able to switch to the specified file, but that hasn't been
the case for a long while now, since the file chooser became async.

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

Avoid shadowing a variable

Move bg_color variable to an inner scope.

[path-bar] Add some comments to explain the async recursion on an I/O callback

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

[path-bar] Only reset the children's styles if the children actually changed

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

Remove unused argument

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

filechooserdefault: Don't unref value twice

Double unref was introduced in eb02dacb370e8b4e6ef62328d0c09673b1234b32

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

treeview: Properly change cursor when row is collapsed

Previously, when the cursor was a descendant of the collapsed row,
the cursor path was set to the collapsed row, but this was not
communicated via cursor-changed events.

treeview: Reorder code a bit

This is mainly in preparation for the next commit, but also to reorder
the code to
1) prepare modification of tree
2) modify tree
3) emit signals

tests: Add new test that is currently failing

treeview: Simplify code

Since the changes in commit 2b3de3dd this simplification is possible.

treeview: Rename function

_gtk_tree_view_find_path() was not a name that not really described what
the function does. And I kept forgetting it. Also, it took the tree view
as an argument and that was completely unnecessary.

tests: Add automatic expanding/collapsing

Makes the test test even more stuff.

tests: Fix the iter stepping code

The old code actually was wrong and could cause assertions and crashes,
in particular when stepping out of a deep tree back onto the highest
layer.

tests: Check that the cursor keeps up to date

This actually demonstrates a bug where the treeview doesn't emit a
cursor-changed signal when the cursor row is deleted.

reftests: Fix test for border-image addition

widget-factory: make one of the two spinbuttons active

themingengine: fix a typo

themingengine: move code to make switch-case more clear

Code is more readable with the default case as the last one in the
switch block.

Updated Russian translation

pathbar: add the linked style class to GtkPathBar

stylecontext: add GTK_STYLE_CLASS_LINKED

"linked" is a style class for linked areas, such as a box containing
buttons belonging to the same control.

gtk/gtkeditable: Some docs fixes

spinbutton: redesign to use an horizontal layout

Rewrite GtkSpinButton to use a touchscreen friendly horizontal layout.
Other features include:
- support for theming buttons with nth-child
- full support for RTL
- use + and - symbolic icons instead of arrows
- general streamlining and cleanup of a lot of related code

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

win32-theme: Make tabs look better

Support border-image in theme_engine_render_frame_gap

This is similar to the normal frame rendering, but has a clip
region applied for the gap.

win32-theme: Add some margins on GtkScale trought to look like win32

win32-theme: Support custom margins in win32 theme parts

Fix up makefile rules for parallel builds

The gtkwin32css.h generation code used a temporary filename that was also
used by some other files, which is racy. Use a unique one.

win32-theme: Make syntax for mixing theme parts more regular

This lets us extend the argument list better

win32-theme: Make notebook tabs slightly better

win32-theme: Fix background colors for new bg drawing

Fix up background-origin now that we correctly handle the background color

css: Always draw the bg color under the bg image

Reftest background-origin with various background-clip values

Update background-area reftest for background-clip

Implement background-clip and background-origin

Add GtkCssArea enum

This will be used for background-clip and background-origin css properties.

Add reftest to test border-style: none

From the css docs at http://www.w3.org/TR/CSS2/box.html:

8.5.1 Border width: 'border-top-width', 'border-right-width', 'border-bottom-width',
'border-left-width', and 'border-width'

Computed value:   absolute length; '0' if the border style is 'none' or 'hidden'

So, if i specify border-style none and a border-width > 0 that should give the
same result as border-width 0.

Make computed value of border-width 0 if border-style none

From the css docs at http://www.w3.org/TR/CSS2/box.html:

    8.5.1 Border width: 'border-top-width', 'border-right-width', 'border-bottom-width',
    'border-left-width', and 'border-width'

    Computed value:   absolute length; '0' if the border style is 'none' or 'hidden'

css: Convert border style parsing to use enums

Rename GTK_CSS_REPEAT_STYLE_NONE to GTK_CSS_REPEAT_STYLE_STRETCH

This is a private type, and the CSS default value is "stretch", which
is how we were parsing and printing NONE anyway.

With this fix we can use the enum stuff for printing/parsing

css: Use enum parsing for background repeat style

css: Break out enum parse/print helper

We need this to parse non-GValue enums

css: Parse enums with new case-insensitive helper

css: Add _gtk_css_parser_try_enum

This helps parsing GType enums in CSS, properly handling
CSS being case insensitive.

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