wayland: Set a default cursor when the window cursor is set to NULL

This change follows on from a change in semantics in Wayland where calling
wl_input_device_attach with nil would make the compositor set the pointer
sprite to it's default cursor sprite.

GtkPrintUnixDialog: Hide unused tabs

Don't show tabs which are empty. This is related to
the recent GtkTable -> GtkGrid conversion. (#663479)

Revert "gtksettings.c: Do not use GdkColor API"

This reverts commit baef3e5f2470cb727407e45a655cec59ebd4a9e5.

This broke the Adwaita theme pretty severely, so I don't think it can
be right. See: http://i.imgur.com/Rq7wc.png

gtksettings.c: Do not use GdkColor API

gtkcolorsel.c: Do not use GdkColor API

gtk/*: Use g_list_free_full() convenience function

modules/*: Use g_list_free_full() convenience function

gdk/*: Use g_list_free_full convenience function

tests: Use g_list_free_full() convenience function

GtkComboBoxText: allow id='' tag in builder markup

Presently, GtkComboBoxText lets you list the items for it in the
GtkBuilder markup like so:

<items>
  <item translatable='yes'>Red</item>
  <item translatable='yes'>Blue</item>
</items>

but has no way to set the "id" column for added items.  Add an id='' tag
for that purpose so that you can do:

  <item id='red' translatable='yes'>Red</item>

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

GtkNotebook: fix crash when DnD tabs between windows

When doing DnD between windows the tab label is set as the child of
the DnD info window. If this is happening the remove method of
GtkNotebook should not unparent the tab label, since it's been already
unparented, belongs to the DnD window and will be properly destroyed
when the DnD is over.

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

gtk/gtkcolorbutton.c: Do not use deprecated GdkColor api

gtkspinbutton: Respect value of inherited 'editable' property

When this property is set to 'FALSE', user's click on the buttons
should not actually update the value but rather result in error bell.

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

treeview: bgo#666685 - Crash when toggling the selected status of a row

The refactoring from e01af5c5c seems to have caused this inadvertently.

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

gtkcolorsel: Do not use deprecated GdkColor api

gtkcellview: Do not use deprecated GdkColor api

gtk/gtkprivate.c: Include stdlib.h

Include stdlib.h for getenv to avoid C4013/implicit declaration of ...
warnings

test/testgtk.c: Fix compilation warning

Fix problems with DND on some X servers

When the X server does not support the shape extension (as some
vnc implementations seem to), our DND code was always seeing
an empty input shape, so drops always missed their target.

http://bugzilla.gnome.org/show_bug.cgi?id=620240

Add symbolic constants for event emission control

Similar to https://bugzilla.gnome.org/show_bug.cgi?id=631413 in GLib,
GTK+ should have symbolic constants for controlling the event handling
instead of a mere boolean value.

The GDK_EVENT_PROPAGATE and GDK_EVENT_STOP constants map to FALSE and
TRUE respectively.

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

textdisplay: use g_list_free_full

comboboxtext: Set entry-text-column and id-column props in the constructor

This is cleaner than setting the value in the _new() functions and
makes binding and GtkBuilder instantiation easier. Bug #612396, comment 65.

Bug 667018-gtk/gtkwindow.c: Include deprecated/gtkstyle.h

This is needed for the prototype of gtk_widget_reset_rc_styles, to avoid
C4013/implicit declaration of ... warnings of that function

gtk/gtkspinbutton.c: Remove lingering ';'

...so that C89 compilers will not complain

Updated Hebrew translation.

gtk/Makefile.am: Update VS projects completion

Take the sources in gtk/deprecated/ into account-VS does not like '/'
in file paths.

Update Visual C++ projects

-Update include paths for the GTK+ main projects to be consistent across
 the board
-Update demo projects to add a new demo source file, transparent.c

Update gdk/gdkconfig.h.win32

Apparently the deprecation warning macros are placed into gdkconfig.h
during the configure stage, so put these in the pre-configured
gdkconfig.h.win32 as well, as their definitions are needed for all builds.

Remove an unnecessary warning

Don't warn if there's more than one mnemonic label for a widget,
and skip invisible labels.

Bug 665015

Add a Since tag

gtk_css_provider_to_string is a post-3.0 addition.
Fixes bug 659516

Drop -uninstalled.pc files

This has already been done in GLib, so follow suit here.

XIM: gracefully deal with non-X backends

Patch by Young-Ho Cha, bug 660309.

GtkImContextSimple: fix tentative match handling

When switching to the compact table format, we inadvertedly lost
the 'tentative match' handling which allows to complete longer
compose sequences even if a prefix is a complete sequence.
This was pointed out in bug 666710.

Cosmetic changes to comments

Updated POTFILES.skip

settings: Cache style properties

This way we don't need to compute them every lookup. (That's not the
real reason though - the real reason is that I want to add new APIs that
require the caching because they return consts).

modifierstyle: Move Private typedef to public header

reftests: Add test for empty grid with spacing

This used to crash.

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

Fix problems with sticky handling

Reading a card32 property into a long may lead to undefined high
bits, so mask them off. Also, make the conditions for setting and
unsetting the stick flag opposites, to avoid unintended changes.
Patch by John Lindgren, bug 666842

Update various EWMH interactions

The current EWMH has added 'source indication' fields to a number
of client messages. Set these to 1 to indicate a regular client.
Also fill the button field of _NET_WM_MOVERESIZE.

tests/testgtk: Do not use GdkColor-based api

Updated Spanish translation

gtktextlayout: use g_slice for GtkTextLineDisplay

gtktextbtree: use g_slice

themingbackground: Avoid erroring cairo context

tests: Make widget-factory close when window is closed

cssprovider: Fix leak

Thanks to the heroic valgrinding efforts of Olivier Sessnik, another
leak could be plugged.

Fix scroll_offset handling

scroll_offset can be negative here, so replace a > 0 test
by != 0.

Only push the menu in if the position function said so

Use gtk_widget_get_allocated_width/height where it makes sense

Remove an unneeded variable

Remove an unused struct

combobox: Don't allow an individual cell to cheat

Make sure we only set sensitive when the cell is sensitive and visible.
Also avoid invalid memory accesses if no cells existed.

widget: Print a g_warning() when a cairo_t is invalid

Only print the warning on internal code, so we aren't too verbose.

styleproperties: Move private struct declaration into public header

x11: Don't handle focus events for NULL windows

The check was accidentally removed in the refactoring of
2ea328dfbc820c32a6f425805e632cd1aef45258.

Revert "x11: Avoid spurious focus events on grabs"

This reverts commit db4a6040afd3d1cabbb306037e811f37d62e2e76.

This is causing bad focus confusion e.g. in gnome-terminal.

Deprecate all the public API that is using GdkColor struct

gtkappchooserwidget: Use g_object_clear()

GtkModelMenu: drop extra & added by mistake

application: don't unconditionally unref the GMenuModel in getters

Both the app-menu and menubar models can be NULL, so we shouldn't
blindly unref them, since g_object_unref() doesn't cope with NULL
pointers.

application: mark menu/menubar getters as (transfer none)

Since they do not return a reference to the underlying GMenuModel.

application: use the same variable names in definitions and declarations

gobject-introspection will be confused and mark the methods as not
introspectable otherwise.

GtkAction: Hold a reference to proxy widgets

Previously we expect that a proxy widget holds a reference to the
action via gtk_activatable_do_set_related_action().  However, it
is possible for the widget to still be in a floating state when
it adds itself to the the action.

This is a problem when gtk_action_get_proxies() gets called, because
we return floating objects back to the user.  And language bindings
aren't going to be expecting that.

Fix this by calling ref_sink() and unref().

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

GtkApplicationWindow: fix a typo

And add a link to the screenshots.

tests/testcombo.c: Use GdkRGBA instead GdkColor

gtk/gtkcellview.c: Use new GdkRGBA api instead GdkColor one

gtk/gtktrayicon-x11.c: Use new GdkRGBA api instead GDKColor one

button: deprecate the inner-border style property

Since we allocate the standard CSS border to the button now, and center
the child accordingly, there's no need for an additional inner-border
style property. Deprecate it and ignore its values.

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

button: add CSS borders to the size request

And allocate the button children according to it as well.

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

button: add a border param to gtk_button_get_props()

We'll use it later to compute the child offsets.

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

tests/testgtk.c: Remove death code

gtkmodelmenuitem: don't crash on missing action

Updated Spanish translation

treeview: fix an uninitialised variable warning

Bump version

3.3.6

Fix dist

Fix symbol lists

Updates

Revert "progressbar: Set state flags on draw"

This reverts commit 6ed591871862ec87ee66fe69dca5c42087a12160 which is no
longer needed as the style context now has the state flags updated when going
into draw.

Add another bloatpad screenshot

gdkwindow: plug memory leak

==23282== 64 bytes in 2 blocks are definitely lost in loss record 8,069 of 13,389
==23282==    at 0x4A074CD: malloc (vg_replace_malloc.c:236)
==23282==    by 0x39A1C3E2EA: cairo_region_create (cairo-region.c:196)
==23282==    by 0x6D9AF3D: recompute_visible_regions_internal (gdkwindow.c:964)
==23282==    by 0x6D9B4B8: recompute_visible_regions (gdkwindow.c:1126)
==23282==    by 0x6DA3450: gdk_window_hide (gdkwindow.c:5689)
==23282==    by 0x6D9CED9: _gdk_window_destroy_hierarchy (gdkwindow.c:2042)
==23282==    by 0x6D9D040: gdk_window_destroy (gdkwindow.c:2109)
==23282==    by 0x655B5E4: gtk_entry_unrealize (gtkentry.c:3012)
==23282==    by 0x7068BF3: g_cclosure_marshal_VOID__VOID (gmarshal.c:85)
==23282==    by 0x706710B: g_type_class_meta_marshal (gclosure.c:885)
==23282==    by 0x7066DF9: g_closure_invoke (gclosure.c:774)
==23282==    by 0x7080585: signal_emit_unlocked_R (gsignal.c:3340)
==23282==    by 0x707F619: g_signal_emit_valist (gsignal.c:3033)
==23282==    by 0x707FB71: g_signal_emit (gsignal.c:3090)
==23282==    by 0x679E243: gtk_widget_unrealize (gtkwidget.c:4458)
==23282==    by 0x64E83C7: gtk_bin_forall (gtkbin.c:172)
==23282==    by 0x6548BBD: gtk_container_forall (gtkcontainer.c:2014)
==23282==    by 0x67A966D: gtk_widget_real_unrealize (gtkwidget.c:10253)
==23282==    by 0x672D002: gtk_tool_item_unrealize (gtktoolitem.c:474)
==23282==    by 0x7068BF3: g_cclosure_marshal_VOID__VOID (gmarshal.c:85)

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

Updates

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

notebook: Use the widget state flags as a base for drawing

menuitem: Stop setting state flags on the style context for drawing

menubar: Stop setting state flags on the style context for drawing

label: Use the widget state flags as a base for drawing

label: Stop setting state flags on the style context for drawing

image: Stop setting state flags on the style context for drawing

iconview: Use the widget state flags as a base for drawing items

iconview: Stop setting state flags on the style context for drawing

hsv: Stop setting state flags on the style context for drawing

expander: Use the widget state flags as a base for drawing

eventbox: Stop setting state flags on the style context for drawing

entry: Use the widget state flags as a base for drawing icons and text

entry: Stop setting state flags on the style context for drawing

combobox: Stop setting state flags on the style context for drawing

cellview: Stop checking for sensitivity on draw

This is needless since GtkCellRenderer does it consistently before rendering.

cellrenderertoggle: Use the cell renderer's state flags as a base for rendering

This allows us to pick the underlying widget's state flags that we don't
explicitly set here.