Remove no longer used code

We no longer consult the selector tree to find changes,
so this code is no longer needed.

css: Don't got to the selector tree for change

The tree is optimized for mimizing the decisions, and is built ahead-of-time.
That prevents us from taking advantage of the information in the matcher when
collecting changes.

So, instead do what we used to do for verification: Use the selector tree
for finding the superset matches, then just walk the rulesets to collect
the changes.

Since we are now recomputing the change masks much less frequently, this
slightly less optimized way of computing them is not a problem, and will
let us compute better results in the future, by improving the superset
matcher to be more precise.

cssprovider: Keep selectors around

Don't remove selectors from the rulsets after
creating the selector tree. We want to use them
in future commits.

Merge branch 'matthiasc/css-change-tracking' into 'master'

Better css change handling

See merge request GNOME/gtk!1324

css: Avoid computing change too often

Most of the time when styles need to be recreated, the name and classes
of the css node haven't changed. In this case, the change value will not
change either, since we are computing change under the assumption that
name and classes are unchanged.

So don't recompute the change. This avoids the second match we do to
find the superset, cutting down the number of times we consult the
selector tree.

Revisit gtk_css_style_get_static_style one more time

Move the implementation to where it belongs.

Change the signature of gtk_css_style_get_static_style

It is clearer if this function returns a GtkCssStaticStyle.

Quiet a compiler warning

This was causing warnings in a release build.

Merge branch 'static-style-getter' into 'master'

css: Add gtk_css_style_get_static_style

See merge request GNOME/gtk!1323

css: Add gtk_css_style_get_static_style

This lets us avoid poking directly at the GtkCssAnimatedStyle
struct in gtkcssnode.c.

Merge branch 'matthiasc/css-split-states' into 'master'

Split off some states in css change tracking

See merge request GNOME/gtk!1322

css: Track selected state separately

Like the previous change, this reduces the number of
css nodes reacting to parent-state changes. The remaining
parent-state cases here are due to :link.

Before (numbers from widget-factory with Adwaita):
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2245
hover 562
disabled 859
backdrop 1080
sibling-name 63
sibling-disabled 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 236
parent-hover 5
parent-disabled 91
parent-backdrop 4

After:
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2245
hover 562
disabled 859
backdrop 1080
selected 579
sibling-name 63
sibling-disabled 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 115
parent-hover 5
parent-disabled 91
parent-backdrop 4
parent-selected 144

css: Track backdrop state separately

Like the previous change, this reduces the number of
css nodes reacting to parent-state changes.

Before (numbers from widget-factory with Adwaita):
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2247
hover 562
disabled 859
sibling-name 63
sibling-disabled 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 236
parent-hover 5
parent-disabled 91

After:
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2245
hover 562
disabled 859
backdrop 1080
sibling-name 63
sibling-disabled 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 236
parent-hover 5
parent-disabled 91
parent-backdrop 4

css: Track disabled state separately

Significantly reduces the number of nodes that react to
parent-state changes.

Before (numbers from widget-factory with Adwaita):
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2247
hover 2247
sibling-name 63
sibling-state 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 2226
parent-hover 5

After:
2247 nodes
class 612
name 2246
id 2
first-child 148
last-child 156
state 2247
hover 562
disabled 859
sibling-name 63
sibling-disabled 51
parent-class 586
parent-name 788
parent-id 2
parent-first-child 78
parent-last-child 78
parent-state 236
parent-hover 5
parent-disabled 91

css: Track hover state changes separately

The idea is that this reduce the amount of frequently
changing state that css nodes are sensitive to.

This is going to reduce the amount of style recomputation.

Merge branch 'wip/carlosg/x11-dnd' into 'master'

X11 dnd fixes.

See merge request GNOME/gtk!1321

gdkdrag/x11: Issue XdndLeave on cancellation

According to XDND "The XdndLeave message cancels the session.",
issue one when cancelling a drag, so the dest side has an opportunity
to forget about the GdkDrop.

gdkdrag/x11: Drop timestamp argument from gdk_drag_do_leave()

This is not necessary for the XdndLeave message, nor used.

gdk/x11: Ensure to hide DnD surface after failed operation

