animatedstyle: Fix the debug build

This wasn't caught by our ci.

Merge branch 'gtk-4-im-wayland-controller-focus-in' into 'master'

imwayland: Connect gtk_im_context_focus_in/out to GtkEventControllerKey

See merge request GNOME/gtk!1349

Merge branch 'gdk-device-leak-fix' into 'master'

GdkDevice leak fix

See merge request GNOME/gtk!1358

Merge branch 'css-style-split-2' into 'master'

css: Split styles into groups

See merge request GNOME/gtk!1362

Merge branch 'reset-with-click' into 'master'

Always emit reset for IM preedit with mouse click

Closes #1534

See merge request GNOME/gtk!460

Merge branch 'robertroth/shortcuts-overlay-accel' into 'master'

Application: Do not reuse Ctrl+F1 for help-overlay

See merge request GNOME/gtk!307

Use style values directly

Replace uses of gtk_css_style_get_value with direct access,
throughout the tree. We don't replace all uses, just those
where we are dealing with a fixed property. Be careful to
handle the currentColor special case for color properties.

Compute style change based on values structs

We can save some time here by comparing struct by
struct and avoiding individual values comparisons
as much as possible.

css: Split styles into groups

Introduce refcounted structs for groups of related css properties,
and use them to store the style values. Both GtkCssStaticStyle and
GtkCssAnimatedStyle fill in the structs in GtkCssStyle, and we
can avoid vfuncs for value access, which should be much faster.
We can even start accessing style->core->color directly.

performance test: Produce better numbers

Try to isolate the test runs from each other in the
eyes of the kernel scheduler, by sleeping a bit. And
ignore the first run, to avoid cache effects.

Merge branch 'wip/baedert/widget-add-style-class' into 'master'

widget: Add style class API

See merge request GNOME/gtk!1366

Silence a compiler warning

Merge branch 'gtk-doc-ci' into 'master'

Publish the API reference

See merge request GNOME/gtk!1367

Remove reftests from expected-fail list

With an updated version of the base CI image it seems some tests that
were failing have started to pass again.

ci: Deploy the API reference on our GitLab pages

Now that we can build gtk-doc as a sub-project, we can go back to
publishing the API reference for the master branch every time we
build it.

ci: Update the version of Meson in the CI image

We need at least 0.52.1 to be able to use gtk-doc as a sub-project.

Add gtk-doc as a subproject

We rely on a specific minimum version of gtk-doc to be able to build the
GTK API reference for the new API. In order to be able to use gtk-doc as
a subproject, though, we need to use a recent version of Meson.

widget: Add style class API

Add GtkWidget API for adding and removing style classes, as well as
checking whether a widget has a style class applied.

Everyone has to go through GtkStyleContext for this these days but with
GtkStyleContext eventually going away, it makse sense for GtkWidget to
have API for this.

Merge branch 'fix-doc-typo' into 'master'

Fix a doc typo

See merge request GNOME/gtk!1364

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

Wip/otte/for master

See merge request GNOME/gtk!1365

cssselector: Rework how we handle the bloom filter

Instead of foreaching through all the previous selectors every time we
bloom-filter, just bloom-filter the current element and return a special
value if that filter fails (FALSE). If that happens, don't try
filter-matching more nodes in the caller as we know it's an abort.

Fix a doc typo

cssselector: Reorder functions

This just changes the order of functions in the source code in
preparation for the next commit.

css: Don't do the "all set" test

We have so many properties that it is basically impossible that all of
them are set and the time spent checking is higher than the time saved
if it does indeed happen.

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

popover: Do not validate the css node on show()

See merge request GNOME/gtk!1363

popover: Do not validate the css node on show()

This was a good idea back in GTK3 when popovers were toplevels, but now
they're regular child widgets, so they should behave that way.

Also, with the introduction of the bloom filter, gtk_css_node_validate()
now assumes it's only called on root nodes, so assert that that is the
case.

imwayland: Connect gtk_im_context_focus_in/out to GtkEventControllerKey

GtkEventControllerKey can handle GDK_FOCUS_CHANGE and call
gtk_im_context_focus_in/out directly.

https://gitlab.gnome.org/GNOME/gtk/issues/2390

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

selector: Rework iterating over subnodes

See merge request GNOME/gtk!1361

selector: Rework iterating over subnodes

Instead of a foreach() function, introduce an iterator, so that the
caller can drive the iteration.

