vulkan: Factor out call to opacity op

I want to reuse it for crossfades (see next commit).

vulkan: Convert color op to new method

This is a straightforward and simple port.

vulkan: Create pipeline differently for ops

Instead of creating a pipeline GObject, just ask for the VkPipeline.

And instead of having the Op handle it, just let the renderpass look
up/create the relevant pipeline while creating commands so that it can
insert vkCmdBindPipeline calls as-needed.

Revert "vulkan: Always render clip extents"

This reverts commit 0f184d3270cfac522a05ae06d28cc2f1e32c0e6d.

The renderer is good enough to make use of the clip region.

Or rather: If it isn't, the renderpass should take care of that, not the
render object.

vulkan: Combine textures and samplers again

This reverts most of commit f420c143e0d8947a433704969bab02aea47d82f5
again because it turns out GPUs like combined images and samplers.

But: The one thing we don't revert is allowing the C code to select any
combination of sampler and image:
gsk_vulkan_render_get_image_descriptor() now takes a 2nd argument
specifying the sampler.

This allows the same flexibility as before, we just combine things
early.

This change was inspired by
https://developer.nvidia.com/blog/vulkan-dos-donts/

vulkan: Store shaders in the display

Have a resource path => vkShaderModule hash table instead of doing fancy
custom objects.

A benefit is that shader modules are now shared between all renderers
and pipelines.

vulkan: Initialize ops differently

Instead of creating the op manually, just pass in the renderpass and
have the op created from there.

This way ops aren't really initialized anymore, they are more appended
to the queue, so instead of foo_op_init() we can just call the function
foo_op().

vulkan: Use VkPipeline instead of GskVulkanPipeline

This is in preparation for getting rid of GskVulkanPipelines.

vulkan: Use new ops for repeat nodes

The new code always uses an offscreen, even for children that are
exactly fitting texture nodes.
I would have had to write more code and didn't consider it worth it,
especially because it would have required complicating the
get_as_image() function.

This was the last node using the texture pipeline.

vulkan: Implement fallback using new nodes

With Cairo upload and texture nodes being available, we can do that now.

vulkan: Pass bounds to texture op

This way the node can do the normalization (and maybe stop doing it
in the future).

vulkan: Add a Cairo upload node

... and use it for cairo nodes.

vulkan: Port opacity to new ops

vulkan: Add offscreen and color-matrix op

.. and use them for color-matrix operations.

vulkan: Split out a function

We'll need it elsewhere soon.

vulkan: Add an argument to vfunc

We need this in the future.

vulkan: Remove nonexisting function from header

vulkan: Split out a function

Making that function externally usable allows having render passes
managed externally.

Also remove a nonexisting function from the header.

vulkan: Use new ops for TextureScale nodes

vulkan: Add new renderops for texture rendering

Adds 2 ops:

- Upload
  Creates a new Vulkan image and uploads data into it

- Texture
  Draws a given image

These 2 ops are then used for GskTextureNodes.

vulkan: Split texture caching code

Instead of having one function that gets the image for the texture and
uploads it if it doesn't exist yet, make it 2 functions:

One to get the texture if it exists.
One to assign an uploaded image to the texture.

This way, we can potentially do the upload ourselves.

vulkan: Actually run the op_finish()

It's a no-op for all current ops, so it isn't really necessary. But
that's about to change.

vulkan: Pass context, not uploader

We don't need the uploader when creating the image, only when uploading.

vulkan: Create the first real VulkanOp

Split out the scissor op into its own implementation as a proof of
concept of how ops are meant to look when they are actually working.

vulkan: Allocate render ops differently

Allocate the memory up front instead of passing the Op into it.

This way, we can split ops into their own source file and use
init/finish style to use them.

vulkan: Invent a new abstraction

GskVulkanOp is meant to be a proper abstraction of operations
the Vulkan renderer will be doing.

For now it's an atrocious clunky piece of junk wedged into the
renderpass codebase.

It's so temporary that I didn't even adjust indentation of the code.

vulkan: Use the actual RenderOp type

... instead of the generic one.

This is again preparation for future changes.

vulkan: Use a byte array for render ops

This allows allocating only as much memory as is needed for each op.

We don't do that yet, this is still preparation.

vulkan: Split out a function

This is preparation for future changes.