The drag source might be cached and held alive, only disposed after
future drag begin operations. Ensure the drag surface gets hidden
properly or might might stay transparent but mapped till then.

gdk/x11: Ensure to sync type list after sending XdndEnter

Otherwise we are not ensured the GdkDrop is up-to-date, might even
be a stale one from an older DnD operation.

gdk/x11: Serialize gtype content formats before sending Xdnd enter

This ensures GType handlers get serialized to mimetypes at the time
the drag dest inspects the XdndTypeList property.

Speed up gtk_widget_pick

Add early exits, and avoid as much work as
possible.

Fix the filetransfer portal initialization

Reusing the cancellable only works if you don't throw
it away after first use.

Try harder to make ci not timeout

Make getting the file transfer portal proxy more
async, and avoid leaking a bus connection. This
causes GTestDBus to hang at the end of the test
run.

css: Drop unsused bitmasks

GtkCssProvider was maintaining bitmasks for the
set properties in each ruleset. The masks are never
used, so drop them.

css: Remove an unused function

_gtk_css_style_property_get_mask_affecting was superseded
by GtkCssStyleChange back in 2015.

testsuite: Improve --generate support

We don't want annoying spew from GTest when
using --generate, since it interferes with just
piping the output to a text file.

quiet a compiler warning in release builds

Use bundle id instead of package name for the macOS preview as filename is now changed

fontchooser: Fix up the previous commit

fontchooser: Fix objects-finalize test

The treeview is evil and keeps reference cycles in the
form of various tree row references. That gets cleaned up
if you explicitly gtk_widget_destroy the treeview. But since
07f2024bfca22331121, the scrolled window no longer destroys
its child, exposing this issue as a reference leak in
the objects-finalize test.

The font chooser widget is affected here because it calls
gtk_tree_view_scroll_to_path from init(), which creates one
of those reference cycles. Work around this in the font
chooser by unsetting the tree view model in dispose, which
clears up this cycle.

Fix encoding symbolic pixbufs

This was broken in 1a931da0465966b62b56e145b82e691ece67af91.

Update Indonesian translation

testsuite: Skip GtkDragIcon

GtkDragIcon needs a special surface, so skip it
in the notify tests.

Updated Spanish translation

Update Indonesian translation

Merge branch 'adwaita-cross-fade-fix-gtk4' into 'master'

Adwaita: Fix syntax error in cross-fade() (GTK4)

Closes #2371

See merge request GNOME/gtk!1308

Merge branch 'wip/chergert/fix-dragicon-installation' into 'master'

build: fix gtkdragicon header installation

See merge request GNOME/gtk!1306

Handle lack of file transfer portal

Don't block for 25 seconds trying to get a bus
connection. Give up after 1 second.

gtk-demo: Use a default action in the infobars demo

This makes the question infobar activatable by clicking
anywhere in it.

Adwaita: Add hover for clickable infobars

infobar: Activate default action on clicks

When we have a default action set on the infobar, activate it
for clicks anywhere in the infobar.  Also add an .action style
class in this case, so we can add a hover highlight to the infobar
just for this case.

Merge branch 'wip/chergert/textview-propagate-undo-state' into 'master'

textview: propagate GtkTextBuffer can-undo/redo action state

See merge request GNOME/gtk!1311

textview: propagate GtkTextBuffer can-undo/redo action state

This ensures the action state for undo and redo are propagated from the
buffer. Doing so means menu items now properly show sensitivity.

Merge branch 'wip/chergert/textview-undo-menu-item' into 'master'

textview: add undo/redo menu actions

See merge request GNOME/gtk!1310

textview: add undo/redo menu actions

Adwaita: Fix syntax error in cross-fade()

cross-fade() requires two images.

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

build: fix gtkdragicon header installation

Updated Spanish translation

Updated Spanish translation

Merge branch 'wip/jimmac/updated-checkmark' into 'master'

Adwaita: update checkbox & radio

Closes #2291

See merge request GNOME/gtk!1289

Adwaita: update checkbox & radio

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

Merge branch 'closure-annotations' into 'master'

Fix incorrect use of GIR closure annotations.

See merge request GNOME/gtk!1305