This allows doing stuff inbetween callbacks and avoids closures when
more than one data object should be passed.

As a side effect I even get a small, but noticeable performance
improvement in the 2-10% range depending on benchmark, I guess that's
because there's no function pointer passing going on anymore.

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

CSS improvements

See merge request GNOME/gtk!1360

styleproperty: Remove ability to query properties

With gtk_style_context_get() gone, this is no longer used.

stylecontext: Get rid of gtk_style_context_get()

... and replace all the calls to it with
gtk_style_context_peek_property().

All the calls are hacks that need to go away btw.

testcalendar: Don't do dumb stuff with font sizes.

If you want to test font stuff, it's 2020 and we have an inspector.

filechooser: Remove default size machinery

The FileChooser ToolKit (fctk) had its own machinery to handle default
sizes which was completely busted and trying to marshal random numbers
through the widget hierarchy that maybe made sense in 2012 but don't do
now.

Get rid of it, just keep the dialog's GSetting - which funnily enough
used to be written by the dialog but written by the widget.
But that's fctk for you.

cellrenderertoggle: size based on -gtk-icon-size

Instead of hardcoding 16px.

stylecontext: Remove gtk_style_context_get_section()

The inspectors gets values differently these days.

stylecontext: Remove GTK_STYLE_PROPERTY_* defines

They shouldn't be (and aren't) used anymore.

css: Use the bloom filter for change matching

Instead of just doing radical change matching on the node itself, also
consider the parent nodes via the bloom filter.

This means a radical change is now also one where the parent
name/id/classes change, but since that's considered a radical change on
the parent already, those things are slow anyway.

Improves the benchmark times for CSS validation during backdrop
transitions in widget-factory from 45ms to 35ms on my machine.

selector: Make :not() selectors not radical

:not() selectors cannot be radical because the bloomfilter only knows if
a value is set in any of the nodes, but cannot determine the opposite
(if a value is not set in at least one node), but that would be required
for:not() selectors.
However, this is very unlikely to happen in the real world, so it's not
worth optimizing.

Unfortunately, change tracking could know this, so by excluding the
:not() selectors from radical changes, the change tracking will now pick
them up. If that turns out to be a performance problem, we need to add a
special category for radical not filters, so change tracking and bloom
filters can deal with them.

The testcase demonstrating the problem in widget-factory has been
extrated and added.

testsuite: Improve diff_with_file() function

Properly handle diff(1) failing.

In this particular case, the test passed a NULL input file to the diff
(that was fixed, too) and then diff only found one input file and
aborted.

But without this fix, we'd also not catch other abortion reasons for
diff() - as long as it exited in any way, we were happy.

css: Add fast-path for parent selector matching

Add a fast path for parent selector matching that uses a bloom filter to
quickly discard selectors that can't possibly match.

Keep in mind that we match using a bloom filter, so we might
accidentally include too many selectors when hash/bucket collisions
occur.
That's not a correctness problem though, because we'll do a real check
afterwards.

The idea for this change is taken from browsers, in particular WebKit.

cssnodedeclaration: C99-ify the code

zero-sized arrays are allowed now.

selector: Hash differently

This will be relevant for a bloom filter. And bloom filters want 12bit
hashes, so we try to produce hash values < 4096.

cssnode: Convert name + id from interned string to GQuark

The reason for this is simply that I want to get hash functions that
have their values close together, so they can fit in a smaller range
(the goal here is 12 bits). By using GQuark, we get consecutive numbers
starting with 1 (and applications have <1000 quarks usually), whereas
interned strings can be all over the place.

As a side effect we also save 64 bytes per declaration.

css: Get rid of GtkCssMatcher

css: Pass node through the lookup code

... instead of passing a matcher.

The matcher code is now local to the CssProvider/CssSelector machinery.

cssnode: Remove init_matcher() vfunc

It's unused.

cssnode: Remove ability to set GType as fallback name

Remove GtkWidgetPath

... and all associated demos and tests.

Remove gtkwidgetpath.h include where no longer needed

Revert "cssmatcher: Inline node values into matcher"

This reverts commit a75529f3c00736856fa20f19e920614be5a48fbb.

It just complicates code and there are no performance numbers that would
warrant such a commit.

css: Move utility function

The function was not selector-specific, so putting it with all the other
utility functions makes more sense.

Also use the utility function in the node declaration printing.

gdk/x11: Properly unref removed device in XI2 device manager

