Merge gtk4 (4.6.0+ds1-4) import into refs/heads/workingbranch

Disable optimized GResource embedding

This seems to be triggering failures on armel and armhf. Second-guessing
GResource seems to be more trouble than it's worth.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4617
Signed-off-by: Simon McVittie <smcv@debian.org>
Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-optimized-GResource-embedding.patch

Disable clipboard test

This regularly times out on buildds, but I can't reproduce the hang
locally.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4229
Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-clipboard-test.patch

Skip some known-failing tests on mips-family architectures

Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Skip-some-known-failing-tests-on-mips-family-architecture.patch

templates: Remove html5shiv

Internet Explorer is dead, or really should be, so let's not generate
references to external resources that can be a privacy breach.

Gbp-Pq: Topic debian
Gbp-Pq: Name templates-Remove-html5shiv.patch

Disable web fonts for now

The way this works is still changing upstream, and getting the
correct licensing and source code for the Red Hat font (ITP: #986999)
is not really in the scope of packaging gi-docgen.

Forwarded: not-needed, Debian-specific

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-web-fonts-for-now.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: 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: Name reftest_compare_surfaces-Report-how-much-the-images-diffe.patch

png: Correct endianness for big-endian machines

libpng wants to receive samples in either RGB or RGBA order, whether
each sample is big-endian or not. This resolves test failures in
testsuite/gdk/memorytexture.c (and a lot of reftests) on s390x, and
probably the PowerPC family too.

Modifying the test to show the color in use and write out the PNG bytes
to a file, and running the memorytexture test on s390x, produces a PNG
that loads with the correct color values in GIMP (on an x86_64 machine),
which seems like evidence that this is the correct change and not just
compensating errors.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4616
Signed-off-by: Simon McVittie <smcv@debian.org>
Forwarded: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/4357

Gbp-Pq: Name png-Correct-endianness-for-big-endian-machines.patch

build: Tell glib-compile-resources to make symbols internal where possible

Partial solution to https://gitlab.gnome.org/GNOME/gtk/-/issues/4598

Signed-off-by: Simon McVittie <smcv@debian.org>
Forwarded: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/4334

Gbp-Pq: Name build-Tell-glib-compile-resources-to-make-symbols-interna.patch

Make our stack noexec

The change to use ld and objcopy for resources
had some side-effects: it leaked a few symbols
and made our stack executable. We don't want that.

Use -z nonexecstack and --strip-all to avoid this.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4598

Gbp-Pq: Name Make-our-stack-noexec.patch

gtk4 (4.6.0+ds1-4) unstable; urgency=medium

  * Release to unstable

[dgit import unpatched gtk4 4.6.0+ds1-4]

Import gtk4_4.6.0+ds1-4.debian.tar.xz

[dgit import tarball gtk4 4.6.0+ds1-4 gtk4_4.6.0+ds1-4.debian.tar.xz]

Import gtk4_4.6.0+ds1.orig.tar.xz

[dgit import orig gtk4_4.6.0+ds1.orig.tar.xz]

Merge gtk4 (4.4.1+ds1-3) import into refs/heads/workingbranch

Disable clipboard test

This regularly times out on buildds, but I can't reproduce the hang
locally.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4229
Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-clipboard-test.patch

Skip some known-failing tests on mips-family architectures

Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Skip-some-known-failing-tests-on-mips-family-architecture.patch

testsuite: Don't test the old gl renderer

This was removed in GTK 4.5, and two of its tests crash when run on
mipsel.

Forwarded: not-needed, upstream removed this renderer already

Gbp-Pq: Topic debian
Gbp-Pq: Name testsuite-Don-t-test-the-old-gl-renderer.patch

templates: Remove html5shiv

Internet Explorer is dead, or really should be, so let's not generate
references to external resources that can be a privacy breach.

Gbp-Pq: Topic debian
Gbp-Pq: Name templates-Remove-html5shiv.patch

Disable web fonts for now

The way this works is still changing upstream, and getting the
correct licensing and source code for the Red Hat font (ITP: #986999)
is not really in the scope of packaging gi-docgen.

Forwarded: not-needed, Debian-specific

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-web-fonts-for-now.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: 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: Name reftest_compare_surfaces-Report-how-much-the-images-diffe.patch

gtk4 (4.4.1+ds1-3) unstable; urgency=medium

  * debian/control.in: Don't build ffmpeg backend on architectures where
    ffmpeg doesn't build
  * debian/rules: Build with cloudproviders support (except on Ubuntu)
  * debian/rules: Enable sysprof support on Linux

[dgit import unpatched gtk4 4.4.1+ds1-3]

Import gtk4_4.4.1+ds1-3.debian.tar.xz

[dgit import tarball gtk4 4.4.1+ds1-3 gtk4_4.4.1+ds1-3.debian.tar.xz]

Make fast forward from 4.4.1+ds1-1

[dgit --quilt=unapplied]

Disable clipboard test

This regularly times out on buildds, but I can't reproduce the hang
locally.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4229
Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-clipboard-test.patch

Skip some known-failing tests on mips-family architectures

Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Skip-some-known-failing-tests-on-mips-family-architecture.patch

testsuite: Don't test the old gl renderer

This was removed in GTK 4.5, and two of its tests crash when run on
mipsel.

Forwarded: not-needed, upstream removed this renderer already

Gbp-Pq: Topic debian
Gbp-Pq: Name testsuite-Don-t-test-the-old-gl-renderer.patch

templates: Remove html5shiv

Internet Explorer is dead, or really should be, so let's not generate
references to external resources that can be a privacy breach.

Gbp-Pq: Topic debian
Gbp-Pq: Name templates-Remove-html5shiv.patch

Disable web fonts for now

The way this works is still changing upstream, and getting the
correct licensing and source code for the Red Hat font (ITP: #986999)
is not really in the scope of packaging gi-docgen.

Forwarded: not-needed, Debian-specific

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-web-fonts-for-now.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: 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: Name reftest_compare_surfaces-Report-how-much-the-images-diffe.patch

Release to unstable

d/control.in, d/rules: Gate libgtk-4-media-ffmpeg with a build-profile

If we only build it for experimental uploads, but there is no version
currently in experimental, then its entry in the overrides file will be
removed after a while, causing the next upload to be considered NEW.

Release to unstable

Import gtk4_4.4.1+ds1.orig.tar.xz

[dgit import orig gtk4_4.4.1+ds1.orig.tar.xz]

Make fast forward from 4.4.0+ds1-5

[dgit --quilt=unapplied]

Disable clipboard test

This regularly times out on buildds, but I can't reproduce the hang
locally.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/4229
Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-clipboard-test.patch

Skip some known-failing tests on mips-family architectures

Forwarded: no

Gbp-Pq: Topic debian
Gbp-Pq: Name Skip-some-known-failing-tests-on-mips-family-architecture.patch

testsuite: Don't test the old gl renderer

This was removed in GTK 4.5, and two of its tests crash when run on
mipsel.

Forwarded: not-needed, upstream removed this renderer already

Gbp-Pq: Topic debian
Gbp-Pq: Name testsuite-Don-t-test-the-old-gl-renderer.patch

templates: Remove html5shiv

Internet Explorer is dead, or really should be, so let's not generate
references to external resources that can be a privacy breach.

Gbp-Pq: Topic debian
Gbp-Pq: Name templates-Remove-html5shiv.patch

Disable web fonts for now

The way this works is still changing upstream, and getting the
correct licensing and source code for the Red Hat font (ITP: #986999)
is not really in the scope of packaging gi-docgen.

Forwarded: not-needed, Debian-specific

Gbp-Pq: Topic debian
Gbp-Pq: Name Disable-web-fonts-for-now.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: 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: Name reftest_compare_surfaces-Report-how-much-the-images-diffe.patch

Standards-Version: 4.6.0 (no changes required)

More d/copyright updates

Gbp-Dch: ignore

Stop enabling a Meson option that no longer exists

This was merely a warning with Meson 0.59, but is an error with
Meson 0.60.

Closes: #998551

d/copyright: Update

Rebase patch series, dropping patches that were applied upstream

d/copyright: Update

New upstream release

Update upstream source from tag 'upstream/4.4.1+ds1'

Update to upstream version '4.4.1+ds1'
with Debian dir 5dce23ba28eab97b341a935fb9ce0f846344d457

New upstream version 4.4.1+ds1

4.4.1

NEWS: Updates

gtk: Hold reference on controller until events are fully handled

Fixes premature controller unreferencing in
gtk_event_controller_handle_event() if the controller itself is being
removed while handling the event.

levelbar: Fill the whole space for discrete level bars

Ideally this would be using box layout, but it overrides measure() so it's
not possible - so reimplement it instead. Fix an accidentally int division
along the way.

macos: fix scale on macOS 12 beta

This is a port of the fix in the quartz backend to the new macOS backend.

From the original commit:

In macOS-12.sdk CGContextConverSizeToDeviceSpace returns a negative
height and passing that to CGContextScaleCTM in turn causes the cairo
surface to draw outside the window where it can't be seen. Passing the
absolute values of the scale factors fixes the display on macOS 12 without
affecting earlier macOS versions.

tools/generate-uac-manifest.py: Fix UAC manifest .rc

The resource compiler in the Windows 11 SDK does not allow one to include
winuser.h directly in resource scripts (.rc) with a rather cryptic error
message, so fix generating the .rc file to embed the UAC manifest by including
windows.h with WIN32_LEAN_AND_MEAN instead.

gtk: Fix version resource on for Windows 11 SDK

The rc.exe that comes with the Windows 11 SDK does not allow one to include
winuser.h directly in the .rc scripts, so make sure that it is not included
by gtk-win32.rc.body.in, but instead include windows.h with WIN32_LEAN_AND_MEAN
defined.

calendar: Add an explanatory comment

On the magic numbers used to get the abbreviated day names.

calendar: Fix abbreviated day names

... if the current locale has a different starting day than Sunday.

This needed 2 fixes:
* We need to take into account `calendar->week_start` when
  creating/adding the appropriate `day_name_labels` field
* we were only calculating `calendar->week_start` _after_ attaching the
  `day_name_labels`, so it was still set to 0 (the default value).

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

broadway: Quiet initialisation failures

Broadway is the only GTK+ backend that throws an error on stderr for a
"display server" connection failure.

This causes problems when gtk_init_check() is used and unexpected error
output is generated such as with hotdoc, which fails when generating a
GTK plugin's documentation instead of overlooking the issue.

"Unable to init server: Could not connect: Connection refused"

broadway: Quiet initialisation failures

Broadway is the only GTK+ backend that throws an error on stderr when
failing to initialise, which causes problems when gtk_init_check() is
used and unexpected error output is generated.

This causes hotdoc to fail when generating a GTK plugin's documentation
instead of failing quietly.

"Unable to init server: Could not connect: Connection refused"

wayland: Mark opaque_region as dirty on hide

Otherwise if we hide and show a window we recreate a new surface,
breaking the compositor's association, but potentially not resend this
data for the new surface.
This matches what we do for input_region.

filechooserportal: Take a ref on transient-for

This may fix a crash that has been reported in
file_chooser_portal_data_free.

Fixes: #4314

inspector: Export gtk_inspector_init

Quietly export this function mainly for the benefit
of libadwaita, which can can use this to install its
implementation of the gtk-inspector-page extension
point.

searchenginemodel: finalize search results

It is necessary to signal the search engine that we are finished and
that we found something for it to reliably show the results. It would
sometimes work anyway since it is sufficient if any backend signals
completion. However if GtkSearchEngineModel was the only backend
returning results then things would break.

contentdeserializer: Plug a small memleak

Found by asan in ci. We intern the mimetypes
when they are registered, so there is no need
to leak this string.

a11y: Simplify atspi context a bit

We don't really need a bus-address property
that gets copied for every single object.
We keep the address in object data on the
display anyway. Just use it from there.

This gets rid of a nice amount of strdups
at startup.

docs: Mention popovers in the migration guide

Things have changed around popovers, we should mention
that.

entry: Clarify docs

Mention that max length is in characters.

coloreditor: Don't unref unless we own it

Calling gtk_widget_class_bind_template_child does
*not* give you a reference that you need to unref.
It manages the reference for you. So calling
g_clear_object on such a member is wrong.

builder: Don't leak refs

We sometimes end up setting an object on a GValue
that we are then not interested in. We need to
unset it, or we'll leak the ref.

gtktextview: Fixed arrow key crash.

When pressing the keyboard arrows to move around when the insertion point is
hidden, it causes an assertion error in blink_cb.

Insertion point blinks should only be scheduled when blinking is enabled and the
insertion point is visible.

Closes #4275

gtkcomposetable: Accept long replacement strings

This change removes the assertions limiting replacement strings in the compose table to be less than 20 characters.
The limit seems arbitrary, is not required, will break some users' setups, and problems with it result in applications not launching.

Fixes #4273

Mention that after setting a button child the user is responsible for a11y relations

resources: Treat svgs as xml

They can have their whitespace removed just the same.

Clean up gesture icons

These had duplicate drawing in them, and were
saved with all the extra Inkscape data.

stackswitcher: Expand child buttons

Fixes #4196

filechooserwidget: Return an id in get_choice()

gtk_file_chooser_widget_get_choice() is supposed to return the option
id of the choice, but it currently is returning the option label.

Return the option id instead.

filechooserwidget: Match choice id from "options"

When choices are added to the file chooser widget, the options of
that choice are stored object data under the "options" key. However,
gtk_file_chooser_widget_set_choice() was checking for "choices".

Retrieve the options from the "options" key stored data object data.

gsk: Add a test for transform_bounds

gtkcolorbutton: Fix documentation

testsuite: remove box-order test

The test used to test that GtkBox ordered it's children left-to-right in
CSS, no matter the text direction or pack-type.

But there is neither a pack-type anymore nor does GTK4 do that.

So that test has been broken for yers, it just didn't render anything
wrong.

Do not require wayland-protocols as dependency in the .pc file

Basically, I was building some packages on Guix. I figured out that
wayland-protocols was listed among propagated-inputs for gtk+ package
(gtk-3-24). propagated-inputs holds a list of runtime dependencies,
that should be available to any other package that depends on gtk+.
While discussing we clarified that wayland-protocols is not runtime
dependency. So I moved it to native-inputs of gtk+ package, which
means that, this dependency will be available only to gtk+ package and
only at build time. Once moved, building of other applications that
depening on gtk+ started to fail.

Investigation showed that, all .pc (pkg-config) files prepared by gtk+
package, was including:

Requires.private: ... wayland-protocols ...

Since it becomes requirement, other applications was failing with
missing dependency wayland-protocols of dependency gtk+, for instance:

-- Checking for module 'gtk+-3.0'
--   Package 'wayland-protocols', required by 'gdk-3.0', not found

While actually wayland-protocols is not even a build time dependency
of application that depends on gtk+. Advertisement of such
requirement, is a bit misleading, because one does not need it at
runtime, especially applications based on gtk.

theme: make toggled tbuttons in toolbars visible

- do what Alex does in libadwaita

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

Update the README

Remove the mention of GNU (since that has not been case
for a long time, effectively), state that GTK is hosted
by the GNOME project, and point to GNOME as a place
for donations.

testsuite: Call g_test_init in test binaries

Without it, we mess out on G_DEBUG=fatal-warnings,
so our ci does not alert us that we run a bunch of
tests which spit out warnings.

Fix generation of accessible description in presence of described by relations

The loop going through the relation list should have stopped on the NULL sentinel value, however it never accesed the next list element.

testsuite: Fix a few memory leaks

These are keeping the asan build in ci from passing.

Set correct accessible relations for GtkMenuButton

Up until now, as the focus was moved to the inner button, it was not possible for
assistive technologies to determine the correct labels and descriptions
because developers could set them only for the parent widget.
Now, the proper relations are added so the labels should be picked up properly.

Fixes #4254

media-controls: Make play button flat

Match the volume button.

searchbar: Vertically center the close button

stack: Dispose children before emitting items-changed

This makes sure that the `GListModel` returned by
`gtk_stack_get_pages()` actually has the items removed before
`items-changed` is emitted.

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

GtkApplication: Call setlocale early

If we don't call setlocale early enough,
--help output may come out misformatted.

Fixes: #4234

textview: Invalidate Pango contexts

We need to invalidate the Pango contexts when
font settings change. Use the new helper
gtk_widget_update_pango_context to make it less
likely that we forget to update some things.

widget: Provide a helper for updating pango contexts

Rename an internal function

It occupies a name I want to reuse for something else.

Improve the gtk_accelerator_parse() docs

We need to escape the modifiers in angular brackets, or Markdown will
consider them as HTML tags.

We also should document the modifiers we're parsing.