application: return a value

gtkapplication.c: Fix syntax error

Previous commit accidentally removed the closing } from
gtk_application_end_session(). Add it back to unbreak the build.

Add a trivial implementation of the new apis

For non-X11, this is just a placeholder until we get around
to moving the EggSMClient implementations over.

Document the new apis

Add gtk_application_end_session

This function allows applications to request that
the user session be ended by logout/shutdown/reboot.

GtkApplication: Add an inhibit api

This lets applications block logout and similar actions ahead
of time. Currently only implemented for D-Bus, but Windows has
very similar API since Vista.

GtkApplication: Add logout notification

This is fairly basic, allowing applications to learn when
the session manager is about to end the session, and possibly
block this. The only implementation at this point is using the
org.gnome.SessionManager D-Bus interface of gnome-session. It should
be straightforward to port the EggSMClient implementations for
Windows and OS X.

Update Visual C++ property sheets

"Install" (and hence compile with glib-compile-schemas.exe) the new
org.gtk.WindowState.gschema.xml gschema file.

GtkWindow: new API to store state in GSettings

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

GtkBuilder: support parsing GVariant properties

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

Updated POTFILES.skip

wayland: Break an implicit grab when asking to be resized

An implicit grab is created inside GTK+ when the button is pressed down on a
window. The semantics of wl_shell_surface_resize means that you don't get a
corresponding release event that would ordinarily break the implicit grab. So
we must do it as part of the resize request.

wayland: Make the event source use _gdk_display_get_next_serial

wayland: Implement the get_next_serial vfunc on GdkDisplay

gtkspinbutton: Don't hilight buttons when !editable

Don't hilight the buttons on mouse over when 'editable' property is set to
'FALSE.

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

wayland: The shared cursors are now installed in the weston directory

Signed-off-by: Rob Bradford <rob@linux.intel.com>

GtkApplication: Rename a member for clarity

'session' is a bit ambiguous (and GtkApplication may grow session
management support at some point), so rename the bus connection
to 'session_bus'.

Another attempt at fixing menu positioning corner cases

The code for moving the menu into monitor / workarea was duplicated,
once for the push-in scenario and once for without. The problem with
the second case is that we've stored the menu position before adjusting
it. That made us remember an out-of-monitor position that then later
triggered _another_ copy of this code in the size-request implementation.

Unify this to only have one copy of code, and only store the menu
position after adjusting it to be inside the monitor. This fixes both
statusicon menus that get popped up from the panel, outside the workarea,
to not have scroll arrows, and the gedit language menu which was not
placed in the monitor at all after the initial workarea commit.

As a side-effect of this change, we now make large scrolling menus
occupy the full height of the workarea. Before this change, we were
keeping either the top or bottom edge put while shrinking the menu
to fit in the monitor.

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

label: Avoid deprecation warning

label: Actually mark visited labels as visited

A recalculate will clear the labels and compute them again, which will
clear the information if something was visited...

reftests: Add a test for latest commit

We want to make sure we notice when we mess up label attribute ordering
again...

label: Make attr_list_merge() use sane merging rules

Instead of "attribute with later start index wins, make sure the
attribute list that is merged from takes precedence. This now gives the
multiple attribute lists we use in the label an order:

1) gtk_label_set_attributes()
  These attributes override everything. It's what the function's there
  for after all.
2) markup of label
  Other user-specified attributes come next.
3) attributes for links
  When we apply custom attributes on parts of the text, we put them
  last. We don't want to mess with what the user does. Also, we change
  color and underline, so we usually have something to show.

label: Rename variable

The effective attributes aren't that anymore. They're now just the
attributes from the parsed markup.

label: Don't excessively recalculate

We don't merge attributes anymore, so no need to call the function that
used to do this job.

label: Redo attribute application

- Don't compute link color attributes until layout creation
This is useful as a performance enhancement, because we don't have to
lookup the property after setting the text, so multiple markup sets
don't cost style lookups.
- Don't merge attrs into effective_attrs
We do this when applying link colors now. Keeping them separate allows
invalidating them separately.

ilabel: Redo gtk_label_get_link_colors()

Don't pass out an allocated color, assign it instead. Gets rid of having
to call gdk_color_free() all the time.

label: Split out function that merges attr lists

gtk-demo: Prove that links work fine now

Also, doodling is fun.

label: Redo links parsing

Instead of getting confused by applied underline or color tags in the
regular markup, we store the link start/end when we actually parse the
text. As a bonus, we can avoid rescanning links when creating the
markup.

label: Move function into only caller

API: Deprecate key snooping

You really shouldn't be doing this. If you have to, you can use
gdk_window_add_filter() or listen to raw events on XI2.

a11y: Remove keysnooping support

As previously announced, key snooping is something that we should not
support in GTK. In particular because AT-SPI made every key press a
synchronous DBUS call / spawned a nested main loop with the key
snoopers. This breaks responsiveness, order of key presses and internal
consistency when the AT decides to remove key events.

gtkspinbutton: Ignore keynav when !editable

Don't allow changing of value through keyboard when 'editable' property is
set to 'FALSE.

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

Improve PICKONE_* handling in GtkPrinterOptionWidget (bug #543520)

This patch modifies GtkPrinterOptionWidget to support loading them
again. It also allows the user to enter values from the dropdown list
to select the item.

Set cups Custom print options correctly (bug #543520, patch by Marek Kašík)

This patch fixes the cups print backend to pass Custom options with
the "Custom." prefix to cups if neccessary.

Use the right icon-name for missing images

Patch by Erik van Pienbroek, bug 629878.

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