testsuite: Skip tests that miss charset support

This came up when running the tests in a fresh
toolbox missing the glibc-gconv-extra package.

Merge branch 'wip/corey/at-spi-context' into 'main'

atspicontext: Cache empty atspi bus address

See merge request GNOME/gtk!5276

Merge branch 'show-hide-cleanup' into 'main'

gtk: Stop using gtk_widget_show/hide

See merge request GNOME/gtk!5273

atspicontext: Log more helpful warnings when the a11y bus doesn't exist

atspicontext: Cache empty atspi bus address

We are caching the bus address as data on the display object when it
exists, but fail to set the data when the bus address doesn't exist.
That causing excessive calls to GetAddress when the accesssbility
bus doesn't exist. Make sure to cache a non-existent accessibility
bus by setting the "" string.

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

Revert "builder: Stop using GtkBuildable.get_id"

See merge request GNOME/gtk!5274

Merge branch 'wip/carlosg/osk-activation-in-widgets' into 'main'

Handle OSK activation in GtkText/GtkTextView widgets

See merge request GNOME/gtk!5269

Revert "builder: Stop using GtkBuildable.get_id"

This reverts commit 92edd4c476917e83c5f50e5eab8aa5a0ad240793.

This commit changed behavior that GNOME Builder relies on.

tools: Stop using gtk_widget_show/hide

constraint editor: Stop using gtk_widget_show/hide

icon browser: Stop using gtk_widget_show/hide

gtk-demo: Stop using gtk_widget_show/hide

widget-factory: Stop using gtk_widget_show/hide

gtk: Stop using gtk_widget_show/hide

gtk_widget_set_visible and gtk_window_present
are better alternatives, and calling gtk_widget_show
on newly created widgets is no longer necessary
anyway.

Merge branch 'wip/corey/tree-expander' into 'main'

treeexpander: Add hide-expander and indent-for-depth properties

Closes #4969

See merge request GNOME/gtk!5268

imcontextwayland: Set up OSK activation gesture on non text widgets

With GtkText and GtkTextView (and in extension, all their subclasses)
handling OSK activation activation, this gesture is only useful for
all text input widgets that are not subclasses of these 2 widgets,
e.g. the VTEs and crosswords of the world.

These still do need a hand in handling OSK activation, so only
set up the gesture for such cases.

gtktextview: Invoke OSK on button/touch taps that move/undo selection

If the ::release handler is invoked, the press/release happened without
drags in between. Additionally check that there is no selection at all.

This makes OSK invoked on taps that move the caret around, while tapping
in the selection invokes edition popup and text handles without bringing
in the OSK.

gtktextview: Claim drag gesture after an actual selection change

This way, the drag gesture lets the click gesture ::release handler
happen if there was no actual changes to the selected text (i.e.
too short drags). This matches the ::release handler behavior match
the situations in which the OSK was being invoked by the wayland
GtkIMContext.

gtktext: Invoke OSK on button/touch taps that move/undo selection

If the ::release handler is invoked, the press/release happened without
drags in between. Additionally check that the press did not happen within
the selection, and that there is no selection at all.

This makes OSK invoked on taps that move the caret around, while tapping
in the selection invokes edition popup and text handles without bringing
in the OSK.

gtktext: Claim drag gesture after an actual selection change

This way, the drag gesture lets the click gesture ::release handler
happen if there was no actual changes to the selected text (i.e.
too short drags). This matches the ::release handler behavior match
the situations in which the OSK was being invoked by the wayland
GtkIMContext.

imcontextwayland: Implement activate_osk() vmethod

This method does explicitly what the builtin GtkClickGesture in
the IM context is doing implicitly. The gesture will be dropped
after porting is done.

immulticontext: Implement activate_osk() vmethod

The multicontext lets this call go through to the active
underlying GtkIMContext.

imcontext: Add private activate_osk() vmethod

This method is so far private for both external GtkIMContext
implementations and external GtkIMContext users, and is meant
to activate the OSK in the environments where this may happen.

Revert "gtktext: Claim gesture more selectively"

This reverts commit ae54374a514aa33b34f95eee94b4a2848b8a16d9.

Merge branch 'bilelmoussaoui/gi-since' into 'main'

g-i: Add missing since annotations

See merge request GNOME/gtk!5271

g-i: Add missing since annotations

treeexpander: Minor formatting changes

treeexpander: Add indent-for-depth property

