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.

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.

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

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

Make wayland load cursors on demand

Copy just enough of libwayland-cursor to make our own
loading. This lets us drop the dependency on libwayland-cursor,
and changes the startup cost for cursor theme loading
from 25ms to 0.1ms.

At the same time, simplify the handling of scaled cursors -
instead of creating an array of theme objects, just make a
single theme object provide all scaled cursor sizes.

Revert "css: Print selectors"

This reverts commit da5d5243241941ba130fc0644e66986f967a2ef3.

And this one too. The commits are there, if people need this
kind of statistics again, they can just be cherry-picked.

Revert "css: Print out relative costs of selectors"

This reverts commit 5dcce0c0bdccf5b55a15ff45247b2fc06bf77f7f.

I forgot to ifdef a few things in here, so lets take this out
again.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1348

Adwaita: Drop selection mode

This is not really a feature that GTK has; and it lets
us drop a significant amount of css.

Adwaita: Drop menu remnants

Menus are gone.

css: Print out relative costs of selectors

Count how often each tree node is visited, and print the number
at the end. This gives a good indication what selectors are costly
and should be avoided. #ifdefed out.

Adwaita: Remove a wildcard rule for tooltips

These rules are expensive, and don't seem to make
any difference for typical tooltip content.

Adwaita: Drop menuitem support

menuitems don't exist anymore.

css: Print selectors

Add code that prints all the selectors when a theme
is loaded. #ifdefed out.

widget: Use gsk_transform_transform_point() in gtk_widget_pick()

This also covers the 2D_AFFINE case.

gl renderer: handle cross-fade nodes with invisible children

gl renderer: Handle blur nodes with invisible children

gl renderer: Make render_rounded_clip_node clearer

testsuite: Add test cases for last commit

Easy enough to test

gl renderer: Add builder offset correctly for non-affine modelviews

GskTransform: Add gsk_transform_transform_point()

Equivalent of gsk_transform_transform_bounds() and
graphene_matrix_transform_point() respectively.

gl renderer: Shorten shaders a bit

cssimageicontheme: Avoid copying colors

cssimagepaintable: Implement is_computed()

cssimagescaled: Implement is_computed()