wayland: Don't leak all surfaces

X11 does add an extra reference to surfaces that gets released when the
DestroyNotify event arrives.
Wayland doesn't ave such an event, so that reference never gets
released.

This fixes a copy/paste error introduced in commit 590f3dfa1fcb.

wayland: Don't insta-crash when a surface gets disposed

We want to remove the event queue from the list of event queues, not the
surface.
Otherwise the freed queue stays in the list and the next time an event
comes in, we access invalid memory.

Fixes thinko introduced in commit 7fafa5133b5.

Luckily, we leak all surfaces, so this problem never occured.

Merge branch 'focus-testsuite-fixes' into 'main'

text: Fix fallout from focus changes

See merge request GNOME/gtk!5841

testsuite: Actually wait for focus

We were failing to properly iterate the main
context here. Oops.

tests: Check is-focus instead of has-focus

We are not presenting windows here.

text: Fix fallout from focus changes

We don't get proper notification for when the toplevel
is-active property changes, so monitor has-focus, and
update cursor blinking.

Merge branch 'wip/otte/gles-fixes' into 'main'

GLES 2 fixes

See merge request GNOME/gtk!5837

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

window: Don't mark widget prematurely as has-focus

See merge request GNOME/gtk!5836

testsuite: Make GLES testsuite run use GLES 2

We want to support GLES 2, so make sure we test that support.

Also force-disable common extensions we don't explicitly check for and
don't want to accidentally use.

gles: Don't use vertex array objects

They're not needed and GLES doesn't technically support them, even
though GTK had been using them via epoxy sneakily using the
GL_OES_vertex_array_object extension behind our back.

text: Only claim primary when focused

It does not make sense otherwise, and has unfortunate
side-effects on Wayland.

wayland: Improve logging for primary selection

window: Don't mark widget prematurely as has-focus

has-focus is defined is-focus && toplevel::is-active.
We were forgetting to look at is_active when handling
focus widget changes.

Merge branch 'better-snapshot-nodes-docs' into 'main'

gtk: Improve documentation on returned nodes for snapshot

Closes #5747

See merge request GNOME/gtk!5831

Tweak the wording

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

inspector: Don't randomly emit application signals

See merge request GNOME/gtk!5834

listlistmodel: Add a cache

Cache the last looked up item and use it for looking up the next item if
it's closest. This massively speeds up iteration over the model, because
each call to get_item() will be adjacent to the previous one.

Improves performance of the inspector quite a bit.

testsuite: Add some more exhaustive testing to listlistmodel

inspector: Don't randomly emit application signals

When the variant-editor emits a callback, it might not actually have
edited the value in question. Try to detect that by only emitting
signals if the value changed.

Merge branch 'gbsneto/vulkan-buffer-release' into 'main'

gsk/vulkan/render: Download image before reset

See merge request GNOME/gtk!5832

gsk/vulkan/render: Download image before reset

gsk_vulkan_render_download_target() currently resets the uploader
objects before downloading the image that it produces. This is
problematic because there might be unreleased buffers and images
in the command queue.

In particular, this can make validation layers complain about the
glyph atlas - of all things! - upload buffer being released while
still being used by the command queue.

Fix that by resetting the uploader after downloading the image.

gtk: Improve documentation on returned nodes for snapshot

Closes: #5747

Merge branch 'fix_scalebutton_orientation' into 'main'

ScaleButton: fix CSS name in documentation, fix orientation doing nothing

See merge request GNOME/gtk!5819

Merge branch 'check-openuri-portal-version' into 'main'

openuriportal: Detect if the interface isn't there

Closes #5733

See merge request GNOME/gtk!5826

Merge branch 'gbsneto/vulkan-mailbox' into 'main'

gdk/vulkancontext: Use more appropriate present mode

See merge request GNOME/gtk!5828

gdk/vulkancontext: Use more appropriate present mode

Check if the driver supports MAILBOX and prefer using it; in its
absense, checkif the driver supports IMMEDIATE and prefer using
it; finally, if neither of them are supported, use the guaranteed
to be supported FIFO mode.

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

