gdk/win32: fix `hr` set but not used

A number of warnings are produced:

[23/1038] Compiling C object gdk/win32/libgdk-win32.a.p/gdkinput-dmanipulation.c.obj
../gdk/win32/gdkinput-dmanipulation.c: In function 'reset_viewport':
../gdk/win32/gdkinput-dmanipulation.c:354:11: warning: variable 'hr' set but not used [-Wunused-but-set-variable]
  354 |   HRESULT hr;
      |           ^~

Try to do something sensible instead.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix g_string_free warning

../gdk/win32/gdkclipdrop-win32.c: In function 'transmute_cf_shell_id_list_to_text_uri_list':
C:/msys64/ucrt64/include/glib-2.0/glib/gstring.h:72:5: warning: ignoring return value of 'g_string_free_and_steal' declared with attribute 'warn_unused_result' [-Wunused-result]

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix GDK_NOTE redefined warning

In file included from ../gdk/win32/gdkdrag-win32.c:201:
../gdk/win32/gdkprivate-win32.h:45: warning: "GDK_NOTE" redefined
   45 | #define GDK_NOTE(type,action)                             \
      |
../gdk/win32/gdkdrag-win32.c:40: note: this is the location of the previous definition
   40 | #define GDK_NOTE(a,b)

Fixes: bc159207bd0 ("gdk: Drop old debug macros")
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'label_get_active_uri' into 'main'

Fix hovered link URI not returned by gtk_label_get_current_uri

See merge request GNOME/gtk!5864

Merge branch 'icontheme-has-icon-consistency' into 'main'

icontheme: make has_(g)icon consistent with lookup

Closes #5709

See merge request GNOME/gtk!5865

Merge branch 'null-display' into 'main'

gdkdisplaymanager: Add missing nullable to display name

See merge request GNOME/gtk!5866

gdkdisplaymanager: Add missing nullable to display name

icontheme: make has_(g)icon consistent with lookup

Make `gtk_icon_theme_has_icon` and `gtk_icon_theme_has_gicon` also
consider unthemed icons. This makes their behavior consistent with the
actual (documented) lookup behavior.

Fixes: #5709 and makes the workaround in nautilus@b643a00b obsolete

Fix hovered link URI not returned by gtk_label_get_current_uri

This is needed for a query-tooltip handler, as mentioned in the
documentation, when there has been a hover timeout.

Maintain the previous behaviour when the link is clicked and follow the
existing documentation regarding selectable labels.

A notify::cursor handler can now also be used to retrieve the URI of the
link under the cursor.

Merge branch 'ebassi/button-can-shrink' into 'main'

Add GtkButton:can-shrink

See merge request GNOME/gtk!5552

Add GtkMenuButton:can-shrink

Map the GtkMenuButton property to the underlying GtkButton widget.

Update Turkish translation

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

surface: Clean up drawing code

See merge request GNOME/gtk!5861

surface: reformat function

Make the function follow usual coding conventions.

And while doing that, remove duplicate functionality.

surface: Refactor code

Move the early exit conditions to the top and turn them into early exits
instead of nesting if statements.

surface: Fold function into its only caller

No other changes

surface: Stop maintaining an unused linked list

wat?

surface: Remove in_update tracking

It's 2023, we use frame clocks now and don't have nested surface drawing
anymore.

surface: Stop tracking the active update area

It's unused.

Update Russian translation

Update Polish translation

Merge branch 'gtkfilesystemmodel-emit-items-changed-on-modifying-attributes' into 'main'

filesystemmodel: Emit items-changed when modifying attributes

Closes #5758

See merge request GNOME/gtk!5853

filesystemmodel: Emit items-changed when modifying attributes

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

Port filechooserwidget to GtkSignalListItemFactory

See merge request GNOME/gtk!5858

filechoosercell: Drop show-time property

Now that we are using GtkSignalListItemFactory, the convoluted show-time
property is no longer necessary.

filechooserwidget: Port to GtkSignalListItemFactory

BuilderListItemFactory isn't quite suited for our purposes, primarily
because you can't pass user data to BuilderListItemFactory.  Because
we can't get the data we are using a workaround to get the
GtkFileChooserWidget ancestory, which used to work, but with the
recent list view changes no longer doesn't.  Use GtkSignalListItemFactory
with the GtkFileChooserWidget as the user data.

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

This reverts commit 2a70093a309481cf633086a5e8f0c8bc6d2c89f0.

This fix was incomplete, the actual fix is to use signal factory.

Revert "filechoosercell: Store type_format in filechoosercell"

This reverts commit dd407dab000dc4de7d078c6270c93d303a1c18c6.

