a11y: Remove an overzealous optimisation

The result of calling update_property needs
to be that the property is marked as set
afterward, even if the value we pass happens
to match the default value.

After this change, scrollbars have value-now
show up as zero in the accessiblity page of
the inspector, even when that matches the lower
bound.

Test included.

Fixes: #5886

Merge branch 'a11y-nest-button-redux' into 'main'

Revert "dropdown: Shuffle accessible roles around"

See merge request GNOME/gtk!6111

Merge branch 'main' into 'main'

Revert "Use gsk_matrix_transform_point3d consistently"

Closes #5902

See merge request GNOME/gtk!6112

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

memoryformat: Add gdk_memory_format_get_depth()

See merge request GNOME/gtk!6110

gdk: Replace prefers_high_depth with depth

Now that we track depth, we can also pass it into the GDK frame code.

For now it's just passed along, code acts the same as with
prefers_high_depth.

rendernode: Change to gsk_render_node_get_preferred_depth()

Instead of just tracking preferred_high_depth(), track the actual depth
we'd like to have.

memoryformat: Add gdk_memory_format_get_depth()

Replace gdk_memory_format_prefers_high_depth with the more generic
gdk_memory_format_get_depth() that returns the depth of the individual
channels.

Also make the GL renderer use that to pick the generic F16 format
instead of immediately going for F32 when uploading textures.

inspector: Handle a11y being disabled

Revert "Use gsk_matrix_transform_point3d consistently"

Revert commit 440d56a4

a11y: Special-case nested buttons

Special-case nested buttons in our name computation,
since it is hard to reconcile all the a11y attributes
being on the wrapper, but the focus ending up on the
button inside.

This is a pragmatic approach that works. The only
downside is that the wrapper and the button end up
with the same name+description, but at least orca
seems to only read the focus elements' ones.

Revert "colordialogbutton: Shuffle accessible roles around"

This reverts commit 343b9d246ff09e24842020e907a681984ea6d610.

Unfortunately, this makes it so that the focus ends up on
the 'generic' accessible, not the one with the label, and
orca remains quiet.

Revert "dropdown: Shuffle accessible roles around"

This reverts commit 5ec0b07bafaa0d874fb2e7e0674e46f9d8f8378d.

Unfortunately, this makes it so that the focus ends up on
the 'generic' accessible, not the one with the label, and
orca remains quiet.

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

testsuite: Add some more a11y tests

See merge request GNOME/gtk!6109

dropdown: Shuffle accessible roles around

Make the internal toggle button generic, so that
the a11y checker doesn't complain about it not
having a label. And mark the icons in the popup
as presentational.

colordialogbutton: Shuffle accessible roles around

Make the color button itself take the button role,
and make the internal toggle button just be generic.

This solves the problem that labelled-by relations
that are set up in ui files via mnemonics point at
the toplevel, not the toggle button.

gtk-demo: a11y improvements

Make the clipboard demo come up clean in the
a11y checker.

testsuite: Add some more a11y tests

Test that overriding roles works, both
via g_object_new, and via ui files.

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

gtk-demo: Fix a crash

Closes #4522

See merge request GNOME/gtk!6108

Apply suggestions from review

video: Make the overlay clickable

And add missing accessible labels at the same time.

Fixes: #4522

inspector: Cosmetic fixes

Align the values on the a11y page to the right.

mediacontrols: Add accessible labels

gtk-demo: Misc a11y improvements

gtk-demo: Fix a crash

This was broken in 6b2c088a29e4a819.

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

colordialogbutton: Sync color

See merge request GNOME/gtk!6107

inspector: Limit the width of a11y page

Ellipsize labels that can contain long
content to prevent a super-wide window.

gtk-demo: Misc a11y fixes

Add some missing labels.

inspector: Realize AT ontexts

Otherwise we don't get change notification...

a11y: Set has-popup consistently

Set the has-popup property when a widget
has a context menu.

colordialogbutton: Sync color

Make sure the color of the swatch and the button
are initially in sync. As a side-effect, this
ensures that the swatch has its accessible label
computed at the outset.

gtk-demo: Cosmetics

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

Don't use "Tab list" as an accessible label

Closes #4839

See merge request GNOME/gtk!6106

Don't use "Tab list" as an accessible label

It does not add any extra information.

Fixes: #4839

Merge branch 'dropdown-sections' into 'main'

dropdown: Add section support

See merge request GNOME/gtk!6102

gtk-demo: Add an example for dropdown sections

Make one of the dropdowns in the Selections
demo have sections.

dropdown: Add section support

Add a header-factory property and pass it through
to the listview in the popup.

Merge branch 'fix-c4013' into 'main'

gtkprintoperation-win32.c: Fix build

See merge request GNOME/gtk!6099

Merge branch 'a11y-computed-name' into 'main'

atcontext: Update name computation

See merge request GNOME/gtk!6088

reference/gtk/coordinates: Fix typos of “widget’s”

Merge branch 'better_gtkstacksidebar' into 'main'

GtkStackSidebar: Explicitly mark the sidebar items as labeled by the labels

See merge request GNOME/gtk!6100

