tests: Mark gltexture as expected to fail on big-endian machines

Forwarded: not-needed, workaround

Gbp-Pq: Topic debian
Gbp-Pq: Name tests-Mark-gltexture-as-expected-to-fail-on-big-endian-ma.patch

gstreamer-player: Use GtkName Prefix to avoid issues if actual library is included

Origin: https://gitlab.gnome.org/3v1n0/gtk/-/commits/gst-player-subproject
Forwarded: not-needed

Gbp-Pq: Topic debian
Gbp-Pq: Name gstreamer-player-Use-GtkName-Prefix-to-avoid-issues-if-ac.patch

media: Use gstplayer from meson subproject

Origin: https://gitlab.gnome.org/3v1n0/gtk/-/commits/gst-player-subproject
Forwarded: not-needed

Gbp-Pq: Topic debian
Gbp-Pq: Name media-Use-gstplayer-from-meson-subproject.patch

gstreamer-player-1.0: Do not install it if it's a subproject

Origin: https://gitlab.gnome.org/3v1n0/gtk/-/commits/gst-player-subproject
Forwarded: not-needed

Gbp-Pq: Topic debian
Gbp-Pq: Name gstreamer-player-1.0-Do-not-install-it-if-it-s-a-subproje.patch

subprojects: Add gst-player-1.0 as subproject

Origin: https://gitlab.gnome.org/3v1n0/gtk/-/commits/gst-player-subproject
Forwarded: not-needed

Gbp-Pq: Topic debian
Gbp-Pq: Name subprojects-Add-gst-player-1.0-as-subporject.patch

Disable inscription-markup.ui reftest

It's rendered 1px to the left of where it should be, which is not a
serious problem for practical use of GTK, but is a test failure.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/5099
Forwarded: not-needed, workaround

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-inscription-markup.ui-reftest.patch

reftests: Allow minor differences to be tolerated

Based on an earlier patch by Michael Biebl, as used in Debian's GTK 3
packaging, with additional inspiration from librsvg's reftests.

Each .ui or .node reftest can have an accompanying .keyfile file
like this:

    [reftest]
    tolerated-diff-level=20
    tolerated-diff-pixels=1000

If the image differs, but the number of pixels that differ is no more
than tolerated-diff-pixels and the differences are no more than
tolerated-diff-level, then we treat it as a success with warnings, save
the .diff.png for analysis, and use g_test_incomplete() to record the
test-case as "TODO".

Signed-off-by: Simon McVittie <smcv@debian.org>
Forwarded: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/3195
Applied-upstream: no, upstream want reftests to be a strict pass/fail with identical results required

Gbp-Pq: Topic debian
Gbp-Pq: Name reftests-Allow-minor-differences-to-be-tolerated.patch

reftest_compare_surfaces: Report how much the images differ

In unattended/non-interactive/autobuilder environments where the images
are not trivially accessible, this provides a way to distinguish between
totally different rendering and more subtle issues.

Signed-off-by: Simon McVittie <smcv@debian.org>
Forwarded: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/3195
Applied-upstream: no, upstream want reftests to be a strict pass/fail with identical results required

Gbp-Pq: Topic debian
Gbp-Pq: Name reftest_compare_surfaces-Report-how-much-the-images-diffe.patch

tests: mark headless tests as flaky

See baaa7482489b0f7446ef1e0130e527a169c43b0c

Gbp-Pq: Name tests-mark-headless-tests-as-flaky.patch

Revert "build: Drop the install-tests option"

Debian and Ubuntu uses these installed tests as part of their
autopkgtest system

This reverts commit 3121f88265ac61733e257f5335473d6f139f158c.

Gbp-Pq: Name Revert-build-Drop-the-install-tests-option.patch

Revert "tests: Stop copying the tool tests"

This reverts commit bb3a6214cbeaa3211be007192e536bd473323836.

This is actually used by the installed tests
which we also had to revert dropping (in a separate patch)

Gbp-Pq: Name Revert-tests-Stop-copying-the-tool-tests.patch

gsk: Always use RGBA on big-endian architectures

The memorytexture test started failing on big-endian CPUs while using
"desktop" GL (not GLES) in commit b0e26873 "gsk: Use has_bgra in more
places". The subsequent commit f5812808 "gsk: Restore bigendian support"
was meant to fix this, but it seems it did not.

Always taking the RGBA code-path on big-endian might not be optimal,
but it seems to result in the test passing again.

Signed-off-by: Simon McVittie <smcv@debian.org>
Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/6260
Bug-Debian: https://bugs.debian.org/1057782
Forwarded: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/6653

Gbp-Pq: Name gsk-Always-use-RGBA-on-big-endian-architectures.patch

Release to unstable

d/control: Remove pandoc build-dependency

