Fix syntax

Updates for 3.98

Merge branch 'wip/carlosg/implicit-grabs-on-leave' into 'master'

Handle implicit grabs on leave events

Closes #13

See merge request GNOME/gtk!1397

gtkmain: Wait for GDK_CROSSING_UNGRAB leave event with implicit grabs

If there is a passive grab and the pointer leaves the window we would
receive a GDK_CROSSING_NORMAL event when the pointer moves outside
the window, and a GDK_CROSSING_UNGRAB event when we do release the
button and the implicit grab.

We currently would react to the first, but want to react to the
second. In the time between both events, the client would still receive
pointer motion that will reach the implicitly grabbed widget.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/13

gtkroot: Make enum a typedef

Seems a typo, and prevents building as GtkRootProperties is deemed
to be defined multiple times.

Merge branch 'a11y-fixes' into 'master'

A11y fixes

See merge request GNOME/gtk!1396

testsuite: Misc. updates of expected output

The a11y dumps for action and header bars
have changed in harmless ways.

infobar: Implement forall

As a container, that is what you have to do.

testsuite: Clean up a11y menu tests

We don't need some of the tests anymore, since
we just have popovers now.

testsuite: Improve --generate support

When using the test binary to generate expected
output, don't initialize the test machinery, since
that pollutes the output with noise.

tests: Fix infobars in ui files

GtkInfoBar no longer exposes the content_area
as internal child.

testsuite: Update expected output

colorchooser: Use a composite accessible

This makes the content show up in the a11y tree again.

tests: Update expected output

The stackswitcher is no longer a box, so it doesn't
have the horizontal state anymore.

testsuite: Update some a11y tests

GtkMenuButton::use-popover no longer exists, remove it
from test files.

stackswitcher: Use a composite accessible

This makes the buttons show up in the a11y tree again.

a11y: Add an accessible for widgets with children

We've started to turns containers into widgets which
just happen to have children, and some of these need
to be exposed to the a11y stack.

This adds a very minimal implementation, it does not
currently emit change notification when children are
added or removed.

a11y: Report children of widgets

We want children of composite accessibles to be
reported.

Merge branch 'flatpak-build-sassc' into 'master'

Flatpak build sassc

See merge request GNOME/gtk!1392

Build sassc as a dependency for flatpaks

Downloading the subproject during the GTK build fails
in a flatpak build due to lack of network access.
flatpak-builder insists on having these things explicitly
spelled out as dependencies.

Merge branch 'WindowsPrintDriverPreferencesMaster' into 'master'

Fixed bug that prevented GTK from storing printer driver preferences in the…

See merge request GNOME/gtk!397

notebook: Update arrows when text direction changes

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/103

Merge branch 'kill-simple-search' into 'master'

Remove simple search engine

See merge request GNOME/gtk!1393

Merge branch 'better-test-isolation-2' into 'master'

Better test isolation

See merge request GNOME/gtk!1391

css tests: Ignore ".csd" vs ".solid-csd"

It it hard to control which of the csd style classes we get,
since it depends on details of the X server or compositor.

Explicitly ignore this difference by replacing .solid-csd
with .csd in the output.

testsuite: Make tests run without a11y bus

Not having an a11y bus around causes warnings from
at-spi, which make tests fail. Explicitly ignore those
warnings.

testsuite: Fix bloomfilter style tests

The results were expecting solid-csd, which is not what
we usually have.

Merge branch 'master.msvc' into 'master'

Fix building recent GTK master on Visual Studio

See merge request GNOME/gtk!1369

Merge branch 'public-emojichooser' into 'master'

emojichooser: Make public

See merge request GNOME/gtk!1382

Merge branch 'wip/otte/css' into 'master'

Some CSS cleanup

See merge request GNOME/gtk!1389

emojichooser: Add profiler marks for loading

This is a long-running operation, and worth showing
in traces.

icontheme: Fix various memory leaks

Tasks need to be unreffed too.

widget: Fix a memory leak with class actions

This code was doing the wrong thing even if the parent class
doesn't have actions.

Add a lib64 variant of gtk.supp

icontheme: Remove some dead code

frameclock: cosmetics

Redo the profiler marks for the frame clock signals
to look more uniform.

gdk: Silence some compiler warnings

These cause unused variable warnings in release builds.

treestore: Silence compiler warnings

This was causing compiler warnings in release builds.

notebook: Don't trigger criticals in dispose

When we dismantle our children in dispose, we
trigger a11y children-changed signals which end
up calling back into the notebook. Handle this
without critical warnings.

textview: Don't update action state in dispose

Only update the action state when we have a text buffer,
there is no need to do it otherwise, since we are going
to get a buffer before we get shown. This avoids triggering
the action state updates from finalize, which is a bad
time to be recreating the action muxer.

entry: Avoid criticals in dispose

The accessible gets properties of the entry, and
resetting the entry icons triggers accessible change
notification, so do that before we dismantle the entry
too far to respond to a g_object_get () call.

Don't export color picker functions

