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

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.

widget: Redraw when font options change

Its the right thing to do, even if it is a rare event.

reftest-compare: Treat colour channels as undefined if alpha is zero

If the alpha channel is zero, it doesn't matter what the values of the
red, green and blue channels are: the pixel is still fully transparent.
On most architectures, fully transparent pixels end up all-zeroes
(fully transparent black), matching what's in the reference PNG file;
but on mips*el the blend-difference and blend-normal tests get all-ones
(fully transparent white) and a test failure.

Resolves: https://gitlab.gnome.org/GNOME/gtk/-/issues/4227
Signed-off-by: Simon McVittie <smcv@debian.org>

build: fix implicit-fallthrough errors with clang

The clang build fails due to -Werror=implicit-fallthrough being
on by default and some fallthrough cases not being marked as such.

Use G_GNUC_FALLTHROUGH or duplicate the code in those cases.

Fix wrong gettext ITS target

The installed ITS rule filename is "gtk4builder.its". The .loc file
is wrongly pointing to old "gtkbuilder.its" which makes gettext fail
on systems without GTK3 installed.

Release an extra reference on GtkExpander:child

The child of a GtkExpander is owned directly by the expander whenever
the "expanded" flag is unset.

We are adding an additional reference to the child of an expander when
expander is not expanded.

Additionally, if a GtkExpander is disposed while not expanded, we need
to explicitly release the reference on the child widget that we own.

This reference leak was masked in GTK3 by GtkContainer removing each
child from the parent container by recursively calling
gtk_widget_destroy().

docs: Fix documentation for gtk_icon_view_create_drag_icon

Fixes: #4190

build: Fix detection for pre-compiled css files

The default theme changed from Adwaita to Default and this tripped up
the logic to detect if the tarball builds contain pre-built css files or
not. Fix this by looking at pre-compiled css files in themes/Default/
instead of themes/Adwaita/.

docs: Expand gdk_clipboard_set() docs

Update Latvian translation

Update Catalan translation

Update Russian translation

Merge branch 'cherry-pick-ac8c4245' into 'gtk-4-4'

css: Don't throw warnings on broken URL in image css

See merge request GNOME/gtk!4098

css: Don't throw warnings on broken URL in image css

If a URL can't be loaded, we might end up with a NULL file. Handle that
case properly by creating an invalid image instead and don't crash or
complain to stderr when files are NULL.

This was broken since 0886ade182f1a958024202d5c20248d57fcdb053

A new reftest has been included. We need a reftest instead of a
CSS parser test, because the error only becomes visible when
compute()ing the actual image.

Fixes #4373

(cherry picked from commit ac8c4245b2fb2f1fe8e660c4ad2541efb39cf3d6)

Update Latvian translation

Merge branch 'cherry-pick-3c9c7a0e' into 'gtk-4-4'

build: Make sure ifuncs exist when using fp16

See merge request GNOME/gtk!4089

build: Make sure ifuncs exist when using fp16

Fixes msys compiles on Windows (and everywhere else hopefully).

Fixes #4285

(cherry picked from commit 3c9c7a0e727eb6380bee29120736dba6da2232a3)

Update Croatian translation

Merge branch 'gtk-4-4nodate' into 'gtk-4-4'

Drop date attribute from our appdata files

See merge request GNOME/gtk!4077

Drop date attribute from our appdata files

in order to make builds reproducible.
See https://reproducible-builds.org/ for why this is good

This was suggested by Matthias Clasen as an alternative to MR !3929

Merge branch 'backport-mr-3930-to-gtk-4-4' into 'gtk-4-4'

Fix DND coordinates on Windows

See merge request GNOME/gtk!4049

Update Icelandic translation

Update Brazilian Portuguese translation

Merge branch 'cherry-pick-d89e82d4' into 'gtk-4-4'

Merge branch 'wip/otte/for-master' into 'gtk-4-4'

See merge request GNOME/gtk!4055

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

x11: Don't try to move destroyed windows

See merge request GNOME/gtk!4054

(cherry picked from commit d89e82d4a082febfe44d3504786fcab91eab62a3)

f40ce51a x11: Don't try to move destroyed windows

Update Basque translation

Fix DND coordinates on Windows

Make the DND indicator not interfere with the drag and drop operation

libgtk-4-dev Suggests gtk-4-examples

Closes: #995752

Update Slovak translation

Update Basque translation

debian/rules: Correctly enable auto features for deb build

Update Romanian translation

Update Icelandic translation

Update Indonesian translation

Merge branch 'cherry-pick-d61c71c3' into 'gtk-4-4'

macos: Actually set the vfuncs

See merge request GNOME/gtk!4007

macos: Actually set the vfuncs

(cherry picked from commit d61c71c3780e06cdc13599aa3a334f3124bab276)

Update Hebrew translation

Updated Spanish translation

Update Portuguese translation

Update Croatian translation

Update Croatian translation

Update Polish translation