Merge branch 'dboles/EventControllerKeyModifiersDocs' into 'main'

EventControllerKey: doc fixes re Modifiers

See merge request GNOME/gtk!6186

EventControllerKey: Remove redundant/unliked "See"

Saying "See [Thing]" is not very helpful without it being a link to
Thing, and we already get that for free as the "Type:" of the param.

EventControllerKey: Remove wrong signal arg, which

caused the resulting parameter to be called keyval, but it must be state

Update Georgian translation

Merge branch 'nielsdg/fix-sectionmodel-signal-doc' into 'main'

sectionmodel: Fix GIR comment for sections-changed

See merge request GNOME/gtk!6185

sectionmodel: Fix GIR comment for sections-changed

Make sure to end the signal name with a colon so GIR recognizes the
signal. This should also fix the problem that the documentation for that
signal is currently missing in the rendered gi-docgen output.

Merge branch 'vulkan-resize-fix' into 'main'

vulkan: Wait for device to be idle before create/recreating swapchain

See merge request GNOME/gtk!6183

vulkan: Wait for device to be idle before create/recreating swapchain

Wait for device to be idle because this function is also called in
window resizes.
And if we destroy old swapchain it also destroy the old VkImages,
those images could be in use by a vulkan render.

This fixes a issue reported in Mesa repository when running
GTK with Xe KMD.

Fixes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9044
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>

Merge branch 'fix-validate-crash' into 'main'

buildertool: Exit orderly

Closes #5948

See merge request GNOME/gtk!6182

buildertool: Exit orderly

The validate command does need a display connection,
for better or worse. So exit in an orderly fashion
if we don't have one, instead of crashing.

Fixes: #5948

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

textbtree: Remove unnecessary check

See merge request GNOME/gtk!6179

vulkan: Change the clip intersection check

Intersection with a roudned clip takes too long.

Instead, rename the function to may_intersect() to be clear about what
it does and then just intersect with the regular rectangle.

vulkan: intersect rects also for CLIP_NONE

If we don't clip anything, we stil have bounds - either the framebuffer
size or (more likely) the scissor rect. And we don't want to draw
anything that is outside these bounds.

So clip in those cases, too.

Stops gtk4-demo --run=listbox from trying to render the whole listbox
instead of only the visible parts.

rendernode: Implement proper GSK_IS_RENDERNODE()

Use G_TYPE_CHECK_INSTANCE_TYPE() instead of just checking for != NULL.
After all, this is a GTypeInstance.

Also fixes some gcc complaints when checking
  node == NULL || GSK_IS_RENDERNODE (node)
which gcc was convinced would be always true.

textbtree: Remove unnecessary check

Since a93614409ed79e1297469bdc1de3cb69663de71e we don't allocate the
stack anymore, so this NULL check is unnecessary now - and it's flagged
by compilers.

Merge branch 'dboles/image-notify-storage-type' into 'main'

Image—Notify when :storage-type changes from EMPTY

See merge request GNOME/gtk!6170

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

gtklistitemmanager: Stop leaking item

Closes #5940

See merge request GNOME/gtk!6171

gtklistitemmanager: Stop leaking item

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

Image—Notify when :storage-type changes from EMPTY

PROP_STORAGE_TYPE was only notified if it was changing *to* EMPTY, in
gtk_image_clear_internal(). We did not notify when it changes *from*
EMPTY to something non-empty. We should as not doing so is confusing,
e.g. if a user wants to bind :storage-type to :visible if non-empty,
which I just did! So, in functions that apply an ImageType, now notify.

Also do so in gtk_image_set_from_definition, declared in imageprivate.h,
even though none of the function there are currently used anywhere.
(Should they be removed?)

Update Hebrew translation

Merge branch 'wip/carlosg/switch-stylus-buttons' into 'main'

gdk/wayland: Switch behavior of BTN_STYLUS/STYLUS2 as middle/right click

Closes #5935

See merge request GNOME/gtk!6168

gdk/wayland: Switch behavior of BTN_STYLUS/STYLUS2 as middle/right click

This mapping of stylus evdev input event codes into GDK button numbers
makes gdk/wayland inconsistent with gdk/x11, so depending on the backend
the same button middle-click pastes or right-click pops up menus.

