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()

cssimagerecolor: Implement is_computed()

gl renderer: Move rect transformation to the vertex shader

No need to do this for every fragment.

gl renderer: Transform rounded rect on the GPU

Change the RoundedRect struct we use in our shaders so we can transform
it using (affine) matrices.

colorbutton: Use a css name

This was the sole widget to show up as 'widget' in
the css tree.

testsuite: Update mark names

We've changed some of the profiler mark names to
be clearer and more unique. Update the tests that
look for those marks to use the new names.

Adwaita: Use child selectors in many more places

These are much cheaper to match than descendent
selectors, so use them whenever we know the element
is a direct child.

text: Don't compute invisible char in init

This causes us to create a pango layout and
validate the css style, a high-overhead
operation. Just do it when the entry is set
to be invisible.

Fix Emoji completion

This was broken in various ways; Tabbing between pages
was causing segfaults, and using the hover state like
this does not work anymore, with hover now being completely
managed by GTK. Use focus instead, and also fix up the
style.

Replace .emoji with emoji for css

Adwaita: Drop .context-menu

We are not using it anymore, and yet fonts on a bold
textview are fine. So no need for this.

Adwaita: Just use .menu, instead of .appmenu

No need for this one-off class, there's no other menus
that could show up here.

Merge branch 'fix-avahi_service_resolver_cb-crash-master' into 'master'

printing: Fix crash in avahi_service_resolver_cb (master)

See merge request GNOME/gtk!1347

printing: Fix crash in avahi_service_resolver_cb

printer_name_compressed_strv is NULL-terminated array
of gchar*, which means N+1 memory should be allocated.

Otherwise, if the printer name has no empty components
(which is usually the case), printer_name_compressed_strv[N],
which should contain the NULL sentinel, will actually lie
just outside of allocated memory, which is UB.

In my case, it led to crashes inside g_strjoinv
when Print... dialog is opened in evince.

    #0  0x00007fad2ce1bad7 in __strlen_avx2 () at ../sysdeps/x86_64/multiarch/strlen-avx2.S:96
    #1  0x00007fad2d04d88d in g_strjoinv (separator=separator@entry=0x7fad0c9bc508 "-", str_array=str_array@entry=0x556b017f0200) at ../glib-2.60.7/glib/gstrfuncs.c:2585
    #2  0x00007fad0c9b8a89 in avahi_service_resolver_cb (source_object=<optimized out>, res=<optimized out>, user_data=0x7fad08020ee0) at /var/tmp/portage/x11-libs/gtk+-3.24.13/work/gtk+-3.24.13/modules/printbackends/cups/gtkprintbackendcups.c:3223
    #3  0x00007fad2d1f8ed3 in g_task_return_now (task=0x556b017a8b00 [GTask]) at ../glib-2.60.7/gio/gtask.c:1209
    #4  0x00007fad2d1f987d in g_task_return (task=0x556b017a8b00 [GTask], type=<optimized out>) at ../glib-2.60.7/gio/gtask.c:1278
    #5  0x00007fad2d1f9dec in g_task_return (type=G_TASK_RETURN_SUCCESS, task=<optimized out>) at ../glib-2.60.7/gio/gtask.c:1678
    #6  0x00007fad2d1f9dec in g_task_return_pointer (task=<optimized out>, result=<optimized out>, result_destroy=<optimized out>) at ../glib-2.60.7/gio/gtask.c:1683
    #7  0x00007fad2d24b6af in g_dbus_connection_call_done (source=<optimized out>, result=0x556b017a8bc0, user_data=0x556b017a8b00) at ../glib-2.60.7/gio/gdbusconnection.c:5747
    #8  0x00007fad2d1f8ed3 in g_task_return_now (task=0x556b017a8bc0 [GTask]) at ../glib-2.60.7/gio/gtask.c:1209
    #9  0x00007fad2d1f8f09 in complete_in_idle_cb (task=0x556b017a8bc0) at ../glib-2.60.7/gio/gtask.c:1223
    #10 0x00007fad2d02d2c0 in g_main_dispatch (context=0x556b00eee090) at ../glib-2.60.7/glib/gmain.c:3189
    #11 0x00007fad2d02d2c0 in g_main_context_dispatch (context=context@entry=0x556b00eee090) at ../glib-2.60.7/glib/gmain.c:3854
    #12 0x00007fad2d02d658 in g_main_context_iterate (context=context@entry=0x556b00eee090, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at ../glib-2.60.7/glib/gmain.c:3927
    #13 0x00007fad2d02d6df in g_main_context_iteration (context=context@entry=0x556b00eee090, may_block=may_block@entry=1) at ../glib-2.60.7/glib/gmain.c:3988
    #14 0x00007fad2d22248d in g_application_run (application=0x556b0116f130 [EvApplication], argc=<optimized out>, argv=<optimized out>) at ../glib-2.60.7/gio/gapplication.c:2519
    #15 0x0000556b002e55a1 in  ()
    #16 0x00007fad2ccd6f1b in __libc_start_main (main=0x556b002e50d0, argc=2, argv=0x7ffe1057fa88, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7ffe1057fa78) at ../csu/libc-start.c:308
    #17 0x0000556b002e567a in  ()

    (gdb) p printer_name_compressed_strv[0]
    $4 = (gchar *) 0x556d4a4be430 "Brother"
    (gdb) p printer_name_compressed_strv[1]
    $5 = (gchar *) 0x7f9dbc011090 "MFC"
    (gdb) p printer_name_compressed_strv[2]
    $6 = (gchar *) 0x556d4a51ba50 "7860DW"
    (gdb) p printer_name_compressed_strv[3]
    $7 = (gchar *) 0x401 <error: Cannot access memory at address 0x401>

Add profiler marks around css selector tree building

This is a big part of theme loading, and worth
showing in traces.

Adwaita: Remove .entry-tag class

Not used in GTK.

Adwaita: Remove .rubberband class

All GTK widget are using a rubberband element now.