It isn't needed any more, and pandoc is currently uninstallable in
unstable (#1057309, #1053686).

Release to unstable

Add proposed patch to fix test failure on big-endian CPUs

Closes: #1057782

Build-Depend on dh-sequence-gnome

used by Ubuntu to pull in dh_translations

releasing package gtk4 version 4.12.4+ds-1

New upstream release

Update upstream source from tag 'upstream/4.12.4+ds'

Update to upstream version '4.12.4+ds'
with Debian dir bb344c20763a3a49aa1f106346878e34b17e38e0

New upstream version 4.12.4+ds

4.12.4

Fix warnings when compiling with -pedantic

Add a debug node for CSS outlines

All the other CSS render operations have one, so outlines should, too.

This was found by tracking down a random border node in a rendering.

gtk: fix NULL pointer dereference

`gtk_window_get_default_size()` claims width/height are optional-out
arguments, but defers to `gtk_window_get_remembered_size()` which
may dereference a NULL-pointer.

Since `gtk_window_get_remembered_size()` is only called by
`gtk_window_get_default_size()`, collapse it into the latter
and perform the NULL check there.

inspector: Replace "no glyph" with space glyph

This makes right-aligning the text work again.

inspector: Use the right font for the fps overlay

Initialize the font only when we have a widget that provides it. That
way we don't pick a terrible default font, but the widget's preferred
one.

Some minor fpsoverlay improvements

Avoid some allocations.

inspector: Speed up the fps overlay

Do less work!

docs: Replace deprecated gtk_widget_show with gtk_widget_set_visible

gl: Fix padding in icon library

The previous code would not pad the left side with the left column of
pixels but with the top row.

Test attached.

gsk: Fix texture-scale node Cairo drawing

The code was interpreting x/y coordinates wrong sometimes and causing
things to not be drawn at the correct offset.

Testcase included.

rendernode-tool: Add --undecorated to show command

This is to get exactly just the rendernode drawn when testing, and not
any extra drawing code.

widget-factory: Remove insert-image icon

This icon is already present in the hicolor icon theme.

widget-factory: Remove view-grid-symbolic icon

No longer needed as it's included in the hicolor icon theme now.

icons: Add view-grid-symbolic

This showed up as missing icon in the file chooser when used with
the hicolor icon theme.

Resolves: https://gitlab.gnome.org/GNOME/gtk/-/issues/6184

Adjust tests

a11y: When using rule 2.E for computing accessible name, use it only if appropriate

We were using it in all cases, so, we were using it to compute descriptions,
and also for non-embedded controls. That was overriding descriptions
set, for example, in Gnome settings, and was causing the value of spinboxes
to be read multiple times.

builds: Require -Zc:preprocessor for Visual Studio debug builds

This flag is actually required for the debugging code to successfully build,
so check that it is really there for debug-enabled Visual Studio builds.

MSVC Builds: Don't enable -utf-8 explicitly

We already require a Meson release that enables -utf-8 by default, so we don't
really need to explicitly enable it here.

build: Add msvc_recommended_pragmas.h

We really always want to force-include msvc_recommended_pragmas.h to check for
things at compile time so that we can avoid stuff like missing includes or
attempting to return a value in a function that is supposed to have a
void-return-type.

The current problem is that, as indicated in the Visual Studio CI job, that we
couldn't locate msvc_recommended_pragmas.h during the build if GLib is built
as a subproject, and/or when msvc_recommended_pragmas.h is not in the paths
indicated by %INCLUDE%, meaning that the aforementioned issues would not be
caught by CI, which will then break builds on Visual Studio for people when
msvc_recommended_pragmas.h is found during their builds.

It would also be nice to be quiet from the warnings that we can really
disregard anyways.

So, add a copy of msvc_recommended_pragmas.h from GLib and update the build
files to look for it in build-aux/msvc, so that it can always be used during
the build, especially by the CI.

Fix swizzle values for some memory formats

For opaque formats with 3 channels, we should use the default
GL_ALPHA, but for opaque formats with an ignored 4th channel,
we must use GL_ONE.

translated bn.po and hi.po

Resolves:https://gitlab.gnome.org/GNOME/gtk/-/issues/6164

gl: Remove optimization that does the wrong thing

Drawing a texture-scale node like a texture node when the filter is set
to "linear" doesn't work, because the texture node switches to
trilinear when mipmaps are available.

gl: Make sure render_texture() sets the right format for high depth

Setting the format got lost when converting this coe to the texture
builder, because that codepaths avods the texture sniffing and always
uses RGBA8.

gdk: Make float32 report its true depth

I have no idea how this is the only value that is wrong.

Release to unstable

d/control: Use ${gir:Provides} to generate virtual package names

d/control: Add ${gir:Depends} to -dev package

This will add dependencies on gir1.2-pango-1.0-dev and so on.

d/control: Add more gir1.2-x-y-dev build-dependencies

Gbp-Dch: ignore

d/control: Build-depend on relevant gir1.2-*-dev virtual packages

This will help with eventually converting some or all of them into real
packages, to allow GIR XML to be omitted when cross-compiling or
bootstrapping.

Update Russian translation

Merge branch 'invalid-client-rects-4-12' into 'gtk-4-12'

[4.12] GdkWin32: ignore invalid client rects

See merge request GNOME/gtk!6545

GdkWin32: ignore invalid client rects

Gdk-Win32 uses GetClientRect() internally to query the surfaces coordinates,
but this API may fail in some transient contexts (observed when iconifying
a maximized window).
Check if the rect area is null, and don't update the surface position in
that case. This will keep the current surface size, until Win32 notifies
the new valid window state later.
This prevents using a nulled next_layout for toplevel size computation,
which would break widgets allocation once notified on gtk side.

Fixes #5724
Closes #5724

Release to unstable

lbgtk-4-dev: Add Provides for included GIR XML files

Representing these by systematic names gir1.2-*-dev, and perhaps
eventually separating them into their own binary package, will help to
enable cross-compilation with GObject-Introspection removed by a build
profile.

Helps: #1030223

d/rules: Remove remnants of generating d/control from d/control.in

d/rules: Stop generating d/control.in from d/control

The variable substitutions obscured more than they really helped.

Substitute GTK_BINVER_DEP as a substvar rather than using sed

This will make it easier to stop making d/control a generated file.

Merge branch 'wip/chergert/for-4-12' into 'gtk-4-12'

cherry-pick bgra/vertex-array checks to 4.12

See merge request GNOME/gtk!6541

gsk: Use vertex arrays when we can

Use the new has_vertex_arrays api to determine
whether we can use vertex arrays in GL.`

Fixes: #6173

glcontext: Add api to check for vertex arrays

Vertex arrays are available in GL and in GLES >= 3.

We don't check for the GLES extension that provided
vertex arrays in older GLES, since that requires
using different API.

This api avoids version checks all over the place.

gsk: Restore bigendian support

Restore the bigendian support that was lost in b0e26873f66a8f52,
by just not using GL_BGRA with GLES on bigendian. Should be a
very rare combination, but still.

gsk: Use has_bgra in more places

The glyph and icon libaries were also checking for GLES to
decide if data needs to be transformed from BGRA to RGBA.

Use the new has_bgra getter instead.

This will probably break on bigendian, because the
GL_BGRA + GL_UNSIGNED_BYTE combination is not equivalent
to the cairo format on bigendian, but this was already
broken for the gl format information that we get from
gdk_memory_format_gl_format.

glcontext: Check for GL_EXT_texture_format_BGRA8888

Check for this GLES extension and add a private getter.

Update French translation

Update Turkish translation

Merge branch 'macos_ci' into 'gtk-4-12'

Backport macOS CI to 4-12 to use new runner

See merge request GNOME/gtk!6497

gdk: Use subpixel_layout on macOS

(cherry picked from commit 9aeb5be8ad91f83030f3c5e5bdd68a47b4b1a970)

gdk: Remove a leftover reference to the renamed variable prefers_high_depth

(cherry picked from commit aa888c0b3f775776fe3b71028396b7a8c6adb1d6)

ci: Backport CI yaml from main

This makes the CI yaml identical to the one in main, which changes
two things:
- modernize workflow configuration
- use a new macOS runner for macOS CI

Update Romanian translation

Update Spanish translation

Merge branch 'no-emus-in-bitfields-4.12' into 'gtk-4-12'

[4.12] Stop using enums in bitfields

See merge request GNOME/gtk!6491

Stop using enums in bitfields

The C standard does not specify whether the underlying type of an enum
is signed or unsigned, and until C23 there was no way to control this
explicitly. GCC appears to make enums unsigned unless there is a
negative value among cases of the enum, in which case it becomes signed.
MSCV appears to make enums signed by default.

A bitfield of an enum type (which is not specificied in the C standard
either) behaves as if it was an instance of a numeric type with a
reduced value range. Specifically, a 'signed int val : 2;' bitfield will
have the possible values of -2, -1, 0, and 1, with the usual wraparound
behavior for the values that don't fit (although this too is
implementation-defined).

This causes the following issue, if we have:

typedef enum
{
  GTK_ZERO,
  GTK_ONE,
  GTK_TWO
} GtkFoo;

struct _GtkBar
{
  GtkFoo foo : 2;
};

and then assign bar.foo = GTK_TWO and read it back, it will have the
expected value of 2 (aka GTK_TWO) on GCC, but a value of -2 (not
matching any of the enum variants) on MSVC.

There does not seem to be any way to influence signedness of an enum
prior to C23, nor is there a 'unsigned GtkFoo foo : 2;' syntax. The only
remaining options seems to be never using enums in bitfields, which is
what this change implements.

In practice, this fixes GdkPipeIOStream crashing with an assertion when
trying to copy-paste in-app in MSVC builds on GTK.

Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>

Update Russian translation

Update Turkish translation

releasing package gtk4 version 4.12.3+ds-1

* debian/changelog: Mention bug fixes for 4.12.3 from upstream NEWS.

New upstream release

Update upstream source from tag 'upstream/4.12.3+ds'

Update to upstream version '4.12.3+ds'
with Debian dir 24db8d6ef94eb48755b60abcf4a00ec1468721df

New upstream version 4.12.3+ds

Update Catalan translation

Post-release version bump

4.12.3

print portal: Report errors properly

The portal printoperation inmplementation
relies on the file printbackend to be available.
If it isn't, we should report a proper error
status insetad of running into assertions deep
inside the printoperation code.

gsk/gl: remove TypeNode conformity checking for renderjob

We don't need to be calling type node conformity checking from the tight
loop of the renderjob. Hoist that into the private header and use that
intead through via the Class pointer.

gsk: remove excessive type checking within GSK

Anything that includes gskrendernodeprivate.h will get an alternate form
of ref/unref for render nodes which does not need to do type checking on
the parameter. We can expect that things are correct within GTK itself and
this saves excessive amounts of TypeNode conformities checking.

printoperation: fix some strange line spacing

printoperation: add some assertions

Let's assert that we schedule the idle callback exactly once.

These assertions are not perfect because if the callback executes before
we schedule it, then the assertion itself would be a use-after-free,
since I'm using the PrinterFinder to track whether the callback that
frees it has been scheduled. But in practice when using loupe's print
dialog, I was noticing the callback scheduled twice before it was
executed. The assertion would have caught this problem.

printoperation: fix another case where operation may complete twice

This is a little tricky. At first, I thought we had a codepath where we
fail to schedule the idle that completes the print operation: if we take
the gtk_print_backend_printer_list_is_done path for each printer
backend, then printer_list_done_cb() is never executed and we never
schedule the idle. But in fact, in this case, then backends == NULL at
the bottom of find_printer(), and we'll schedule the idle there, so it's
OK. Except it's not really OK, because we'll schedule it even if a
printer was already found, resulting in the callback completing twice
and a double free.

Simplify this. Schedule the idle in find_printer() only if there are
*initially* no backends, not also if all backends are immediately ready
and already removed from consideration. Instead, always call
printer_list_done_cb() for every backend in find_printer_init(). After
the previous commit, printer_list_done_cb() will schedule the idle when
appropriate.

printer_list_done_cb() additionally disconnects signals that we did not
connect in this codepath, but it does so using
g_signal_handlers_disconnect_by_func, which is harmless. Otherwise, the
only extra work it's doing is scheduling the idle, and that's exactly
what find_printer_init() is missing.

printoperation: fix case where operation may complete multiple twice

If we are the final backend, then after removing ourselves there is no
backend remaining. We will schedule the idle even if it has already been
scheduled. This idle is required to run exactly once and executing it
twices results in a double free that crashes loupe when printing. It
also causes the user callback to execute twice, which could cause
similar problems.

Fixes #6122

gsk/gl: use GdkArrayImpl for tracking modelview

Like the previous change, this uses GdkArrayImpl instead of GArray for
tracking modelview changes. This is less important than clip tracking
simple due to being used less, but it keeps the implementation synchronous
with the Clip tracking code.

gsk/gl: use GdkArrayImpl for Clip tracking

We can end up spending a lot of time in g_array_maybe_expand() through the
use of g_array_set_size() for clip tracking. That is somewhat due to the
simple nature of GArray being size-dynamic. Instead, we can use
GdkArrayImpl and let the compiler do what it does best to elide some
work and hoist other work into the calling function.

This also fixes a potential UAF in gsk_gl_render_job_push_contained_clip().

gdk: add missing G_END_DECLS to gdkarrayimpl.c

broadway: Do not add an extra reference when caching textures

We just created a GdkTexture, so we don't need to acquire a reference if
we're transferring the ownership to the node cache.

broadway: Plug another leak

When getting a colorized texture we're downloading the texture as a
Cairo surface, and then feeding it to another texture, but we never drop
the reference of the new surface.

docs: Clarify the behaviour of gdk_texture_new_for_surface()

Cairo surfaces are not GObject instances, so we should be more explicit
in the behaviour of the memory management, to avoid leaks.

broadway: Plug a leak in the GSK renderer

We are leaking the Cairo image surface when creating a new node.

Fixes: #6120

array: Compute new size properly

Using "1 << x" means that we are shifting a signed 32bit integer, but we
want a gsize, which is an unsigned 64bit integer.

So now we don't overflow anymore if the array reaches a size of 2GB.