csscolorvalue: Don't copy rgbas when applying function

We get const pointers to the colors, so just use those and unref the
values later.

gl renderer: Look at shadow color in the outset shadow cache

It would probably be better to not do this and always render the outline
in plain white, then later recolor it but do this for no, just for
correctness.

cssimageradial: Use array + length instead of GArray

This makes sense since we're not going to change the (amount of) colors
after parsing.

cssimagelinear: Use count+array for the color stops, not GArray

This makse sense but will also make later changes to GtkCssImageLinear
simpler.

cssimagefallback: Don't compute new image if only a color is set

Themes might use e.g. image(red), which is a constant value and will
never change. In that case, the fallback image has ->color set, but not
->images. If that's the case and the computed color is the same as
the one we already have, just return the already existing image.

Remove GtkCssRgbaValue

The differenciation between a literal color value and an RGBA value
caused problems in various situations. Just treat the two the same but
don't allow access to the rgba value of a non-literal color value.

This gets rid of around 1.6k rgba values in the widget-factory.

dump css value stats

csssshadwovalue: Remove unused function prototype

cssnumbervalue: Add early-outs to transition code

we don't need to do the calculation at all if the progress is 0 or 1
anyway.

We also sometimes transition from 0 to 0 etc., so we can short-circuit
that as well by doing the fast pointer-equality check and relying on the
singletons.

csscornervalue: Accept other values if x == y

Most corners are square, so x == y. In that case, just accept either of
them. This makes the corner value unnecessary.

In fact none of the corner values in the widget-factory are needed, so
this spares us around 500 corner value allocations.

css value stats before:

GtkCssBgSizeValue: 23
GtkCssIdentValue: 25
GtkCssPositionValue: 81
GtkCssCornerValue: 556
GtkCssArrayValue: 143
GtkCssStringValue: 33
GtkCssPaletteValue: 29
GtkCssImageValue: 2765
GtkCssColorValue: 1452
GtkCssFilterValue: 3
GtkCssRgbaValue: 1092
GtkCssShadowValue: 708
GtkCssEaseValue: 33
GtkCssBorderValue: 2
GtkCssTransformValue: 11
GtkCssDimensionValue: 882
GtkCssShadowsValue: 584
SUM: 8428

and after:

GtkCssColorValue: 1452
GtkCssFilterValue: 3
GtkCssRgbaValue: 1092
GtkCssShadowValue: 708
GtkCssEaseValue: 33
GtkCssBorderValue: 2
GtkCssTransformValue: 11
GtkCssDimensionValue: 882
GtkCssShadowsValue: 584
GtkCssBgSizeValue: 23
GtkCssIdentValue: 25
GtkCssPositionValue: 81
GtkCssArrayValue: 143
GtkCssStringValue: 33
GtkCssPaletteValue: 29
GtkCssImageValue: 2765
SUM: 7872

8428 to 7872 is a 556 reduction (6.5%)

asdf

cssarrayvalue: Don't allocate memory when parsing array values

We probably won't find CSS with more than 128 values in array.

cssarrayvalue: Allow calling array API on non-arrays

Just allow calling _get_nth() and _get_n_values() on every kind of css
value. This way we can allow all values in places where only array
values would be allowed before.

This spares us around 1000 array values in the widget factory.

css value stats before:

GtkCssFilterValue: 3
GtkCssRgbaValue: 1092
GtkCssShadowValue: 708
GtkCssEaseValue: 33
GtkCssBorderValue: 2
GtkCssTransformValue: 11
GtkCssDimensionValue: 882
GtkCssShadowsValue: 584
GtkCssBgSizeValue: 23
GtkCssIdentValue: 25
GtkCssPositionValue: 81
GtkCssCornerValue: 556
GtkCssArrayValue: 1130
GtkCssStringValue: 33
GtkCssPaletteValue: 29
GtkCssImageValue: 2765
GtkCssColorValue: 1452
SUM: 9415

and after:

GtkCssBgSizeValue: 23
GtkCssIdentValue: 25
GtkCssPositionValue: 81
GtkCssCornerValue: 556
GtkCssArrayValue: 143
GtkCssStringValue: 33
GtkCssPaletteValue: 29
GtkCssImageValue: 2765
GtkCssColorValue: 1452
GtkCssFilterValue: 3
GtkCssRgbaValue: 1092
GtkCssShadowValue: 708
GtkCssEaseValue: 33
GtkCssBorderValue: 2
GtkCssTransformValue: 11
GtkCssDimensionValue: 882
GtkCssShadowsValue: 584
SUM: 8428

9415 to 8428 is a 987 reduction (10.4%)

cssvalue: Add type names for all classes

This is important to have for debugging and e.g. to print statistics for
the individual css value types

csscolorvalue: Compute alpha/shade/mix of color literals directly

The values of these are never gonna change so we can as well create
literal color values from the resulting colors instead.

csscolorvalue: Don't resolve literal color values

Merge branch 'adwaita-performance' into 'master'

Adwaita: Remove wildcards

Closes #2380

See merge request GNOME/gtk!1330