GtkStackSidebar: Explicitly mark the sidebar items as labeled by the labels

That improves the reading of the GtkStackSidebar widget considerably.
Previously, Orca would not read the items at all, now it does.

gtkprintoperation-win32.c: Fix build

Include the needed headers so that we don't break the build with C4013
warnings, which are treated as errors if msvc_recommended_pragmas.h is
found during build configuration.

Merge branch 'listitem' into 'main'

signallistitemfactory: finish prototype change

See merge request GNOME/gtk!6098

Merge branch 'no_random_screen_coords' into 'main'

gtkatspicomponent: Don't return unitialized stack data as screen positions of widgets

See merge request GNOME/gtk!6097

signallistitemfactory: finish prototype change

Complete the API change from commit be1729b316d ("signallistitemfactory:
Update signal prototype").

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

gtkatspicomponent: Don't return unitialized stack data as screen positions of widgets

We can not compute the correct value, but that does not mean we should return
basically random values from an unitialized stack space.

Rather than that behavior, return zeros concistently.

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

vulkan: Move some code

See merge request GNOME/gtk!6096

vulkan: Repurpose debug flags for image uploads

Now that we don't use the old environment variables anymore to force
staging buffer/image uploads, we don't need them.

However, we do autodetect the fast path for avoiding a staging buffer
now, and we might want to be able to turn that off for testing.

So add GSK_DEBUG=staging that does exactly that.

vulkan: Remove gsk_vulkan_image_new_from_data()

This is unused now that all the code uses map/unmap.

The only thing that map/unmap doesn't do that the old code did, was use
a staging image instead as alternative to a staging buffer for image
uploads.

However, that code is not necessary for anything, so I'm sure we can do
without.

vulkan: Add upload fastpath

If the memory heap that the GPU uses allows CPU access
(which is the case on basically every integrated GPU, including phones),
we can avoid a staging buffer and write directly into the image memory.

Check for this case and do that automatically.

Unfortunately we need to change the image format we use from
VK_IMAGE_TILING_OPTIMAL to VK_IMAGE_TILING_LINEAR, I haven't found a way
around that yet.

vulkan: Add gsk_vulkan_memory_can_map()

.. nd use it to assert memory is mappable when mapping it.

vulkan: Use map/unmap for fallback images

vulkan: Render fallback into vulkan memory

Use the new map/unmap image upload method for Cairo node drawing:
1. map() the memory
2. create an image surface or that memory
3. draw to that image surface
4. success

There's no longer a need for Cairo to allocate image memory.

vulkan: Use new upload method for texture uploads

gsk_vulkan_image_new_from_texture() now uses the direct copy via
gdk_texture_downloader_download_into().

vulkan: Add a new way to upload data into images

As an alternative to gsk_vulkan_image_new_from_data() that
takes a given data and creates an image from it, add a 3 step process:
  gsk_vulkan_image_new_for_upload()
  gsk_vulkan_image_map_memory()
  /* put data into memory */
  gsk_vulkan_image_unmap_memory()

The benefit of this approach is that it potentially avoids a copy;
instead of creating a buffer to pass and writing the data into it before
then memcpy()ing it into the image, the data can be written straight
into image memory.

So far, only the staging buffer upload is implemented.

There are also no users, those come in the next commit(s).

vulkan: Move some code

Add gsk_vulkan_image_new_from_texture() and use it.

Also rewrite the actual code from using Cairo surfaces to using
GdkTextureDownloader.

vulkan: Constify upload function

Merge branch 'wip/carlosg/pad-detection' into 'main'

gdk/wayland: Create pad devices on enter

See merge request GNOME/gtk!6094

gdk: Include pads in GDK_SEAT_CAPABILITY_ALL

The GDK_SEAT_CAPABILITY_TABLET_PAD stood awkwardly out of the
ALL value. Even though it's not a keyboard, its focus has more
resemblance to it, so it should be part of this group together
with keyboards.

gdk/wayland: Create pad devices on enter

We were creating the pad device on wp_tablet_pad.done, but
at that time we do not know what tablet it is associated with,
thus we cannot get appropriate vid/pid/name properties for it.

To get that, we need to wait for the pad to enter a surface,
at that time we do know what tablet it is associated with, so
we can get better information about the device.

There are pads that may plausibly "change" tablet between
one .enter event and the next (e.g. Wacom Express Key Remote),
but this situation is highly unlikely. The pad devices created
are thus persistent until that situation happens.

Merge branch 'mcatanzaro/gtk-show-uri-deprecated-for' into 'main'

Change recommended replacement for gtk_show_uri()

See merge request GNOME/gtk!6093

a11y docs: Some updates

Clarify what UI properties go into the name computation,
and mention the inspector tools for accessiblity.

Change recommended replacement for gtk_show_uri()

Problem is GtkFileLauncher is unable to handle all the types of URIs
that are supported by gtk_show_uri(), e.g. help: URIs. GtkUriLauncher
avoids this problem.

Another problem is that GtkUriLauncher is just generally a better choice
for launching URIs, since you don't have to create a GFile in order to
use it. Porting code is slightly simpler.

The documentation still mentions both GtkFileLauncher and GtkUriLauncher
as options, but most people will use whatever the compiler recommends
when it prints the deprecation warning.

Merge branch 'window-clear-resize_cursor-on' into 'main'

window: Clear resize_cursor on leave notify events

See merge request GNOME/gtk!6090

window: Clear resize_cursor on leave notify events

When the pointer leaves the window surface, gtk_window_capture_motion
will not be called anymore, so priv->resize_cursor may remain non-NULL
indefinitely without this.

If update_cursor is later called (via gtk_window_maybe_update_cursor) on
a virtual enter notify event (e.g. because the pointer entered a
descendant surface), it would previously re-set the window surface
cursor to priv->resize_cursor, which could result in the wrong cursor
shape being shown for descendant surfaces.

This affected mutter-x11-frames, see
https://gitlab.freedesktop.org/xorg/xserver/-/issues/1557.

One could also say that if the pointer leaves the window surface, it's
trivially not over any window edge.

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

Matthiasc/for main

See merge request GNOME/gtk!6091

ci: Collect artifacts from the asan build

listitemmanager: Quiet a compiler warning

Merge branch 'a11y-overlay' into 'main'

inspector: Add an a11y overlay

Closes #5883

See merge request GNOME/gtk!6085

Fix a crash in gtk-demo

Fixes #5889

docs/reference/gtk/running: Incl GDK_DEBUG=portals

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/5871

Merge branch 'focus-modal-windows' into 'main'

win32: Focus modal windows when clicking on unfocused parent

See merge request GNOME/gtk!6051

Merge branch 'center-dialogs' into 'main'

win32: Center newly created transient windows

Closes #5407

See merge request GNOME/gtk!6050

Update Czech translation

inspector: Add an a11y overlay

Add an overlay that shows a11y issues.

For now, this checks for:
 - abstract roles being used
 - elements without labels
 - required attributes
 - required context

menubutton: Horizontally expand child

Set hexpand on the outer box so we don't propagate that expand.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/5883

menubutton: Normalize label layout

- Remove unnecessary halign=center and hexpand=0 dance
- Set hexpand=false on the outer box so we don't propagate hexpand

Merge branch 'misc-a11y-fixes' into 'main'

Various a11y improvements

See merge request GNOME/gtk!6089

gtk4-demo: Some a11y improvements

Add some labels in the pickers demo.

filechooser: Some accessibility improvements

togglebutton: Set a11y properties on realize

The pressed property is required, so we need to
make sure that it is set.

fontchooser: Add one more accessible property

aboutdialog: Set some accessible properties

menubutton: Set accessible relations

In the case where we create the popover, mark it
as labelled by the button.

shortcutswindow: Add some accessible properties

passwordentry: Change an accessible role

The caps-lock icon is really an alert. Maybe
this is in vain, since orca ignores these images.

atcontext: Change the way we handle fallback

The tooltip text should only be considered after
all other means are exhausted. but it can be used
for both the name and the description.

See https://www.w3.org/TR/accname-1.2/

atcontext: Update name computation

Implement this sentence from the "Accessible Name
and Description Computation 1.2" spec:

    If the root node's role prohibits naming,
    return the empty string ("").

See https://www.w3.org/TR/accname-1.2/

Merge branch 'wip/alice/menu-button' into 'main'

menubutton: Horizontally expand child

Closes #5883

See merge request GNOME/gtk!6084

Merge branch 'master' into 'main'

Reset chars_changed_stamp in _gtk_text_btree_unref

Closes #5544

See merge request GNOME/gtk!6087

Popover: If can't get widget rect, zero output ptr

cherry-pick of commit a6d40b610be6acb635d8732c11bb6d866bf2e95d

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/893#note_1766079

Reset chars_changed_stamp in _gtk_text_btree_unref

This causes an "Invalid text buffer iterator" warning to be produced if a TextIter is used after the TextBuffer is disposed.

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

mediafile: Load extension at startup with GTK_MEDIA

See merge request GNOME/gtk!6086

vulkan: Handle new nodes being added correctly

When nodes are added, nothing was warning us that we need to bump
N_RENDER_NODES.

Make sure that that's no longer necessary by refactoring the code to
remove the define.

vulkan: Do intersection check for every node

This is more expensive, but it finds more cases, and in particular it
catches corner cases like empty nodes or fully clipped nodes that might
otherwise make the kernel throw signals in our direction.

vulkan: Handle empty rects in intersects_rect()

Apart from the none case, this was already handled, so we just check if
the rect is empty now.

mediafile: Load extension at startup with GTK_MEDIA

When the GTK_MEDIA env var is set, check at startup that it works, not
only when the first MediaFile is instantiated.

This has the fortunate side effect that it prints help output for
GTK_MEDIA=help at startup, too.

Merge branch 'fontchooser-a11y' into 'main'

button: Improve accessible setup

See merge request GNOME/gtk!6081

menubutton: Horizontally expand child

Set hexpand on the outer box so we don't propagate that expand.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/5883

menubutton: Normalize label layout

- Remove unnecessary halign=center and hexpand=0 dance
- Set hexpand=false on the outer box so we don't propagate hexpand