Make the wayland backend consistent with X11, so that a GNOME wayland
session gets these buttons consistently mapped across all kinds of
clients.

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

GdkTexture: Don't mention private new_from_surface

The stuff about Cairo Surfaces is in gdktextureprivate.h, & so end users
will not be able to use them; we shouldn't confuse by implying they can.

docs/list-widget: links, DirList, correct Sections

* Add links to various symbols.
* Mention DirectoryList in the "ready-made choices available" section.
* Don't say that GridView can display headers: it makes no attempt to.

Merge branch 'ebassi/issue-5934' into 'main'

Lower the Python requirement

Closes #5934

See merge request GNOME/gtk!6167

Do not use bleeding edge Python

The match operator was added in Python 3.10, which is a bit too new for
some downstreams.

While at it, let's fix the flake8 errors and warnings.

Fixes: #5934

Add flake8 configuration file

Ignore long lines; most of our Python scripts generate code or other
types of files, which makes long lines a necessity.

We should validate all our Python script in our CI as well.

AlertDialog: Remove spurious/unmatched backtick

ListBase: Fix a typo

Merge branch 'update-uac-script-format' into 'main'

tools/generate-uac-manifest.py: Fix formatting (unify with copy in GLib)

See merge request GNOME/gtk!6164

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

gsk: Catch values < 0 before bad things happen

See merge request GNOME/gtk!6165

testsuite: Add a test for mask out of bounds effects

Inverted alpha masks have an effect on the source, even if the mask
doesn't cover the source at all - or worse, is completely clipped out.

The GL renderer handles this fine, but Cairo and Vulkan had
optimizations that got this wrong.

gsk: Fix luminance in Cairo and GL renderer

In particular, fix the combination of luminance and alpha. We want to do
  mask = luminance * alpha
and for inverted
  mask = (1.0 - luminance) * alpha
so add a test that makes sure we do that and then fix the code and
existing tests to conform to it.

rendernode: Work around a Cairo bug

When color-matrix modifying a clear surface, the surface would remain
clear according to Cairo.

That's very unfortunate when we prepare a mask for inverted-alpha
masking.

build: Include the right things

If we build our own targets, we need to include those.

This is only relevant when adding new shaders because meson will
complain that the (unused) sources don't exist as it tries to include
those.
And that will make the build.ninja file not be generated which would
have build those shaders and would have allowed to copy them into the
sources.

Note that this makes builds with glslc not care about all the shader
files being included with the sources, but we have CI to check that.

rendernode: Mask nodes with different modes are different

So treat them as such.

Fixes the node editor not updating when I edit the mask mode.

gsk: Catch values < 0 before bad things happen

In particular, catch radius values being < 0 by return_if_fail()ing in
the rendernode creation code, and by erroring out in the rendernode
parser.

I try too much dumb stuff in the node editor.

glcontext: Fix typo in Apple extension name

Post-release version bump

4.11.4

tools/generate-uac-manifest.py: Fix formatting

As this script is now also used in GLib, unify the formatting between
GLib and GTK. Make the formatting of the script conformant to the
Black[1] tool, as GLib requires, and add a copyright header to this
script.

[1]: https://black.readthedocs.io/en/stable/, see also
$(glibsrcroot)/.gitlab-ci/run-bash.sh

Merge branch 'fix_atcontext_refleaks' into 'main'

a11y: Fix some GtkATContext reference leaks

See merge request GNOME/gtk!6160

a11y: Fix some GtkATContext reference leaks

`gtk_accessible_get_at_context()` is transfer-full, so the returned
reference needs to be dropped. This was missing in a couple places.

Merge branch 'gdk-win32-input-fixes' into 'main'

GdkWin32 input fixes

Closes #5877

See merge request GNOME/gtk!6131

Merge branch 'wip/chergert/map-as-sectionmodel' into 'main'

maplistmodel: implement GtkSectionModel

See merge request GNOME/gtk!6154

Add section model tests for GtkMapListModel

maplistmodel: Handle the ::sections-changed signal

Wrapper section models need to listen to and pass
on the ::sections-changed signal from the underlying
model.

Image: Fix missing closing backtick

Merge branch 'dboles/gtk-demo_dnd_dark' into 'main'

gtk-demo/dnd: Fix, generalise detecting dark theme