This is not public API, so it shouldn't be exported.

Read bookmarks async

Do not block the main thread while reading the bookmarks file.
This speeds up initial setup of a GtkPlacesSidebar.

placessidebar: Change some initial property values

Giving these properties the initial value that the file
chooser wants for them avoids some calls to update_places
in the setup code.

testperf: More flexible mark selection

Allow selecting marks not just by their name, but
also by the message detail.

testperf: Allow measuring start time

This can be used e.g. to measure how long it takes us
to get the first frame on screen.

stylecontext: Remove unused typedef

css: Make some local variables const

css: Remove unused function

style context: Remove parent relationship

We don't have foreign drawing anymore, so we don't need this.

stylecontext: Remove gtk_style_context_new()

It's private now, no more foreign drawing of any sort.

To be fair, it was entirely useless without widget paths already.

testsuite: Remove the stylecontext test

Stylecontexts are on their way out and I'm removing API that the
testsuite was relying on, so remove the tests.

Put the useful parts of the tests elsewhere.

stylecontext: Remove the "changed" signal

Instead, always go directly to the GtkWidget::css_changed() call.

widget: Replace style-updated signal with css_changed vfunc

1. Rename the thing
2. Turn it from a signal to a vfunc
3. Pass the GtkCssStyleChange to it

We don't export any public API about the GtkCssStyleChange yet, it's
just a boring opaque struct.

pathbar: Remove unneeded code

The icons here are GIcons, they don't depend on the theme.

cssnode: Remove redundant check

We assert that cssnode->parent is NULL, no need to check it again.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1388

widget: Skip not-drawable widgets in snapshot_child

Otherwise we transform the snapshot just to not do anything.

maplistmodel: Add an example to the docs

treelistmodel: Small documentation tweaks

widgetfocus: Remove unnecessary check

Mapped widgets are always realized.

widget: Remove _gtk_widget_is_drawable()

Use _gtk_widget_get_mapped() instead.

Drawable checked visible && mapped and mapped widgets are always
visible, so that was an unnecessary check.

Merge branch 'wip/otte/icontheme' into 'master'

Do simple icontheme API cleanups

See merge request GNOME/gtk!1378

Rename GtkIcon to GtkIconPaintable

tooltip: Don't css-validate

That's a job for toplevels now.

See 17ca95a1617b009a76b27a8572671855435985e7 for the same comit on
popovers.

icontheme: Remove GENERIC_FALLBACKS

Instead, rely on people passing fallbacks explicitly.

Alternatively, GThemedIcon provides the functionality to create
fallbacks, which is what GtkImage and the testsuite now use.
That method is slightly better, too, so the expected test results
have been updated accordingly.

icontheme: Pass fallbacks as optional argument to lookup_icon()

This way, we can remove gtk_icon_theme_choose_icon() completely.

icontheme: Remove contexts

There is no way to query contexts or do anything useful with them.

So don't keep track of them and don't make them an argument in public
APIs with the docs saying "I don't know what to use here, maybe read
some spec somewhere".

icontheme: Clean up includes

icontheme: Remove gtk_icon_get_base_size() and gtk_icon_get_base_dir()

Those functions are unused and the documentation says "Returns some
random number that the icon theme creator chose" which does not seem at
all useful and an implementation detail.

So get rid of it.

Revert "Break out cairo color matrix recoloring to gdk_cairo_image_surface_recolor()"

We're not in the business of adding Cairo APIs. That's Cairo's job.

Also, we don't need this API anywhere like the original commit claimed,
so there's no need to make it available in any way.

This reverts commit afa6cc23692f83f0d38de24c9cf58a7a494c01d2.

icontheme: Remove crayon APIs

We expose no API to get at any colors for drawing symbolics, so we
shouldn't have APIs to draw with them.

Apart from that, those APIs look like a box of crayons, not like an
icontheme.

icontheme: Remove gtk_icon_theme_rescan_if_needed()

If the icon theme needs to be rescanned, it should just do it.
If it doesn't, there's no need for this function.

Ergo: Don't have the function.

icontheme: Remove NO_SVG and FORCE_SVG flags

If the icon theme loads SVGs or not is an implementation detail and
should not be exposed in public API.

icontheme: Remove GTK_ICON_LOOKUP_USE_BUILTIN

It's unused.

icontheme: Make text direction a lookup argument

Most users were just forgetting to set the proper flags.
And flags aren't the right way to set this anyway, it was just
acceptable as a workaround during GTK3 to not break API.

icontheme: Remove async APIs

Widgets would not use them properly. In fact, the only user was using
them wrong.

As icons are loaded async by default, this call isn't necessary.

icontheme: Remove gtk_icon_theme_get_default()

The API encouraged wrong usage - most of the users were indeed wrong.

Use the correct version instead:
  gtk_icon_theme_get_for_display (gtk_widget_get_display ())

a11y tests: Ignore "active" flag on toplevel

All the a11y tests were failing for me with a window state diff
like this:

-  state: active enabled resizable sensitive showing visible
+  state: enabled resizable sensitive showing visible