When set to TRUE, indent-for-depth indents each level of
depth with an additional indent

treeexpander: Add hide-expander property

When set to TRUE hide-expander hides the expander icon in a
GtkTreeListRow.

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

Merge branch 'wip/corey/spi-logging' into 'main'

atspicontext: Log failure to connect to a11y

See merge request GNOME/gtk!5270

Merge branch 'work/zzag/wl-output-release' into 'main'

gdk/wayland: Prefer wl_output_release() over wl_output_destroy()

See merge request GNOME/gtk!5243

atspicontext: Log failure to connect to a11y

GTK depends on the a11y infrastructure to be in place unless GTK_A11Y is
set to none.  It appears that despite that, users attempt to
get around the a11y requirement without setting GTK_A11Y.
This can cause, amongst other issues, performance problems
with gtk applications.  Log failure to connect to the a11y
bus.

Merge branch 'fix-keymap-notify' into 'main'

Don't emit ::notify from a getter

See merge request GNOME/gtk!5267

Don't emit ::notify from a getter

GdkX11Keymap ddetermines the direction of the
keymap on demand, and was emitting notify::direction

Merge branch 'wip/corey/demo-file-dialog' into 'main'

gtk-demo: Application demo use open_finish()

See merge request GNOME/gtk!5264

gtk-demo: Application demo use open_finish()

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

gtk-demo: Cosmetics

See merge request GNOME/gtk!5265

gtk-demo: Cosmetics

Merge branch 'bilelmoussaoui/g-i' into 'main'

g-i: Add missing since annotation

See merge request GNOME/gtk!5259

Merge branch 'wip/smcv/xfail' into 'main'

testsuite: Avoid using should_fail

See merge request GNOME/gtk!5248

g-i: Add missing since annotation

Merge branch 'dialog-improvements' into 'main'

gtk-demo: Test cancellation support for file dialog

See merge request GNOME/gtk!5257

file dialog: Handle Escape better

Closing the dialog by hitting Escape should
have the same effect as clicking the cancel
button. Make it so.

color dialog: Handle Escape better

Closing the dialog by hitting Escape should
have the same effect as clicking the cancel
button. Make it so.

gtk-demo: Test cancellation support for file dialog

Add a timeout that closes the file dialog after
20 seconds, to test programmatic cancellation.

testsuite: Don't create .test files for flaky or failing tests

These tests can be run manually, but are not suitable for use as an
acceptance test, so let's not make frameworks like Debian's autopkgtest
run these when they run ginsttest-runner in the most obvious way.

Signed-off-by: Simon McVittie <smcv@debian.org>

testsuite: Use separate setups for unstable tests instead of should_fail

There are two possible interpretations of "expected failure": either
the test *must* fail (exactly the inverse of an ordinary test, with
success becoming failure and failure becoming success), or the test
*may* fail (with success intended, but failure possible in some
environments). Autotools had the second interpretation, which seems
more useful in practice, but Meson has the first.

Instead of using should_fail, we can put the tests in one of two new
suites: "flaky" is intended for tests that succeed or fail unpredictably
according to the test environment or chance, while "failing" is for
tests that ought to succeed but currently never do as a result of a
bug or missing functionality. With a sufficiently new version of Meson,
the flaky and failing tests are not run by default, but can be requested
by running a setup that does not exclude them, with a command like:

    meson test --setup=x11_unstable --suite=flaky --suite=failing

As a bonus, now that we're setting up setups and their excluded suites
programmatically, the gsk-compare-broadway tests are also excluded by
default when running the test setup for a non-broadway backend.

When running the tests in CI, --suite=gtk overrides the default
exclude_suites, so we have to specify --no-suite=flaky and
--no-suite=failing explicitly.

This arrangement is inspired by GNOME/glib!2987, which was contributed
by Marco Trevisan.

Signed-off-by: Simon McVittie <smcv@debian.org>

Merge branch 'deprecate-infobar' into 'main'

Deprecate GtkInfoBar

See merge request GNOME/gtk!5246

Merge branch 'wip/carlosg/gsk-clipping' into 'main'

gsk: Extend the blur radius for outset shadow extents

See merge request GNOME/gtk!5247

gdk/wayland: Improve readability of wl_output binding code

Provide OUTPUT_VERSION macro to indicate the implemented version of
wl_output in gtk rather than use a magical number.

