vulkan: Rebuild the precompiled shaders

We forgot that with all the changes.

vulkan: Take offscreen fromat from context

We want to create offscreens in compatible formats, and in particular we
want to ideally use the same format as rendering would use.

vulkan: Use 3 descriptor sets, not 3 bindings

It turns out variable length is only supported for the last binding in
a set, not for every binding.
So we need to create one set for each of our arrays.

[ VUID-VkDescriptorSetLayoutBindingFlagsCreateInfo-pBindingFlags-03004 ] Object 0: handle = 0x33a9f10, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0xd3f353a | vkCreateDescriptorSetLayout(): pBindings[0] has VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT but 0 is the largest value of all the bindings. The Vulkan spec states: If an element of pBindingFlags includes VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT, then all other elements of VkDescriptorSetLayoutCreateInfo::pBindings must have a smaller value of binding (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-VkDescriptorSetLayoutBindingFlagsCreateInfo-pBindingFlags-03004)

vulkan: Use the right flags

Somebody (me) had flipped the 2 flags in commit ba28971a183:

[ VUID-vkCmdCopyBufferToImage-srcBuffer-00174 ] Object 0: handle = 0x3cfaac0, type = VK_OBJECT_TYPE_COMMAND_BUFFER; Object 1: handle = 0x430000000043, type = VK_OBJECT_TYPE_BUFFER; | MessageID = 0xe1b276a1 | Invalid usage flag for VkBuffer 0x430000000043[] used by vkCmdCopyBufferToImage. In this case, VkBuffer should have VK_BUFFER_USAGE_TRANSFER_SRC_BIT set during creation. The Vulkan spec states: srcBuffer must have been created with VK_BUFFER_USAGE_TRANSFER_SRC_BIT usage flag (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-vkCmdCopyBufferToImage-srcBuffer-00174)

vulkan: Set descriptorBindingStorageBufferUpdateAfterBind

It's necessary now that we use storage buffers for gradients:

[ VUID-VkDescriptorSetLayoutBindingFlagsCreateInfo-descriptorBindingStorageBufferUpdateAfterBind-03008 ] Object 0: handle = 0x1e72d70, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0x943cc552 | vkCreateDescriptorSetLayout(): pBindings[0] can't have VK_DESCRIPTOR_BINDING_UPDATE_AFTER_BIND_BIT for VK_DESCRIPTOR_TYPE_STORAGE_BUFFER since descriptorBindingStorageBufferUpdateAfterBind is not enabled. The Vulkan spec states: If VkPhysicalDeviceDescriptorIndexingFeatures::descriptorBindingStorageBufferUpdateAfterBind is not enabled, all bindings with descriptor type VK_DESCRIPTOR_TYPE_STORAGE_BUFFER must not use VK_DESCRIPTOR_BINDING_UPDATE_AFTER_BIND_BIT (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-VkDescriptorSetLayoutBindingFlagsCreateInfo-descriptorBindingStorageBufferUpdateAfterBind-03008)

vulkan: Remove unused declaration from shader

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

vulkan: Add support for different image formats

See merge request GNOME/gtk!6117

vulkan: Check for descriptor indexing extension

By checking if the extension is supported and avoiding devices when it
isn't, we avoid critical warnings later.

vulkan: Refactor function

Instead of checking for one specific extension, instead pass the
extension to check for by name.

This way we can reuse it for different extensions.

vulkan: Add support for high bit depth

vulkan: Don't create unnecessary render passes

Pass the render pass to the pipeline creation function instead of
creating an extra one just for pipeline creation.

vulkan: Put the framebuffer in the renderpass

... instead of having fancy caching.

That caching is complicated and it's not necessary.

vulkan: Store the VkFormat in GskVulkanImage

... and use that info when creating renderpasses.

vulkan: Pick high depth texture for high depth nodes

If a node has a higher depth, pick the RGBA format that has that depth
as the texture format we're renderig to with render_texture().

Support for adapting the swapchain is not part of this.

vulkan: Add format fallback

When a GdkMemoryFormat isn't supported, pick close formats that have a
higher chance of being supported.
Make sure this works recursively and the whole loop always ends up at
R8G8B8A8_UNORM because that one is mandatory.

Roughly, follow these rules:
1. Drop the unpremultiplied
2. Expand channels to include all of RGBA
3. pick swizzle that is RGBA
4. pick next largest depth
5. pick R8G8B8A8_UNORM

vulkan: Allow mapping images as "read" and/or "write"

This way, we unify the code paths for memory access to textures.

We also technically gain the ability to modify images, though I have no
use case for this.

vulkan: Pass format to offscreen creation function

That way, the offscreen can create images of different types.

Its not used in this commit, but will come in handy when we want to
support high bit depth.

vulkan: Remove gsk_vulkan_image_new_for_framebuffer()

Use gsk_vulkan_image_new_for_offscreen() instead, it does the same thing
pretty much.

vulkan: Allow uploading in different formats

This requires quite some code because Vulkan may not support all the
formats and then we need to detect that and fallback properly.

vulkan: Create the view in vulkan_image_new()

All callers want it created anyway.

Plus, we can consolidate things in future commits.

testsuite: Unify skipping memorytexture test

... and make it handle more cases of failure, in particular OpenGL and
Vulkan being unsupported by the system.

Merge branch 'more-a11y-tweaks' into 'main'

A11y improvements

See merge request GNOME/gtk!6116

docs: Expand role docs

Clarify that presentation and none
can be used interchangeably.

dropdown: Use presentation instead of none

testsuite: Add a vulkan method to memorytexure test

This uses the newly added NULL-surface renderer.

vulkan: Make gsk_renderer_realize() work with NULL surface

Pretty much copy what GL does and just use the default display to create
GPU-related resources without the need for a display.

This also adds gdk_display_create_vulkan_context() but I've
kept it private because the Vulkan API is generally considered in flux,
in particular with our pending attempts to redo how renderers work.

vulkan: Actually reset the buffer size

Fixes a bug introduced in d1135f9e3c7.

Luckily the buffer was large enough that all my testing didn't catch it
because it took a few minutes to overflow.

vulkan: Renaming fix

This rename was a long time ago...

modelbutton: Set a11y shortcuts

printdialog: Some a11y improvements

Add proper roles and labels in some places.

printdialog: Give the page range entry a label

This is mainly to pacify the a11y checker.

notebook: Make sure tabs are labelled

windowcontrols: Cosmetics

dropdown: Avoid accessibility warnings

The image here is just presentational.

infobar: Improve a11y labelling

Treat the close button the same way we treat
the window close button, wrt. to accessibility.

gtk-demo: Miscellaneous a11y improvements

Merge branch 'a11y-remove-value-opt' into 'main'

a11y: Remove an overzealous optimisation

Closes #5886

See merge request GNOME/gtk!6115

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.