x11: Remove commented outdated code

See merge request GNOME/gtk!5827

x11: Remove commented outdated code

... and it's copy/paste into the win32 backend

gdk: Remove unused vfunc

Merge branch 'wip/otte/wl-gloria-gaynor' into 'main'

wayland: Make wl_surface survive until surface is desstroyed

See merge request GNOME/gtk!5825

Merge branch 'wip/exalm/print-dialog' into 'main'

printunixdialog: Remove a redundant style class

See merge request GNOME/gtk!5823

openuriportal: Detect if the interface isn't there

Check the portal version number before trying to use
it. Most importantly, this will detect the case where
the interface isn't supported at all, since the proxy
will report a version of 0 in that case.

Fixes: #5733

Merge branch 'bilelmoussaoui/g-i' into 'main'

g-i: Fix GtkGLArea type name

See merge request GNOME/gtk!5824

wayland: Keep EGL window around when hidden

See previous commit:

We want to keep resources around as long as the surface exists.

wayland: Don't destroy the wl_surface on hide()

We want to keep the wl_surface around, because surfaces create their
resources on construct and keep them until destroyed. See the HWND ond
Windows and the XWindow on X11.

This is relevant for graphics resources, where we want to have access
to the VkSurface and eglSurface while the GdkSurface is hidden.
We also want these surfaces to be permanent and not change during the
lifetime of the GdkSurface.

What we can - and must - destroy however are the xdg surfaces, because
those handle visibility on screen.
And we also need to ensure no buffer is attached, so that during the
next creation of the xdg surface we don't get a protocol error.

wayland: Split out a function

We have a create_wl_surface(), create a matching destroy_wl_surface().

wayland: Remove useless function

gdk_wayland_surface_maybe_resize() just calls
gdk_wayland_surface_update_size(), so make all callers call that one
instead.

The check that it does is done by the other function again.

wayland: Remove an old workaround

This workaround - were it ever to trigger - is broken today. It destroys
the wl_surface and all associated structs but does not recreate the
xdg_popup or xdg_toplevel struct, so it would cause a hidden window.

The workaround looked a lot different when it was introduced in commit
83b54bab5734f17d01633c1b5a0ce3e4896d2ee0, too - both in what it did and
in what the vfuncs did that it called.

g-i: Fix GtkGLArea type name

printunixdialog: Remove a redundant style class

.view does absolutely nothing in Default style since the whole box is
covered with a GtkNotebook which has its own background, and adds an
unwanted background onto the tab strip in Adwaita.

Merge branch 'dialogs-force-window-destruction' into 'main'

gtk/dialogs: Destroy the window promptly on finish async function

Closes #5741

See merge request GNOME/gtk!5820

gtk/dialogs: Destroy the window promptly on finish async function

Some bindings (GJS!) could add temporary references to the GAsyncResult
argument that we return, and thus to the GTask, which may cause the
dialog not to close when the finish function is called (but at garbage
collection instead!).

To prevent this, just manually destroy the window (by removing the task
data), so that we are not bound to the GTask lifetime anymore.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5741

Merge branch 'filechooserentry-fixes' into 'main'

filechooserentry: Make filtering work again

Closes #5743

See merge request GNOME/gtk!5822

Merge branch 'wip/carlosg/multi-click-text-view' into 'main'

textview: Reset press counter if double/triple clicking on a different line

See merge request GNOME/gtk!5821

Merge branch 'touch-no-double-click-threshold' into 'main'

gestureclick: Don't use threshold for touch

Closes #5580

See merge request GNOME/gtk!5798

Merge branch 'stylus-fixes' into 'main'

gtk-demo: Test stylus-only mode

See merge request GNOME/gtk!5804

filechooserentry: Plug a memory leak

filechooserentry: Make filtering work again

We need to look at the filchooser::filtered-out
attribute to know which files the filesystem model
has filtered away.

Fixes: #5743

textview: Reset press counter if double/triple clicking on a different line

