Make fast forward from 4.12.3+ds-2

[dgit --quilt=unapplied]

tests: Allow longer for a dialog to open

This hopefully makes the test pass more reliably, especially on riscv64.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/6050
Bug-Debian: https://bugs.debian.org/1049434
Forwarded: no, workaround

Gbp-Pq: Topic debian
Gbp-Pq: Name tests-Allow-longer-for-a-dialog-to-open.patch

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

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.

Release to unstable

Declare fast forward from 4.12.3+ds-1

[dgit --quilt=unapplied --overwrite]

tests: Allow longer for a dialog to open

This hopefully makes the test pass more reliably, especially on riscv64.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/6050
Bug-Debian: https://bugs.debian.org/1049434
Forwarded: no, workaround

Gbp-Pq: Topic debian
Gbp-Pq: Name tests-Allow-longer-for-a-dialog-to-open.patch

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

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.

releasing package gtk4 version 4.12.3+ds-1

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

Record gtk4 (4.12.3+ds-1) in archive suite sid

New upstream release

tests: Allow longer for a dialog to open

This hopefully makes the test pass more reliably, especially on riscv64.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/6050
Bug-Debian: https://bugs.debian.org/1049434
Forwarded: no, workaround

Gbp-Pq: Topic debian
Gbp-Pq: Name tests-Allow-longer-for-a-dialog-to-open.patch

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

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

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

gtk4 (4.12.3+ds-1) unstable; urgency=medium

  * New upstream release
    - GtkWindow:
      + Don't assume titlebars are GtkHeaderBar
        (https://gitlab.gnome.org/GNOME/gtk/-/issues/6116)
    - GtkTreeView:
      + Fix a crash in gtk_tree_view_is_blank_at_pos
        (https://gitlab.gnome.org/GNOME/gtk/-/issues/6114)
    - printing:
      + Fix some issues with the portal implementation
        (https://gitlab.gnome.org/GNOME/gtk/-/commit/03679fe6bf5ae391bd7cb21ad0492f530c86cd5e)
    - GSK:
      + Some optimizations in the GL renderer
        (https://gitlab.gnome.org/GNOME/gtk/-/commit/3698cadcd21e2a52c30766d311043093ea557b3f,
         https://gitlab.gnome.org/GNOME/gtk/-/commit/d4d0883a924f94fdc89fad50bce9ecad3128e0f3,
         https://gitlab.gnome.org/GNOME/gtk/-/commit/0030c572d47f551847be0aed8054d4df34afffda)
      + Fix memory leaks in the Broadway renderer
        (https://gitlab.gnome.org/GNOME/gtk/-/issues/6120)
    - demos:
      + Fix a crash in gtk4-demo
        (https://gitlab.gnome.org/GNOME/gtk/-/issues/6113)

[dgit import unpatched gtk4 4.12.3+ds-1]

Import gtk4_4.12.3+ds.orig.tar.xz

[dgit import orig gtk4_4.12.3+ds.orig.tar.xz]

Import gtk4_4.12.3+ds-1.debian.tar.xz

[dgit import tarball gtk4 4.12.3+ds-1 gtk4_4.12.3+ds-1.debian.tar.xz]

New upstream version 4.12.3+ds

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.

gdk: Fix compiler warning

gcc's -Wlto-type-mismatch found the hack, where we copied the wrong
prototype.

treeview: Fix crash in assertion

The fix in commit a267dfac5d25d4ba5c1a95c82999d008fee07ac3 is wrong.
The function can return FALSE in normal operation.

Instead do a check for node == NULL that gracefully returns FALSE instead.

Fixes: #6114

Improve FOO_DEBUG=help output

Explain the all value a bit better.

dialog: Check header bar type before calling track_default_decoration()

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

header-bar: Add a precondition to track_default_decoration()

gtk-demo: Fix a refcounting mishap

gtk_font_dialog_button_new's argument is
transfer full.

Fixes: #6113

docs: Mention for what GtkStringObject:string is useful

docs: Link GtkStringObject from GtkStringList

A frequently asked question is: "what is the object type inside a
GtkStringList", so let's make the answer more obvious.

docs: Add a link to the DropDown:selected property

The main description is a bit too coy, and some newcomers don't
understand the concept of properties and notification.

Update German translation

Update Lithuanian translation

Update Brazilian Portuguese translation

Update Polish translation

Update Esperanto translation

Update Persian translation

Update Hungarian translation

releasing package gtk4 version 4.12.2+ds-1

Build-Depend on fonts-noto-core instead of fonts-dejavu-core

& ttf-bitstream-vera

These days, fontconfig defaults to Noto instead of DejaVu

Also needed to adapt to
53cb64351d081e4c20fcea905370d49ff39636dd

New upstream release

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

Update to upstream version '4.12.2+ds'
with Debian dir a12884d491074d1768a10765e52031eca50682ce

New upstream version 4.12.2+ds

Update Basque translation

Update Ukrainian translation

Update Kazakh translation

Merge branch 'fix-issue-6080-gtk-4-12' into 'gtk-4-12'

GdkWin32: Unref GdkMonitor after calling gdk_monitor_invalidate () [4.12]

See merge request GNOME/gtk!6412

Update Czech translation

Update Galician translation