expander: Rename CSS nodes

The expander icon is renamed from "arrow" to "expander".
The expander widget itself is renamed from "expander" to
"expander-widget" (Better ideas welcome).

This makes it possible to have an "expander" icon in more places then
the GtkExpander widget (in particular in tree lists) and not
confuse it with arrows.

cssimagebuiltin: Don't create cairo nodes for empty icons

GTK_CSS_IMAGE_BUILTIN_NONE is not going to draw anything of course, so
don't bother creating an empty cairo node for it.

gl renderer: Fix an out of bounds read

Fixes #2200

notebook: Avoid a crash

This is fallout from turning the menu into
a popover, causing it to be be a child.

glyph cache: Fix handling of big glyphs

We were putting big glyphs in the cache, in their
own texture, but forgetting to mark the texture
as permanent, so it could be reused, leading to
occasional misrendering. Fix this by marking these
textures as permanent, and explicitly freeing them
when the cache entry gets old.

shadow cache: Remove outdated comments

No comments are better than outdated comments.

gl: Increate the cache check frequency

Otherwise, we spread the cache over more atlases
than necessary, increasing the amount of texture
changes in each frame.

gl: Interleave cache aging

Every few frames, we do extra work for the
cache aging. Arrange for the glyph and icon
caches to not cause extra work on the same
frame, to smooth things out.

gl: Improve debug spew for caches

Dump similar information for both caches,
and correct the unused percentage for
the atlases.

Update stackswitcher CSS

Add the spacing to circular stack switchers back.

stackswitcher: Inherit from GtkWidget

colorscale: Remove some duplicated code

menuitem: Remove unused members

menu: Align function parameters

menu: Remove unused members

menu: Remove menu_queue_resize

The have_layout flag is unused, so this is equal to a normal
gtk_widget_queue_resize().

Merge branch 'glyphcache-fiddling' into 'master'

Glyphcache fiddling

See merge request GNOME/gtk!1132

gl: Make icon cache work like glyph cache

Replace timestamp tracking with an accessed bit
here too, to keep the glyph and icon cache code
similar.

gl: Do less work on glyph caching

There is no need for us to be very precise about
aging the glyph entries. It is enough to check
occasionally and mark old entries. This reduces
the overhead of work we do every frame on the
caches, at the cost of letting glyphs linger
a bit longer in the cache.

iconcache: Fix icon padding

gl: Simplify glyph cache lookup

Make this function more similar to the icon
cache equivalent, and simplify it a bit. We
don't use the boolean return, and we don't need
to look at the age of entry when marking it
used.

gl: Don't use memcmp for comparing cache keys

Some innocent change made us use a stack-allocated
key, and things broke. Lets go back to comparing
cache keys field by field.

Make the scrolling demo more versatile

Allow to swap out the content, so we can compare
text and icon scrolling.

gl: Handle row stride for icon cache upload

Same as the previous commit: Downloading a texture
may in theory give us data with a stride, so handle
that.

gl: Handle row stride for glyph cache upload

In theory, we can have data with a stride here,
so set the necessary parameters to tell GL about
it.

gl: Do less work to maintain caches

Remember which atlases were removed, and only
check those when looking for icons or glyphs
to remove. For most frames, we don't have to
check at all since no atlases were removed.

gl: Speed up icon caching

Avoid expensive padding, and just upload the
image in several slices.

Add an icon scrolling demo

This is good to exercise the GL renderer icon cache.

gl renderer: Get blur node child only once

gl renderer: Shorten function

gl renderer: Avoid copying a rect

gl: Avoid stray use of doubles

Everything else in this code is floats,
so stick to that and avoid unnecessary
precision.

Cosmetics

gl: glyph cache tweaks

Reduce the cost of lookups by storing
the hash value directly.

gl: Shrink CachedGlyph structs slightly

Plug a hole in this struct.

gsk: Store color bit info in text nodes

Keep the 'has color glyphs' info in text nodes,
instead of determining it over and over in both
the vulkan and gl backends.

gl renderer: Remove unused modelview matrix

gl renderer: Don't copy outset shadow node outline

gl renderer: Don't copy colors into render ops

snapshot: Don't move shadows too much

dx/dy are relative to the node bounds, which are already moved by x/y.

gl renderer: Grow unblurred outset shadow outline on the gpu

gl renderer: Fix glsl rounded rect shrinking

Previous code would add rounded corners to a rect with all 0 corners
when growing.

gl renderer: Cosmetics

gl renderer: Only add outset shadow center piece if dx/dy != 0

Updated Spanish translation

Merge branch 'wip/chergert/avoid-renderop-copy' into 'master'

gl: avoid copying RenderOp

See merge request GNOME/gtk!1128

gl: avoid copying RenderOp to GArray

Instead of copying the (rather large) RenderOp to the GArray, we can
simply set the fields directly in the allocated space for the struct.
In most cases, there wont be any allocations to make as the array size
is kept in tact across frame renderings.

gl: use memcmp to compare glyph cache keys

We can just use memcmp here because even in the use of lookup keys with
C99 initializers, we can rely on any space between fields added by the
compiler to be zeroed. So we might as well use wider memory cmopares.

