default impls of width_for_height,hfw should chain directly not use wrapper API

In GtkBin and GtkWidget we tried to provide handy defaults that
call get_width if there's no get_width_for_height and
get_height for get_height_for_width.

However, they used the wrapper API on GtkSizeRequest instead of
chaining directly to the other method implementation.

This could result in all kinds of surprising behavior, for example,
get_width_for_height() would now already include the effects of set_size_request().

If nothing else it's inefficient. But it's just conceptually wrong,
because to chain to another implementation, we should call the other
implementation, not call a wrapper around the other implementation
(when we're already inside a previous invocation of the wrapper,
i.e. compute_size_for_orientation() ends up reinvoking itself
in the same orientation on the same object which it pretty
likely isn't intending to do)

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

Use gint16 for GtkBorder

32K of border ought to be enough for any pixel dimensions. At least
until screens are so huge we start using doubles.

This saves a nice 64 bits of space when we have a GtkBorder
stored somewhere.

Signed integers are used to avoid surprising unsigned math
issues. Just search GTK's whole git log from inception
for "unsigned" if you want to find any number of commits
fixing signed/unsigned bugs.

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

Fix some compilation warnings

GScanner token field is declared as GTokenType instead a guint
See Glib bug: https://bugzilla.gnome.org/show_bug.cgi?id=627890

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

docs: Fix gdk_display_get_core_pointer() docs

Point to gdk_device_manager_get_client_pointer(),
not gdk_display_get_client_pointer().

style: Fix drawing of insensitive text

When writing the original code I erroneously assumed that the current
point of the cairo context would be saved by cairo_save/restore(), but
of course the current point is part of the path and therefor isn't
saved.

Also do a cairo_new_path() before rendering any text so that we are sure
the text ends up at the right spot.

Don't use g_idle_add to schedule idles in GDK

We need to use gdk_threads_add_idle, in order to keep GDK code
under the GDK lock.

Bug 629277

gtk/gtkwidget.c: Use accessor functions to access GtkWindowGroup

Revert "gtk/gtkmain.c: Use accessor functions to access GtkWindowGroup"

This reverts commit 65b9fc3e5bbae5b5efb4568c02df60939956f722.

gtk/gtkmain.c: Use accessor functions to access GtkWindowGroup

GtkWindow: move public members to private structure

Signed-off-by: Christian Dywan <christian@twotoasts.de>

Bump version to 2.90.8

Include gdkkeysyms.h in gdk.h

Followup to bug 629093

Gail: Remove gail_cell_type_add_action_interface, as it no longer does anything

Gail: Take fill/page size into account when returning a range's maximum value

Updated Norwegian bokmål translation

Some more updates

Update potfiles

Really stop exporting text-related variables

Remove GDK_DISPLAY() usage

Update NEWS for 2.90.7

Drop the gdk_display variable from gdk

This was the last exported variable; it wasn't multihead safe,
and there's easy replacement with gdk_display_get_default().
Also drop the GDK_DISPLAY() macro which was just a wrapper around
the variable.

Fix the docs build after GtkItem removal

Adding note to docs of gtk_widget_set_window()

gtk_widget_set_window() does not add a reference to the
owned window... this is an exception to most gtk_foo_set_bar()
functions where as it acts as if it were named gtk_widget_take_window().

Adding note to the docs to avoid people giving the window to the
widget and naturally proceeding to unref the window.

widget: Fix SEGV when no focus widget exists

gail: Add missing include

Avoid potential DLL hijacking in Wintab code

Load Wintab32.dll from an absolute path. A proper Wintab32.dll, if
present, should be in the Windows system directory, so load it from
there.

Use accessor functions

Fix gail build

Adapt to the GDK_KEY changes

Fixed GtkTextView & GtkIconView to consult it's true previous size request

Fixed issues in my previous patch for bug 626939 removing GtkRequisition
cache: these widgets monitor the previous requested size and decide whether
to queue a resize when the content changes based on it's prior request.

Moved GtkSizeRequest cache to GtkWidget->priv

Now that we have a private data installed directly on
the GtkWidget instance it makes no sense to cache the size
requests on widget qdata. This change will generally make
GTK+ memory less fragmented as well as significantly speed
up the size request process.

Use accessor functions in Win32-only code

Don't use the same name for a function that used to be a variable

Rename the gtk_major_version() etc functions I introduced yesterday to
start with gtk_get. Avoids misleading results in client programs whose
developers that don't notice the change or the compiler warnings, and
when recompiling against gtk3 then use the function addresses as the
version numbers.

Fix build of gail after GtkItem removal

 Remove GtkItem

With the demise of GtkList and GtkTree, it has GtkMenuItem as sole
derived class, and is not really adding any value as a separate class.
Its few useful features have been merged into GtkMenuItem.

Bug 629104

gtk/gtkhandlebox.c: Do not use gdk_window_get_deskrelative_origin()

This completes commit bd277fad50704f6a39619cb418204bdfee112b98

porting guide: Update to mention GDK_KEY_ transition

GDK: Prefix key names with KEY_

The keysyms create a lot of potential namespace conflicts for
C, and are especially problematic for introspection, where we take
constants into the namespace, so GDK_Display conflicts with GdkDisplay.

For C application compatiblity, add gdkkeysyms-compat.h which uses
the old names.

Just one user in GTK+ continues to use gdkkeysyms-compat.h, which is
the gtkimcontextsimple.c, since porting that requires porting more
custom Perl code.

gdk: Update gdkkeysyms.h

In preparation for adding a namespace prefix for introspection.

Remove deprecated gdk_window_get_deskrelative_origin() and its implementations

Drop GDK_DEBUG_GC and _IMAGE as GDK doesn't do that any more

Remove a duplicate variable

Bug#629110: gtk_spinner_accessible_get_type always returns null

Use G_CALLBACK() instead deprecated GTK_SIGNAL_FUNC()

docs: Remove GtkFileSelection deprecated docs

GtkWidget: Move public members to private structure

gtk/gtknotebook.c: use accessor functions to access GtkWidget

gtk/gtkarrow.c: use accessor functions to access GtkWidget

gtk/gtkpathbar.c: use accessor functions to access GtkWidget

gtk/gtkruler.c: use accessor functions to access GtkWidget

gtk/gtkseparator.c: use accessor functions to access GtkWidget

gtk/gtkhandlebox.c: use accessor functions to access GtkWidget

gtk/gtkiconview.c: use accessor functions to access GtkWidget

gtk/gtkimage.c: use accessor functions to access GtkWidget

gtk/gtkmenu.c: use accessor functions to access GtkWidget

gtk/gtkmisc.c: use accessor functions to access GtkWidget

gtk/gtkspinbutton.c: use accessor functions to access GtkWidget

gtk/gtktextview.c: use accessor functions to access GtkWidget

gtk/gtktreeview.c: use accessor functions to access GtkWidget

gtk/gtkwindow.c: Fix warning

Cast to GtkSizeRequest

This was added in
commit b654afcb480c4f971436bd82a7b20f86686a9c02

Get rid of the rest of variables exported from libgtk

Especially the gtk_*_type ones in gtktexttypes.h were mentioned in
gtk.symbols presumably by accident. That header isn't even installed,
so no way can they be supposed to be public.

gtk_text_attr_appearance_type is from the installed but "semi-private"
gtktextlayout.h, so drop that one too from gtk.symbols for now.

The use of gtk_text_unknown_char_utf8 is bit of a mess. Code in a few
files knew implicitly that it is three bytes. Define a symbolic name
for the length of it instead. Add an exported function
gtk_text_unknown_char_utf8_gtk_tests_only() that returns a pointer to
it just for the sake of gtk/tests/textbuffer.c. Prefix the variable
with an underscore.

I doubt the usefulness of the test_utf8() in textbuffer.c. If it could
be dropped, gtk_text_unknown_char_utf8_gtk_tests_only() could be
dropped, too.

Replace gtk_debug_flags with getter and setter functions

Preferrably should be made just into a local variable for libgtk like
_gdk_debug_flags for libgdk. But for now used by
gtk/tests/textbuffer.c and modules/printbackends/cups/gtkprintbackendcups.c.

Remove also the docs for the gdk_threads_mutex variable

Turn the gtk version and age variables into functions

Having variables exported from a DLL is slightly painful and
potentially error-prone on Windows, so let's try get rid of them now
when we can. Starting with these.

gtk/gtkwindow.c: Use gtk_size_request_get_size() instead gtk_widget_get_requisition()

Completely removed requisition cache from GtkWidget instance structure.

Since we have a new mechanism for requesting sizes: GtkSizeRequestIface;
it makes no sense to maintain this cache on the GtkWidget structure...
removing the requisition cache however does not break the old "size-request"
signal which is there for backwards compatability reasons.

In any case widget->requisition should not have been accessed,
gtk_widget_get_child_requisition() would have been the correct way
to consult the cache.

This commit also deprecates the newly added gtk_widget_get_requisition()
API and makes it fallback on gtk_size_request_get_size().

tests: Use gtk_widget_set_size_request() instead gtk_widget_set_usize()

gtknotebook: Do not mention deprecated API

demos: Do not use deprecated API

gtksocket: Remove mention to gtk_socket_steal() deprecated function

Do not use deprecated API in docs and examples

gtk_calendar_display_options() was deprecated and removed. Use
gtk_calendar_set_display_options() instead

Remove variables from the platform-independent GDK API

gdk_threads_mutex, gdk_threads_lock and gdk_threads_unlock are removed
from public API. gdk_threads_mutex was deprecated already. Instead of
using gdk_threads_lock and _unlock one was presumably supposed to use
the GDK_THREADS_ENTER and _LEAVE macros, which now simply call the
corresponding gdk_threads_enter() and _leave() functions.

Remove he dllimport/dllexport ugliness for GDK for Windows.

There is still a gdk_display variable being exported by the X11
backend.

gtk.symbols: Update expected ABI

Fix annotations for gdk_window_get_device_position()

Add (out) and (transfer none) as appropriate

add annotations for tranfer and scope so that methods are exported by GI

add array annotation to gtk_builder_add_objects_from_string

add gtk_radio_button_join_group method for bindings

* this mirrors the committed change for gtk_radio_action_join_group in
  commit 85b53969b24d318b219663841e678943516f443a
* Due to object ownership issues it is impossible to correctly use
  get_group/set_group from a GI binding
* join_group is safer because at the binding level it works with individual
  GtkRadioButton objects and not with the list of objects that gets
  modified in the library

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

Removed x/y/x_set/y_set from GtkWidgetAuxInfo

These members of GtkWidgetAuxData can no longer be set after
the removal of deprecated gtk_widget_set_uposition(), now removed
these members completely and removed the access to them from
gtk_widget_size_allocate().

Removed _gtk_window_reposition()

_gtk_window_reposition() was only some extra confusing code, internally
it only calls the public API gtk_window_move()... replaced calls to
_gtk_window_reposition() with direct calls to gtk_window_move() and
removed the redundant internal API (probably there only for some historical
reason).

Use a bitfield for Widget::private_flags, ::state, ::saved_state

This recovers 10 bits, since state and saved_state as uint8 both
wasted 5 bits. Now we can add 10 more flags.

Also, a possible future patch could just do 1-bit bitfields
for the private flags instead of using manual bit twiddling.

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

Avoid segfault on xkbless systems

This was reported in bug 628932 to cause problems on VNC.

Updated Portuguese translation

Updated Slovenian translation

Updated Kazakh translation

Remove use of ::has-separator property

Remove some unused enums

The GtkAnchorType and GtkVisibility enums are not used anywhere,
so remove them.

Warn if a widget requests minimum size > natural size

Since this makes no sense and there's no reason we should
have to check for it elsewhere just in case some widget is
busted.

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

Add more "reserved for expansion" fields to GtkWidgetClass

Only 3 were left. 3.x cycle may need more.

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

Updated French translation

GtkStatusIcon: Remove get/set_blinking() declarations from header.

This was apparently forgotten in
commit 18b47b6cd03f0f6da777ad3a5e2795cbf458b804 .

MS Windows engine: draw elements in the right place.
Work in progress, still remains to be solved the problem of clipping.

Updated Slovenian translation

Don't leak display name

Bug 628656 - _gdk_windowing_get_startup_notify_id memory leak

get_display_name() returns a newly allocated string, which was being fed
directory info a g_strdup_printf() call.

style: Use gtk_widget_get_state() for the entry background

New code sets the entry background to GTK_STATE_ACTIVE when it has focus
and the active color in the default theme looks rather bad.

[l10n] Updated Estonian translation

entry: Use the same state for the flat box of text area and icons

x11: Unused variable

range: Unused variable

window: Use brackets to silence gcc

tests: Remove all code relating to the blink menu item

Document --disable-Bsymbolic option