If we click close enough between lines, and with the maximum distances applied
by GtkGestureClick we could jump between lines when handling double/triple
click for word/line selection.

Ensure that the whole operation stays in the same line and reset the
gesture/counter if we do move between lines, so we start from scratch in the
new line.

scalebutton: fix orientation not applied to scale

scalebutton: fix CSS name in documentation

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

filechooser: Fix the gridview

See merge request GNOME/gtk!5817

filechooser: Fix the gridview

2a70093a309481cf6 was a bit overeager in its porting
to GtkColumnViewCell. The gridview needs to keep its
list items.

Merge branch 'wip/corey/file-chooser-widget' into 'main'

More fixes to filechooserwidget

See merge request GNOME/gtk!5815

Merge branch 'wip/corey/filesystemmodel' into 'main'

filesystemmodel: Notify correct position removed in remove_file()

See merge request GNOME/gtk!5814

filechooser: Set date and time after cell is a child of filechooserwidget

The date/time column relies on the filechooserwidget to format the date
properly.  During bind, the filechoosercell, get the filechooserwidget
ancestor, but now due to changes in the listview, the cell isn't a
child of the filechooserwidget at that point.  Since this is deeply
ingrained into the filechooserwidget, let's keep the same behavior,
but move it to filechoosercell in realize.  Alternatively, we could have
used a signal factory (with the file chooser widget as the user data),
but that would have been a major overhaul.

filechoosercell: Store type_format in filechoosercell

The format of the type column depends on the the type_format, which
is stored in the filechooserwidget.  We get that setting by looking
for the filechooserwidget ancestor, which no longer works after recent
changes to the list views (it was fragile to begin with).  At one point,
the setting appears to have been dynamic, but now it is only loading
from GSettings, so let's simply do the same within FileChooserCell.

filechooserwidget: Finish port to GtkColumnViewCell

Some missing updates from f5dea9a3c217c49f44eedb18822be080d2451dc0

filesystemmodel: Notify correct position removed in remove_file()

32247bc50e63b709ae738a95a2ec71345b5fad75 made several changes to account for the
fact that we no longer have a NULL editable at the beginning of the list
model.  The commit mistakenly left out one change in remove_file(),
which causes the wrong file to be removed.

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

node-editor: Typo fixes

See merge request GNOME/gtk!5812

node-editor: Typo fixes

Fix some typos in the node format documentation.

Merge branch 'scrolled-window-critical' into 'main'

scrolledwindow: Avoid a critical

See merge request GNOME/gtk!5809

testsuite: Check widget properties better

Check that setting a property after resetting
it works. An instance of this was fixed in the
previous commit.

scrolledwindow: Avoid a critical

When setting the child property to NULL, we also need
to unset auto_added_viewport, to avoid triggering a
critical when setting it again.

Merge branch 'warn-show-destoyed-window' into 'main'

Show a warning when a destroyed window is shown

See merge request GNOME/gtk!5122

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

gtk-demo: Add a keyword

See merge request GNOME/gtk!5805

gtk-demo: Stop using gtk_widget_get_allocation

gtk-demo: Add a keyword

Make the demo using the 'bluroverlay' come up when you type 'blur'.

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

range: Fix a copy-paste error

See merge request GNOME/gtk!5803

gesture stylus: Fix condition

Now that the paint demo lets us test this, it has
become apparent that this condition is wrong, and
we don't get the expected events if stylus-only is
FALSE.

gtk-demo: Work without stylus

In the paint demo, don't assume that the event
backlog contains pressure. It won't, if we are
working with a plain old mouse.

gtk-demo: Test stylus-only mode

Add a checkbutton to toggle the stylus-only
mode of GtkGestureStylus, so we can test this.

placessidebar: Stop using gtk_widget_get_allocation

coloreditor: Stop using gtk_widget_get_allocation

range: Fix a copy-paste error

This was showing up as the alpha popup in
the color editor not being positioned correctly.

Merge branch 'gbsneto/vulkan-glyph-fixes' into 'main'

Vulkan glyph fixes

See merge request GNOME/gtk!5802