Adwaita: Remove wildcards

Selectors like *:disabled or *:link have bad performance
implications, since they cause all styles to be recomputed
when the state of the window changes. Replace these by a
list of the elements that are actually affected.

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

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

Improve change computation

See merge request GNOME/gtk!1327

Remove the widget-factory test again

This was failing in ci, in hard to track down ways.

change tests: Update test output

Some of these test cases involve :not, and thus are affected
by our now correct handling of it for change computation.

All of them are affected by the window now being visible.

Remove no longer needed code

We no longer use superset or any matchers to find changes.

Go back to the tree for computing change

Redo the way we compute change

Instead of expecting a superset matcher, call
gtk_css_selector_match_for_change while walking the tree with the
original matcher. This fixes the handling of :not while determining
changes.

Bring back the reserved bit

Revert "Remove no longer used code"

This reverts commit 105acfe90866543904467b13fecdbed108cf43b2.

Add testcases for css change flags

Add various tests for the change flag computation that
we do in the css selector tree.

test1: Just test the basic machinery of this test
test2: Trigger every change flag at least once
test3: Test that multiple states combine as expected
test4: Test negations (known to produce wrong results)
test5: Test a complex selector (not producing the expected
       output atm)
widget-factory.ui:
       The real thing: widget-factory+Adwaita. Note that
       this expedts to be run with GSETTINGS_BACKEND=memory

Note that test4 checks the wrong results that we currently
produce for selectors involving :not. It will have to be
updated when we fix the handling of :not. The widget-factory.ui
testcase will certainly also be affected.

Add a test for css change flags

Add a way to dump css node change values

Add a GTK_STYLE_CONTEXT_PRINT_SHOW_CHANGE flag that
tells gtk_style_context_to_string to include the
change values of nodes in the output. This will
help debugging css change tracking.

style tests: Update expected output

The printing of css nodes has changed, and the window
is visible now.

style tests: Show the window

Not doing so somewhat defeats the point that we want
to verify the regular styles, not some weird initial
conditions.

style tests: Clean up --generate support

Reshuffle things so we don't call gtk_test_init() when --generate
is passed, to avoid polluting the output with test spew.

node tests: Update expected output

The printing of css nodes has changed and the
window is visible now.

node tests: Show the window

This avoids using default styles.

node tests: Clean up --generate support

Reshuffle things so we don't call gtk_test_init() when --generate
is passed, to avoid polluting the output with test spew.

Add an Empty theme for tests

cssnode: Fix a corner case in style computation

When we recomputed the change flags, and the style
did not change, we still need to replace the style
object, since it holds the new change flags.

cssnodedeclaration: Make print output deterministic

This was depending on the memory order of quarks, which
is not a good fit for tests that depend on this output.

Quiet compiler warnings in release builds

Tweak the file transfer portal _again_

This is a neverending story. I was seeing problems in tests where
the nested mainloop was picking up unrelated timeouts.

Break down and make this async. This changes the ordering in which
the (de)serializers are registered. If this is causing issues, we
can introduce priorities or something else.

Merge branch 'gtk-fix-gesture-get-center' into 'master'

gtkgesture: Fix gtk_gesture_get_bounding_box_center() for touchpads

Closes #2316

See merge request GNOME/gtk!1257

demo: Use pinch translation in the gestures demo

Add the pinch translation offset to the pinch gesture demo. That allows
moving the square around in addition to the already-supported rotate and
zoom.

gtkgesture: Fix get-center for touchpad gestures

Touchpad gestures have only a single event sequence. The current
'center' of the gesture is the starting point + accumulated deltas.
Update gtk_gesture_get_bounding_box_center() accordingly.

Fix a few bugs in the css change flag definition

These were found by the new change flag tests.

Merge branch 'fix-imwayland-cursor' into 'master'

imwayland: Fix get_preedit_string cursor position (GTK4)

Closes #2383

See merge request GNOME/gtk!1328

imwayland: Fix get_preedit_string cursor position

GtkIMContext get_preedit_string should return cursor position counted
in characters, but cursor_begin here is counted in bytes. This add the
missing conversion.

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

Preliminary css matcher work

See merge request GNOME/gtk!1326

cssmatcher: Change get_state to has_state

This matches all the other vfuncs, and it gets us
out of keeping several big state flag enumerations
in sync.

Update all callers.

Allow printing css matchers

This can help in debugging css matching.

css: Add a matcher type enum

For now, this just replaces the is_any boolean
by a type field in the class. It will be used in
future commits.

css: Simplify the superset matcher

We use the superset matcher in exactly one place,
so there is no need for the generality of allowing
to ignore different aspects. Just hardcode the one
case we need: ignoring everything except for name,
id and class.

Cosmetic change

Update comments for name changes that happened long ago.

css: Fix the ANY matcher state list

The ANY and matcher was not, in fact, matching any state,
since the list of states was not up-to-date.

The same fix applies to the superset matcher as well.

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

Better css change tracking

See merge request GNOME/gtk!1325

Remove an unused define

We no longer use the GTK_CSS_CHANGE_RESERVED_BIT define.

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