I guess the windows in the CI always gets the focus, but not when
I run it here. Generally focus seems asynchronous and hard to rely
on so I just made the test ignore the active state on toplevels.

tests: Initialize dbus before gtk

These days initilizing gtk may create a connection to the sesson bus,
so we have to initialize GTestDBus before initalizing gtk, or we'll
use the address of the "real" session bus (and remember that in the
global).

To further muck things up, g_test_dbus_up() resets important env
vars like DISPLAY and XDG_RUNTIME_DIR, which we have to re-set.

filechooser portal: Free portal proxy when bus connection closes

This was blocking the clean exit from the testdbus shutdown in
the defaultvalues test. The proxy was keeping the connection alive
which blocks g_test_dbus_down().

css: Fix the initial values optimization

Fix up the index computation. We have duplicate entries
in the type enum, so to go from one of the 'initial' types
to it corresponding type you subtract one, but to find
the size array entry for a type, you divide by 2.

Merge branch 'faster-theme-lookup' into 'master'

Faster icon theme lookup

See merge request GNOME/gtk!1381

icon-theme: Only have one copy of the icon flags enum

It turns out with the icon cache now using the virtual SYMBOLIC_PNG_SUFFIX
flag the two enums are now identical, so lets just use one of them, the
one GtkIconCache (so we move it to the header).

Don't use xsettings or xft defaults in testsuite

This adds a GDK_DEBUG=default-settings flag which disables reads
from xsettings and Xft resources, and enables this for the testsuite.

This is one less way to get different testresults depending on the
environment. In particular, it was failing the css tests for me
due to getting the wrong font size because i have a different dpi.

emojichooser: Make public

This is a useful widget to have, and it has minimal api.

Not having it public forces apps to recreate a lot of
complicated machinery for not good reason, if they need
an Emoji chooser in a different context.

GtkIconCache: Remove unused code

We're not really using the icon theme cache much anymore, as the
individual per-directory hashes are no longer used, so delete all the
unused code.

IconTheme: Make icon lookups faster

Traditionally the icon lookup for a theme has been:

lookup (icon_name, size):
  best_directory = NULL
  forearch theme
    foreach directory in theme
       if dir_size_matches (directory, size) && dir_has_icon (directory, icon-name)
         best_directory = chose_best_size_dir (best_directory, directory)

if best_directory
  return icon from best_directory

However, it turns out that there are a lot of subdirectories which have the same
size, as they differ only in the (essentially useless) "context" value. For example
the "16x16/apps" subdirectory is essentially the same as the "16x16/actions" one.

So, instead rathern than keeping all the directories as separate we store the
all the directories with the same size as a single entity (DirSize) and the
icon lookup in that DirSize looks up not only which suffix to use for that icon
but also which subdir it is in.

Additionally we keep a hashtable with all icon names that are
available in the entire theme (i.e. all DirSizes), which allows use
both to store each icon name only once, but also to do a quick
negative lookup and early exit in case we're looking up an icon that
doesn't exist. This is pretty common because we often look up sets of
icons like "image-png-symbolic", "image-png", "image", expecting some
to fail.

This brings down the time of the initial css validation from 20msec to 15msec for
me when running icon-factory.

icon-cache: Add new function to list all the icons in a directory

This lists the icons in a particular director, with their flags in a
hashtable. We also convert from "icon.symbolic" + SUFFIX_PNG to
"icon" + SUFFIX_SYMBOLIC_PNG.

meson/MSVC builds: Use -utf-8 where available

This avoids the build from erroring out on C4819 (Unicode handling issue in
Visual Studio compiler), notably when running on Chinese, Japanese and
Korean locales.

Also apply -D_USE_MATH_DEFINES, -FImsvc_recommended_pragmas.h and -utf-8 to
the C++ compiler options as well.

Remove last GtkWidgetPath reference

See 49b47c913389dc395a48c2b19ac409412623c524

demos/widget-factory/widget-factory.c: Include stdlib.h

Include it for exit().

tests/testdnd2.c: Don't include unistd.h unconditionally

Instead, on Windows when it is not available, include io.h for close()

Merge branch 'wip/muktupavels/update-documentation' into 'master'

display-x11: don't mention gdk_monitor_is_primary in the docs

See merge request GNOME/gtk!1380

display-x11: don't mention gdk_monitor_is_primary in the docs

https://gitlab.gnome.org/GNOME/gtk/merge_requests/1371#note_699272

css: Fix an oversight in style change computation

Take the 'color == NULL means currentColor' trick into account
when computing the changes between two styles.

Merge branch 'sigma' into 'master'

gdk: Handle small final sigma in gdk_keyval_convert_case

See merge request GNOME/gtk!1351

Merge branch 'initial-style' into 'master'

Make it easy to recognize initial style values

See merge request GNOME/gtk!1375

border: Don't render initial borders

Add a fast exist for comon case that the border
values are unset.

background: Don't render initial backgrounds

Add a fast exit for the common case that the background
values are unset.