This fix was incomplete, the actual fix is to use signal factory.

Update Portuguese translation

(cherry picked from commit 5bc3284dd873ae13ef5aa883cb324b4979c9762c)

Update Hebrew translation

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

testsuite: Be less verbose in accessor-apis test

Closes #5763

See merge request GNOME/gtk!5851

listbase: Don't warn on scroll in empty list

Empty lists can still be scrolled if the scroll happens in the same
frame as the emptying of the list.

Related: #5763

adjustment: sanity-check values when setting them

It's not enough to sanitize values when starting an animation, as the
adjustment can reconfigure itself while the animation runs.
So as a simple way to handle this, we sanitize every value right before
setting it, too.

In the future we might also want to look at sanitizing start/end values
of the animation.

Fixes #5763

adjustment: Split out a function

Merge branch 'dark_theme_fix_hover_toolbar_buttons' into 'main'

theme: fix 'hover' and 'check' colors for buttons under .toolbar

See merge request GNOME/gtk!5844

Merge branch 'dark_theme_increase_selection_color' into 'main'

Increase contrast for text entry selection color on dark theme

See merge request GNOME/gtk!5843

testsuite: Allow accessors in interfaces

There are a lot of cases where properties are implemented in classes but
the getters for these exist in an interface that class implements.

A common Example is g_list_model_get_n_items() being the getter for
GtkWhateverListModel::n-items.
But also property implementations that don't use override_property()
(usually because they have a different default) are handled by this.

testsuite: Be less verbose in accessor-apis test

Only print the tried names when --verbose is passed to the test.

Update Turkish translation

Merge branch 'mask-node-transforms' into 'main'

Fix a comment

See merge request GNOME/gtk!5848

gsk: Use explicit switches

This makes it harder to forget to add new
node types here.

gsk: Mask nodes support 2d transforms

When adding mask nodes, I overlooked that
we have two separate functions for determining
what transforms a node supports without offlines.

Since we claim that mask nodes support general
transform, they must certainly support 2d transforms
as well.

Fix a comment

I tried to improve things in d67101d46e7d8647,
but I made it worse. The original note was meant
to be removed. Pointed out be Sebastian Bacher.

Merge branch 'angle-gles' into 'main'

gdk: prefer GLES when on win32/ANGLE

See merge request GNOME/gtk!5829

gdk/win32: drop some unused variables

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: drop libangle GLES minimum version

GLES 2.0 version is fine now with current gtk according to B. Otte.
Let's use the same minimum requirement for all implementations.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: use GLES when on win32/ANGLE

When using GDK_DEBUG=gl-egl, we end up using GL, but that is not well supported:

Creating EGL context version 3.0 (debug:no, forward:no, legacy:yes, es:no)
Created EGL context[0000000000000004]
OpenGL version: 0.0 (legacy)
* GLSL version: (NULL)
* Max texture size: -1059701680
* Extensions checked:
 - GL_KHR_debug: no
 - GL_EXT_unpack_subimage: yes
 - OES_vertex_half_float: no

** (gtk4-demo.exe:14324): WARNING **: 19:16:41.468: Compile failure in
vertex shader:
ERROR: 0:7: 'gl_Position' : undeclared identifier
---8<---

Use GLES when EGL implementation is ANGLE.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk: drop unused vertex_array_object

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

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

glrenderer: Don't try to use float framebuffers on GLES2

See merge request GNOME/gtk!5845

gl: Only allow RGB(A)8 on GLES2

GLES2 has no idea what 16bit textures even are, let alone floating point.

glrenderer: Don't try to use float framebuffers on GLES2

GLES doesn't know what that is.

theme: more contrast for text entry selection color on dark theme

Part of #5725

theme: fix 'hover' and 'check' colors for buttons under .toolbar

Buttons under .toolbar were using for their 'hover', 'active'
and 'check' colors the default ones from %button_basic_flat
which are very dimmed, so we explicitly darken them.

Part of #5725

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

Try harder to fix focus tests

See merge request GNOME/gtk!5842

testsuite: Skip focus tests when necessary

If our window does not get focus, we can't reliably
test things that depend on widgets being focused.

Merge branch 'ccook/doc-changes' into 'main'

Two corrections that show up in docs

See merge request GNOME/gtk!5838

Try harder to fix focus tests

The focus test now asserts that its window is active.
So we should not run them in parallel with other tests,
since those might steal the focus.

Merge branch 'wip/otte/alt.wl_surface.die.die.die' into 'main'

wayland: Don't leak all surfaces

See merge request GNOME/gtk!5840

testsuite: Fix broken test to not unref surfaces twice