gsk: Extend the blur radius for outset shadow extents

By dividing the blur radius to obtain the clip radius, we may end up
with halved values that result in an overshunk clip mask. Extend this
so that we ensure to cover the last pixel.

Fixes artifacts seen with the cairo renderer in X11 when resizing
windows horizontally, a black 1px high line would be seen in the
top of the window due to these outset bounds being used in clipping.

More mysteriously, also seems to fix resize lag in the GL renderer
(also X11), if e.g. the bottom-right corner of a window is resized
diagonally in bottom-left -> top-right direction, or
bottom-right -> top-left.

Related: https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/2175#note_1599335

Deprecate GtkInfoBar

This widget has a dialog-like API, which is something
we want to get rid of in GTK 5, and libadwaita has
a replacement with AdwBanner incoming.

gsk: Also ceil() top side when converting graphene to cairo rectangles

The outset border for all sides is ceil()ed, except for the top side.
Most often values are already nicely rounded here, but this seems an
oversight.

gdk/wayland: Prefer wl_output_release() over wl_output_destroy()

By using wl_output_release(), GDK lets the compositor to clean up the
output global more nicely.

For example, currently, most compositors remove the global and then
destroy it later after N seconds expire. With this, the compositor could
experiment with destroying the output global once all its resources are
destroyed.

Merge branch 'work/zzag/wl-output-cleanup' into 'main'

gdk/wayland: Destroy zxdg_output_v1 upon GdkWaylandMonitor destruction

See merge request GNOME/gtk!5242

gdk/wayland: Clear GdkWaylandMonitor::output using g_clear_pointer()

Use g_clear_pointer() to make tear down code consistent.

gdk/wayland: Destroy zxdg_output_v1 upon GdkWaylandMonitor destruction

It prevents potentially leaking memory.

Merge branch 'file-filter-properties' into 'main'

filefilter: Add properties for the criteria

See merge request GNOME/gtk!5241

filefilter: Add properties for the criteria

Add construct-only properties for patterns,
suffixes and mime-types. This provides an
alternative way to specify these in ui files.

Related: #5350

Merge branch 'color-picker-portal' into 'main'

colorpicker: Always try the portal first

See merge request GNOME/gtk!5240

colorpicker: Always try the portal first

It does not hurt., and going forward, we want to
prefer portals.

Merge branch 'bilelmoussaoui/g-i' into 'main'

g-i: add missing array annotation to AlertDialog.set_buttons

See merge request GNOME/gtk!5234

Merge branch 'fix-upside-down-labels' into 'main'

gsk: Avoid a crash with negative scales

See merge request GNOME/gtk!5236

Merge branch 'drop-buildable-get-id' into 'main'

builder: Stop using GtkBuildable.get_id

See merge request GNOME/gtk!5239

Merge branch 'stringlist-prop' into 'main'

stringlist: Add a construct-only strings property

See merge request GNOME/gtk!5238

Drop gtk_widget_buildable_parser_finished

No need to have this no-op implementation.
This was the only implementation of this vfunc
in GTK.

builder: Stop using GtkBuildable.get_id

Drop the only implementation of this vfunc,
and stop using it. It isn't needed, and the
only implementation did the same thing GtkBuilder
does anyway.

stringlist: Add a construct-only strings property

This enables creating string lists in ui files
without using custom markup.

Related: #5350

Merge branch 'center-box-props' into 'main'

centerbox: Add properties for child widgets

See merge request GNOME/gtk!5237

centerbox: Add properties for child widgets

This provides and alternative to add children
in ui files.

Related: #5350

gsk: Avoid a crash with negative scales

Don't crash when both scale_x and scale_y are -1.

A test is included.

g-i: add missing array annotation to AlertDialog.(s|g)et_buttons

Merge branch 'builder-parse-attributes' into 'main'

builder: Parse PangoAttrList values

See merge request GNOME/gtk!5228

Merge branch 'wip/carlosg/osk-activation' into 'main'

gtktext: Claim gesture more selectively

Closes #5351

See merge request GNOME/gtk!5230

Merge branch 'kjellahl/menubar' into 'main'

gtk_popover_menu_bar_set_menu_model(): Accept model == NULL

See merge request GNOME/gtk!5232

Merge branch 'shorter-im-context-ids' into 'main'

immmodule: Warn if GTK_IM_MODULE is bad

See merge request GNOME/gtk!5229