When a device is added, there are two references to it by the device
manager, the initial one and the one used for the id_table. Removing a
device only removed the reference added by the id_table resulting in the
GdkDevice being leaked.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/1358

textutil: Stop using GtkStyleContext

We can just use the css style directly.

gskpango: Don't free rgba's we don't own

We don't copy them anymore, so don't free them.

filechooser: Stop using gtk_style_context_get

Just go to the css style directly.

Update Brazilian Portuguese translation

popover: Fix an oversight in gap coord computation

This caused a border to show up between the popover arrow and the
contents.

calendar: Use widgets for the header

This is not meant to be a full GtkCalendar conversion to use widgets
instead of custom drawing, but we lost the arrows in the calendar header
when builtin icons were removed. Using proper button for the year/month
buttons brings them back.

Update Malay translation

Update Malay translation

Update Friulian translation

Update Slovak translation

css: Don't lose inset when computing shadows

This was showing up as reftest failures,
sporadically

gdkseatdefault: Unref removed slave devices

Adding a slave device adds a ref, but removing it does not unref it.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/1358

gsk: Remove some unused defines

These are from the GVariant days of render nodes.

cssdimensionvalue: Remove an outdated comment

It's actually implemented in gtkcssdimensionvalue.c these days.

cssboxes: Try to avoid doing work for rectangular boxes

Use the new gtk_css_dimension_value_is_zero() to check if we really need
to e.g. apply a border radius at all.
We compute css boxes a lot so this makes sense here, it especially shows
up during pick(), where we need the border box.

dimensionvalue: Add gtk_css_{dimension,corner}_value_is_zero()

We can usually omit computations when the dimension value ends up being
zero anyway.

widget: Restructure pick() a bit

Don't even invert transforms if we can do it easily ourselves. This also
fixes a small GtkTransform memory leak.

renderbackground: Avoid unnecessary work in snapshot_background()

Add pure and const annotations to various functions

render: Remove gtk_render_background_get_clip

Remove gtkutils{.c,private.h}

All but one functions are unused. Move the remaining one into
gtkmodules.c, its only caller.

cssparser: make a few local variables const

cssprovider: Parse selectors without a GList

The maximum length of this list in Adwaita is 18, so use a generous
maximum length of 64 for the selector list of a ruleset.

GtkCssImageUrl: Implement is_computed()

gl renderer: Avoid some work when rendering border nodes

cssimage: Add gtk_css_image_to_string()

In line with all the other _to_string() implementations

window: Replace GtkStyleContext use to get surface transform

We have GtkCssBoxes for this these days.

Revert "lookup stats"

This reverts commit 3d9e3390f167a220e8775d08356259f420e045ce.

Revert "css: Split style into groups"

This reverts commit 0df0de0b5db87248ad1371ee1bfc8a733ec079e4.

css: Split style into groups

Adwaita: take out an unnecessary rule

The comment says setting caret-color to currentColor
shouldn't be necessary, and it is right.

Adwaita: drop the *

lookup stats

Fix a scrollbar sizing issue

The recent Adwaita changes inadvertendly made scrollbars
resize on hover. Fix that, by reinstating some lost rules.

Revert "Ensure icon themes are loaded with other themes"

This reverts commit 88d26b75491c9a329a836c0d91a8f85ee3bc77d1.

This change caused GtkShortcutWindow to not work anymore,
and we are fixing icon themes differently, anyway.

Merge branch 'style-context-redux' into 'master'

Stop using GtkStyleContext internally

See merge request GNOME/gtk!1353

popover: Stop using style context getters

drag icon: Stop using style context getters

Just go to the css node directly.

gskpango: Stop using style context getters

Just go to the css nodes directly.

widget: Stop using style context internally

Add a profiler mark around renderer realization

This takes time that is worth calling out.

Adwaita: Fix some entry styling

This was accidentally broken when parent selectors
were introduced here.

GtkSpinner: Pay attention to style changes

Make GtkIcon redraw and resize when style changes
require it.

GtkPaned: Pay attention to style changes

Make GtkPaned redraw and resize when style change
require it.

GtkIcon: Pay attention to style changes

Make GtkIcon redraw and resize when style changes
require it.

css: Update affects flags for icons

We want to differentiate what requires recreating the
texture and what doesn't. In particular, the current
flags are not handling symbolic icons right.

Merge branch 'wayland-cursors' into 'master'

Make wayland load cursors on demand

See merge request GNOME/gtk!1350