Merge branch 'wip/chergert/const-glyph-cache' into 'master'

gl: avoid copying GskGLCachedGlyph in lookup

See merge request GNOME/gtk!1127

gl: avoid copying GskGLCachedGlyph in lookup

This saves a minor amount of CPU time by avoiding the copy of structure
on each lookup (which is short-lived).

textview: use gtk_snapshot_new_with_parent()

As recommended by Timm Bäder at:

https://gitlab.gnome.org/GNOME/gtk/commit/3b959456ac5335639e9642571f59f17ec8ee97f7#note_621655

gl: short-circuit on NULL program

The NULL check is a more inclusive check than each of the individual
op->op checks.

Merge branch 'wip/chergert/textview-fix-selection' into 'master'

textview: fix off-by-one in y_range invalidation

See merge request GNOME/gtk!1126

textview: fix off-by-one in y_range invalidation

Previously, with selection bounds, we could have a state where a line
display with selections set would eroniously stay in the cache. This
aggresively releases those at the boundary cases fixing deselection
drawing state.

gl renderer: Remove rounded rect intersection code

Caused correctness issues.

Fixes #1917

widget: Compare adjusted sizes in size_allocate

gl renderer: Fix scaled fallback node drawing

inspector: Don't add padding to spinbuttons

That looks stupid.

spinbutton: Fix adjustment value thinko

Using ints here is wrong as it makes it impossible to e.g. edit a double
in the [0; 1] range.

Adwaita: add statusbar padding back

We removed the widget margins from the ui file over 2 years ago.

statusbar: Use a bin layout

widget-factory: Remove double border around textview

Replace the scrolledwindow border with two separator on top/bottom.

stacksidebar: Inherit from GtkWidget

The child widgets are completely managed by the stacksidebar itself, so
this has no business being a GtkBin.

Adwaita: Remove superfluous border from stacksidebar

We already get a border from .sidebar

gtk-demo: Fix up sidebar demo style

Remove a superfluous separator and add the icon-dropshadow class to the
gtk logo

popovermenu: Switch to main submenu before mapping

popovermenu: Only close if there's a new focus widget

Makes sense and otherwise we end up closing the popover for no reason

popovermenu: Switch back to "main" AFTER unmapping

Otherwise we might set things to child visible etc. while being
unmapped, which is a violation of the widget invariants of these
properties.

popover2.ui: Stop trying to set GtkModelButton:label

The property is called "text"

testmodelbutton: Remove toggles for removed properties

filechooserbutton: Only destroy existing native dialogs

widget: Create finalize assertions in destroy()

demo: Don't manually add labels to menubuttons

demo: Fix modelbutton demo

label: Use TRUE/FALSE instead of 1/0

dialog: Fix action button rearrangement

The gtk_widget_get_parent() check does not work anymore since the
headerbar adds the buttons to a child box.

widget-factory: Skip xml files in background selection dialog

The default contains a xml file for an animated background, so don't try
loading it as a pixbuf.

widget: Plug layout manager leak

modelbutton: Rework

Create all the widgets on demand and use a box layout to arrange them
instead of manual size allocation.

Also don't inherit from GtkButton

button: Remove unused signal enum members

modelbutton: Use a box layout

modelbutton: Replace map() with root()

modelbutton: Remove end_box

it's unused.

modelbutton: Create accel label on demand

Adwaita: Blue check/radiobuttons

Align with 3.24.

Tiny refactoring

- use checkradio_fg_color for check/radio in treeview, which points to fg_color, so no change of the hex color, thus no change to the parsed CSS

Adwaita: Add color defines from 3.24

Adwaita: make links in infobars legible

- No longer tinted blue, make legible

Fixes https://gitlab.gnome.org/GNOME/gnome-software/issues/751

adwaita: Use border-spacing to style checkbuttons

radiobutton: Fix a warning in a code sample

label: Remove set_selectable_hint

It's just updating the cursor, so do it via update_cursor()

label: Remove gtk_label_realize

We don't need this to update the cursor anymore.

label: Don't update cursor after unsetting select_info

update_cursor() doesn't do anything in the priv->select_info == NULL
case.

label: Change cursor even if unrealized

Cursor don't depend on that anymore.

placesviewrow: popup-menu returns a boolean

Merge branch 'wip/carlosg/tablet-invalid-reads-master' into 'master'

Fix invalid reads on tablet input (master)

Closes #2157

See merge request GNOME/gtk!1122

gdk: Avoid poking possibly freed memory

The event may end up freed after delivery, ensure to keep a ref in order
to emit the matching emulated crossed event matching a proximity event.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/2157

Merge branch 'wip/chergert/quick-fixes' into 'master'

renderer quick fixes

See merge request GNOME/gtk!1124

Merge branch 'wip/chergert/cache-text-render-node' into 'master'

textview: cache paragraph render nodes

See merge request GNOME/gtk!1125

textview: cache paragraph render nodes

We can avoid recreating a number of text nodes from render_para() on
sub-sequent runs if we cache the rendernode instead of just the
PangoLayout.

When used with GtkSourceMap, this can yield a ~7 FPS improvement during
smooth scrolling at the cost of some more memory.