Merge branch 'wayland-foreign-v2' into 'main'

wayland: Also support the v2 of the importer

See merge request GNOME/gtk!5231

wayland: Also support the v2 of the importer

Commit 0ba8e16e1417fdb added support for the export
part of zxdg_foreign_v2, but neglected that the importer
has a v2 as well. Support it.

gtkpopovermenubar: Make model nullable

The description of gtk_popover_menu_bar_new_from_model() and
gtk_popover_menu_bar_set_menu_model() says that model is nullable.
Make it so.

gdksurface: Do not consider GDK_TOUCH_END/CANCEL as popup-dismiss worthy

GDK_TOUCH_END deserves the same treatment than GDK_BUTTON_RELEASE, since it's
subject to the same circumstances (popping up a menu on long press would be
immediately dismissed on release if we handled them there). Ideally, we would
want to match releases that we obtained a press for while grabbed, but as
the popup is also dismissed on GDK_BUTTON_PRESS/GDK_TOUCH_BEGIN, there's no
use for this tracking.

And GDK_TOUCH_CANCEL sounds weird as a reason to dismiss popups, just like
crossing events would.

Related: https://gitlab.gnome.org/GNOME/mutter/-/issues/2512

gtktext: Claim gesture more selectively

Even though button 1 (or touch down) presses do most often have
an effect in one way or another (starting drag, moving focus,
starting selection, ...), there is one situation that they do
immediately nothing: When clicking on the entry does not move
the text caret around. Dragging might start a selection, but
the entry did not do anything just yet, and an immediate
button/touch release should remain at "did nothing".

And that is precisely the hint that the Wayland IM context's click
gesture takes, clicks that do not scroll nor move the caret around,
having the GtkText not claim the gesture in that situation makes
the IM gesture able to do its thing without in-fighting.

This is typically not a problem when the GtkText is embedded in
another GtkEditable implementation (e.g. GtkEntry), since the
IM gesture is inactive and capturing from the parent widget, so
gets a pass that it otherwise doesn't get when both gestures are
in the same widget. This makes it work regardless of GtkText not
being a child of a composite widget, like NautilusQueryEditor
and AdwRowEntry.

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

Merge branch 'wip/fix-tiling-without-gtk-shell' into 'main'

wayland/surface: Set the tiling state from xdg-shell too

See merge request GNOME/gtk!5227

immodule: Accept 'simple' and 'none'

Everybody (including myself) gets this wrong,
so accept 'simple' and 'none' as shorthands
for the official IDs 'gtk-im-context-simple'
and 'gtk-im-context-none'.

immmodule: Warn if GTK_IM_MODULE is bad

If we don't find an immodule matching the
ids in the GTK_IM_MODULE envvar, print a
warning.

builder: Parse PangoAttrList values

Allow specifying PangoAttrList properties in the
serialized format supported by pango_attr_list_to_string().

Test included.

wayland/surface: Set the tiling state from xdg-shell too

We plumbed it from the gtk-shell configure event, but didn't for the
equivalent state passed via xdg_toplevel.configure.

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

gsk: Add an assertion to help static analysis

See merge request GNOME/gtk!5224

stringsorter: Fix one case of collation handling

When not ignoring case, and not collating, we were
returning sort keys that were already freed. Oops.

Pointed out by clang.

gdk: Fix possible memory errors

clang complained that we may end up jumping
to the cleanup code without initializing data
in the jpeg code. Always initialize data to
NULL to prevent that eventuality.

composetable: Fix a memory leak

Pointed out by clang.

gsk: Add an assertion to help static analysis

clang gets wild ideas about negative radii otherwise.

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

Fix spelling errors

See merge request GNOME/gtk!5223

Drop redundant glib version checks

We require GLib 2.72 now.

gdk: Clean up gdkprivate.h

Move apis to more specific headers.

gdk: Rename GDK_SURFACE_TEMP

We use this value only for drag surfaces, so call it that.

Fix spelling errors

These were pointed out by codespell.

Merge branch 'file-dialog-portals' into 'main'

file dialog: Use portals

See merge request GNOME/gtk!5221

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

gtk-demo: Add some keywords

See merge request GNOME/gtk!5220

filedialog: Use portals

We want this new API to be portal-first.
Make it so.

gtk-demo: Add folders to the clipboard demo

This is handy for testing a case where folder
dnd does not work with the file transfer portal.

See #5348