gsk/vulkan/glyphcache: Rework glyphs

The current implementation of the glyph cache deals with atlases by
padding them with 1 pixel at the beginning, at the end, and between
each glyph.

That's cool and all, however, there's a very subtle problem with
this approach: the contents of the atlas are garbage, so this padding
is filled with garbage memory!

Rework the Vulkan glyph cache to draw each and every glyph in a
surface that has 1 pixel border of padding around it. Ensure the
surface is completely black by drawing a rectangle before handing
it to Pango to draw the glyph. Update tx and ty to pick the texture
position adjusted to the 1 pixel padding. The atlas now starts at
position (0, 0), since each glyph individually contains its own padding.

To improve legibility, add a PADDING define and use it everywhere.

gsk/vulkan/pipeline: Simplify pipeline creation

Nothing uses gsk_vulkan_pipeline_new_full() anymore.

gsk/vulkan: Use default blend factors

Vulkan renders text using VK_BLEND_FACTOR_ONE_MINUS_SRC_ALPHA and
VK_BLEND_FACTOR_SRC_ALPHA, but that implies per-channel alpha
blending, which currently produces the wrong results when blending
glyphs with the images beneath them.

Use the default pipeline constructors, which implies using the
ONE and ONE_MINUS_SRC_ALPHA.

gsk/vulkan: Cosmetics

Reorder code to match the order used in shaders.

Tweak the wording of the message.

gestureclick: Use drag threshold for updates

gestureclick: Don't use threshold for touch

When determining double-clicks, don't use the distance
threshold for touch events. It is very hard to double
touch reliably within a few pixels of the same position.

Fixes: #5580

Merge branch 'wip/carlosg/popover-ungrab-on-unmap' into 'main'

gtkpopover: Move GTK grabs to map/unmap

Closes #5730

See merge request GNOME/gtk!5801

gtkpopover: Move GTK grabs to map/unmap

Typically, a popover gets mapped when shown and unmapped when
hidden. A situation there that breaks is where the popover gets
recursively unmapped/unrealized when its root is destroyed.

In that situation, the popover does however unmap (without being
hidden first), moving the GTK grab from show/hide to map/unmap
will handle the previous situations, plus this one.

Fixes things being unclickable if e.g. a modal dialog got a
popover popped up, then got closed via Alt-F4.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5730

Merge branch 'inspector-action-fixes' into 'main'

actionmuxer: Make the compiler warn

See merge request GNOME/gtk!5800

inspector: Redo the actions list

Make the actions list implement GtkActionObserver
for the muxer case, and make updates work.

inspector: Simplify the action editor

No need to listen for changes, the actions list
code will do so and tell us to update.

inspector: Add ActionHolder::changed

We will use this signal to communicate
action changes to the action list widgetry.

inspector: Cosmetics

Make the variant editor use a checkbutton
instead of a togglebutton.

actionmuxer: Add a precondition

This helps with debugging the inspectors action
handling.

Update Bulgarian translation

(cherry picked from commit f15aa82eaf4a8d9498b53e3e416d2ac21d6b24f4)

inspector: Be more careful

Tweak the variant-editor code to handle some
cases more explicitly.

inspector: Plug a memory leak

inspector: Be safer when handling actions

Those query apis are serious about not ignoring
their return value, and may not set their our
arguments to anything if they return FALSE.

actionmuxer: Make the compiler warn

Warn when the boolean return isn't used, since we
may not initialize the out arguments in the FALSE
case (see the previous commits).

textview: Handle a return value

Merge branch '5729_fix_inspector_crash' into 'main'

inspector: fix crash caused by uninitialized GVariant

Closes #5729

See merge request GNOME/gtk!5799

inspector: fix crash caused by uninitialized GVariant

we were not checking the return gboolean of
gtk_action_muxer_query_action() which was
returning FALSE for the crash case, meaning
it didn't set the passed in GVariant, but
we were still using it as it was non-null.

Fixes #5729

Merge branch 'media-controls' into 'main'

mediacontrols: Add tooltip text to play button

See merge request GNOME/gtk!5794