See merge request GNOME/gtk!6157

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

build: Try harder to work with nongnu ld

See merge request GNOME/gtk!6159

build: Try harder to work with nongnu ld

Only try to be fast with gnu ld.

gtk-demo/dnd: Fix, generalise detecting dark theme

Our default theme is now Default, not Adwaita, & HighContrastInverse was
renamed to Default-hc. So these checks did not work anymore. Rather than
hard-coding the new names, & possibly running into the same issue again,
we can just look for the convention of appending -dark to the theme name
and/or the Settings:prefer-dark-theme prop. The latter, we can & likely
SHOULD also apply to all themes - not just ours as before. We also check
for the :dark suffix as that means the theme variant - & before checking
GtkSettings check the GTK_THEME env var, just as GtkSettings itself does

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

build: Move objcopy checks to one place

Closes #5672

See merge request GNOME/gtk!6156

build: Look for ld.bfd

The objcopy+ld approach to fast resource building
relies on behavior that is specific to the binutils
linker, and does not work with the llvm one.

Therefore, check for ld.bfd. We still fall back
to trying with just ld, since I'm not 100% sure
if binutils unconditionally installs ld.bfd.

Fixes: #5672

build: Move objcopy checks to one place

We were doing the same thing in three places.
Move it to the toplevel meson.build, so we
can change it in one place.

Updates

Merge branch 'fix_dropdown_set_expression' into 'main'

GtkDropDown: Force redisplay of the drop-down items after expression change

See merge request GNOME/gtk!6145

dropdown: Handle expression changes

The expression is used by the default factory, so we
need to track whether we are using the default factory
and recreate it if the expression changes.

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

Plug a memory leak in gsk_render_node_serialize

See merge request GNOME/gtk!6155

ci: Ignore more leaks in dependencies

libxkbcommon shows up quite a bit in leak sanitizer
reports. Ignore it.

gsk: Plug a memory leak in mask node fallback

We were forgetting to free the mask pattern.
Found by asan.

gsk: Plug a memory leak in the gl renderer

Found by asan.

Plug a memory leak in gsk tests

Pointed out by asan.

Plug a memory leak in gsk_render_node_serialize

This was introduced in 0d6a6a5997d with named
textures.

maplistmodel: implement GtkSectionModel

This just wraps the underlying GListModel if it is a GtkSectionModel.

Merge branch 'dboles/issue5922-DropTarget-reject-enter-critical' into 'main'

DropTarget: Fix critical if `reject()` drop before `::enter`

Closes #5922

See merge request GNOME/gtk!6152

DropTarget: Fix critical if reject drop pre-enter

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

The docs of `Gtk.DropTarget::accept` say this:
> If the decision whether the drop will be accepted or rejected depends
> on the data, [`::accept`] should return `TRUE`, [`:preload`] should be
> set and the value should be inspected via the `::notify:value` signal,
> calling `gtk_drop_target_reject()` if required.

But this pattern causes a CRITICAL, given these steps:
* Create a `DragSource` and `DropTarget`
* Keep the default `::accept` handler and set `:preload` to `TRUE`
* Connect to `notify::value` and therein call `DropTarget.reject()`
* CRITICAL at `DropTarget.enter()`→`Drop.get_actions()` on NULL instance

We should let the documented case work without a CRITICAL or worse, null
deref. And per @otte on the bug, we should bail earlier before `::enter`
& setting `GTK_STATE_FLAG_DROP_ACTIVE`; neither should occur if rejected

This fixes that, by checking after `start_drop()` when notifications are
thawed, whether any handler has `reject()`ed & set our `drop` to `NULL`.

DropTarget: Fix `if` block indented 1 step too far

Merge branch 'fix-asan-ifunc' into 'main'

Fix fp16 with asan

See merge request GNOME/gtk!6153

Fix fp16 with asan

The IFUNC resolvers that we are using here get
run early, before asan had a chance to set up its
plumbing, and therefore things go badly if they
are compiled with asan. Turning it off makes things
work again.

The gcc bug tracking this problem:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110442

Thanks to Jakub Jelinek and Florian Weimer for
analyzing this and recommending the workaround.

Merge branch 'wip/sadiq/fix-use-after-free' into 'main'

gldriver: Fix a possible use-after-free

See merge request GNOME/gtk!6151