The test doesn't hold 2 references, it holds only one.

The reason one unref can cause a leak is that some backends - like X11 -
only destroy the surface once the DestroyNotify event from the X server
has come in.

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 'ccook/gtklabel-doc-changes' into 'main'

GtkLabel doc corrections

See merge request GNOME/gtk!5839

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.

| method                                                                                | current                                                         | suggestion                                                           |
|---------------------------------------------------------------------------------------|-----------------------------------------------------------------|----------------------------------------------------------------------|
| [get_attributes](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3242) | "use`pango_layout_get_attribute (gtk_label_get_layout (self))`" | "use`pango_layout_get_attributes (gtk_label_get_layout (self))`" [1] |
| [set_yalign](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L5897)     | "(attributes org.gtk.Method.get_property=yalign)"               | "(attributes org.gtk.Method.set_property=yalign)"                    |
| [get_yalign](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L5923)     | "(attributes org.gtk.Method.set_property=yalign)"               | "(attributes org.gtk.Method.get_property=yalign)"                    |
| [set_ellipsize](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3892)  | "to ellipsizei"                                                 | "to ellipsize"                                                       |
| [get_attributes](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtklabel.c#L3239) | "from the labels markup"                                        | "from the label's markup"                                            |

[1] https://docs.gtk.org/Pango/method.Layout.get_attributes.html

| method                                                                                          | current                                                    | suggestion                                                |
|-------------------------------------------------------------------------------------------------|------------------------------------------------------------|-----------------------------------------------------------|
| [Entry/set_invisible_char](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2101) | "(attributes org.gtk.Method.sets_property=invisible-char)" | "(attributes org.gtk.Method.set_property=invisible-char)" |
| [FlowBox::activate](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkflowbox.c#L541)       | "This can be happen either by"                             | "This can happen either by"                               |

Revert "Two corrections and added method linking that show up in docs"

This reverts commit c7f30ecdfdb70fd8d4fcc02a4a576dfee0991e28.

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

Two corrections and added method linking that show up in docs

| method                                                                                          | current                                                    | suggestion                                                |
|-------------------------------------------------------------------------------------------------|------------------------------------------------------------|-----------------------------------------------------------|
| [Entry/set_invisible_char](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2101) | "(attributes org.gtk.Method.sets_property=invisible-char)" | "(attributes org.gtk.Method.set_property=invisible-char)" |
| [FlowBox::activate](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkflowbox.c#L541)       | "This can be happen either by"                             | "This can happen either by"                               |

| method                                                                                                    | current                                                | suggestion                                                   |
|-----------------------------------------------------------------------------------------------------------|--------------------------------------------------------|--------------------------------------------------------------|
| [Box/get_baseline_position](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkbox.c#L461)             | "value set by gtk_box_set_baseline_position()."        | "value set by [method@Gtk.Box.set_baseline_position]."       |
| [Box/get_spacing](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkbox.c#L412)                       | "value set by gtk_box_set_spacing()."                  | "value set by [method@Gtk.Box.set_spacing]."                 |
| [CenterBox/get_baseline_position](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkcenterbox.c#L528) | "value set by gtk_center_box_set_baseline_position()." | "value set by [method@Gtk.CenterBox.set_baseline_position]." |
| [Entry/get_activates_default](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2296)        | "value set by gtk_entry_set_activates_default()."      | "value set by [method@Gtk.Entry.set_activates_default]."     |
| [Entry/get_alignment](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2380)                | "value set by gtk_entry_set_alignment()."              | "value set by [method@Gtk.Entry.set_alignment]."             |
| [Entry/get_extra_menu](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L3727)               | "set with gtk_entry_set_extra_menu()."                 | "set with [method@Gtk.Entry.set_extra_menu]."                |
| [Entry/get_has_frame](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L2340)                | "value set by gtk_entry_set_has_frame()."              | "value set by [method@Gtk.Entry.set_has_frame]."             |
| [Entry/get_progress_pulse_step](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkentry.c#L3372)      | "set with gtk_entry_set_progress_pulse_step()."        | "set with [method@Gtk.Entry.set_progress_pulse_step]."       |
| [Fixed/get_child_transform](https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkfixed.c#L314)           | "set using gtk_fixed_set_child_transform()."           | "set using [method@Gtk.Fixed.set_child_transform]."          |

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

Add GtkButton:can-shrink

For certain kinds of layouts, especially ones where one or both sizes of
a top level is constrained by physical limits, it's acceptable to have
buttons that rely on the minimum size of their contents, rather than the
natural size. It is left to the application authors, or the localization
teams, to ensure that things like translations and font sizes do not
result in a broken UI.

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.