Add missing (closure) GIR annotations

Fix incorrect use of GIR closure annotations.

They were being applied to the callback function, not the data that is passed
to that function.

Merge branch 'textbuffer-missing-api-docs' into 'master'

Add missing public API documentation.

See merge request GNOME/gtk!1301

Merge branch 'menutoolbutton-missing-api-docs' into 'master'

menutoolbutton: Add missing public API documentation.

See merge request GNOME/gtk!1303

Merge branch 'docstring-typos' into 'master'

Fix docstring errors with drag and drop code

See merge request GNOME/gtk!1304

Merge branch 'missing-transfer-notation' into 'master'

Add missing transfer notation for function return types.

See merge request GNOME/gtk!1302

Fix docstring errors with drag and drop code

textbuffer: Add missing public API documentation.

This API was added in a52757874ee.

menutoolbutton: Add missing public API documentation.

This API was added in 03e30431a8a

Add missing transfer notation for function return types.

Adwaita: Fix needs-attention

Stack switchers are now stackswitcher, no longer .stack-switcher.

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

Update POTFILES.in

Merge branch 'dnd-gestures-2' into 'master'

Add new dnd api

See merge request GNOME/gtk!1278

dragdest: More documentation

Update all callers

Reshuffle droptarget api a little bit

Add a ::drag-motion signal back, remove x,y from
the ::accept signal, and only emit ::accept once.

Merge branch 'wip/chergert/gtk4-gtk-main-sync' into 'master'

Rubustness fixes for gtk_main_sync()

See merge request GNOME/gtk!1299

gtkmain: be deterministic in source removal

Fixes gtk_main_sync() to only remove a source if it has not
already executed (and been removed). The previous code was
using gtk_main_quit() directly which would be non-determinstic
based on the previous value in the return register.

gtkmain: short-circuit gtk_main_sync() where no display

If there is no display, we will hit the slow path here which
can introduce long latencies in unit tests. This checks for
a NULL list of displays and simply short-circuits.

Merge branch 'css-font-values' into 'master'

Css font values

See merge request GNOME/gtk!1298

css: Simplify font features

Don't store numbers as GtkCssValues needlessly.

gdk_quartz_keymap_translate_keyboard_state fills consumed_modifiers
incorrectly (closes #2102)

See merge request GNOME/gtk!1225

css: Simplify compute for font variations

Just like font features, font variations are stored
as a hash table mapping strings to pure number values,
which never change during compute.

css: Simplify compute for font features

Font features consist of a hash table of number
values, which will never change during compute,
so there is no point in recreating a new value
with a copy of the hash table, constaining the
same string->number mapping.

Merge branch 'kill-builtin-image' into 'master'

Kill builtin image

See merge request GNOME/gtk!1297

Remove builtin icons altogether

This removes support for GtkCssImageBuiltin and
GtkCssImageBuiltinType from everywhere.

css: Stop supporting builtin images

Change the default value of -gtk-icon-source to 'none',
and stop parsing 'builtin' as a value.

Update Slovak translation

testdnd: Fix up for current api

docs: Expand the dnd migration hints

dragicon: Add docs

docs: Fix a typo

Properly export GtkDragIcon

The get_type function was missing.

Update all users

dragdest: Rename ::drag-motion to ::accept

dragdest: Allow denying drops asynchronously

Add an api to deny a drop, this is meant to be used
when delaying the decision on whether to accept a
drop until after reading the data.

remove debug spew

dragdest: Add a ::drag-enter signal

Mainly for symmetry with ::drag-leave.

testdnd2: Fix the text case

We must take mime types into account when checking.

content provider impl: Take mime types into account

dragdest: Steps towards allowing nested drop sites

Add another dnd testcase

This one tests nested drop sites and interaction between
DND and other gestures.

Update all callers

Add GdkDrop back to signal signatures

Drop gtk_drag_(un)highlight

filechooser: Remove code for sidebar pre-arming

This is just not a feature we can support currently.
Doing it properly would require infrastructure for
a generalized :drag(sensitive) state highlighting
potential drag sites everywhere.

listbox: Stop using gtk_drag_(un)highlight

This api is going away, so just do what needs to
be done manually.