widget: Split out gtk_widget_ensure_allocate_on_children()

allocate() should not be calling into ensure_allocate(), they do a similar job.
In the end, the code does the same work, but it should be easier to follow now.

widget: Simplify size allocation

Don't call ensure_allocate if we've just
done size_allocate. This makes criticals
from reshowing popovers go away.

Merge branch 'wip/carlosg/reuse-search-engine' into 'main'

gtkfilechooser: Keep GtkSearchEngine around for longer

See merge request GNOME/gtk!5617

gtkfilechooser: Keep GtkSearchEngine around for longer

Currently the GtkSearchEngine is torn down every time the search
is stopped, which also means between typed characters. This
prevents any of the optimizations that the GtkSearchEngine can
do in the long run.

Let the GtkSearchEngine stay around for longer, and only be
disposed after search is cancelled, the filechooser moves
onto a different mode than search, or is otherwise unmapped/disposed.

While at it, remove an unused struct field.

Merge branch 'wip/carlosg/recursive-file-search' into 'main'

searchengine: Improve performance for recursive search

Closes #4133

See merge request GNOME/gtk!5611

searchengine: Do not fight filechooser search mode sorting

Provide the hits in an order that is more easily consumed by the
filechooser, and less jarring when populating.

searchengine: Bail out on the first character

Again on massive filesystems, the very first character
is likely to bring a likewise massive amount of search
results that we need to maybe query info for, then create
icons and widgets for. While it's impressive we can do
that, it's also expensive and likely pointless, for the
first character.

Typing a second character is however very likely to
considerably reduce the amount of items to categorize and
show. So start actually searching from there.

Testing on a filesystem with 1434099 files indexed, trying 5
semi-random 1 character searches (n, h, t, i, o) returns on
average 168K items (min. 78771, max. 331471), trying 5
semi-random 2 character searches (no, he, th, in, on)
returns on average 34K items (min. 11133, max. 94961),
which is a more approachable set.

Doing this is enough that typing on a filechooser search
entry feels completely fluid.

searchengine: Ensure some GFileInfo fields

The search provider should make it sure there are some
specific GFileInfo fields set. Fix the mimetype extraction
from the query, and use that to fill in the missing gaps
the best we can.

searchengine: Drop non-recursive search

Recursive search seems to be the hardcoded default, go with it.

searchengine: Populate filesystem model in an idle

When starting a search over a very populated filesystem, it
is possible that typing the first chars will return a too
high number of results. Even though iterating through the
cursor is in itself very fast, extracting the GIO information
from those many files at once is not going to be as fast.

In order to increase interactivity (i.e. not make things
possibly sluggish) iterate the cursor in an idle function
and add search results to the filechooser model little by little.

If the user keeps typing (as it is likely will happen), there
will be better chances to cancel and proceed to the next
query timely. If not, the results will be there soon enough.

Merge branch 'check_null_gfile_location' into 'main'

filechooser: Fix memory leaks

See merge request GNOME/gtk!5610

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

build: Handle introspection a bit better

See merge request GNOME/gtk!5614

filechooser: Fix memory leaks

The returned strings from `file_chooser_get_location()` were never freed.

build: Handle introspection a bit better

Error out if introspection is requested,
but g-ir-scanner isn't found.

And if introspection isn't explicitly disabled
but is required for building the docs, build it.

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

filechooser: Handle pathless files

See merge request GNOME/gtk!5613

Merge branch 'expose_checkable_state' into 'main'

a11y: When an accessible has GTK_STATE_CHECKED, set its ATSPI_STATE_CHECKABLE as well

See merge request GNOME/gtk!5602

filechooser: Handle pathless files

g_file_get_path() can return NULL. Make our
getter for file locations handle that case.

Related: !5610

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

wayland: Don't crash during DND with Cairo renderer

See merge request GNOME/gtk!5612

wayland: Don't crash during DND with Cairo renderer

Attaching buffers with offsets is forbidden now and nobody
checked the Cairo renderer.

text: Don't crash when somebody drops NULL

Merge branch 'wroy-main-patch-03255' into 'main'

MSBuild: Fix gnome.compile_resources current source directory

See merge request GNOME/gtk!5609

Apply suggestion to use '/' over join_paths

searchengine: Improve performance for recursive search

As fancy as property paths are, recursive resolution of files
to a location increases the big O complexity enough that it's
not a great option on large homedirs with many indexed files.

Ensure the files are from the right location through a URI
prefix match, which does hits an index. This may dramatically
improve performance on large indexed trees.

Testing this query in an isolated testcase with a total
1434099 indexed files shows that it can run more than 1500 times
per second in this computer (an average of 15200 queries in
several 10 second runs), which presumably is a tad faster than
anyone can type.

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

Fix compile_resources present source directory

In certain scenarios, address the issue where gnome.compile_resources
fails to transmit the present source directory. This is most notably
visible with MSBuild.

Merge branch 'unbreak-combobox' into 'main'

combobox: Avoid extra queue_resize()

Closes #5644

See merge request GNOME/gtk!5608

Updated Danish translation

combobox: Avoid extra queue_resize()

width-request already ensures it's above the minimum width, so avoid an
extra queue_resize() when setting size request to (-1, -1).

This is the same way as GtkDropDown works. This also unbreaks
GtkComboBox after the recent allocation fix in
75a417e33708dab2bdb2f784a8952e085a12bf03.

Incidentally, this also makes GtkComboBox actually resize its popup as
intended (that was broken before).

I don't think this is ultimately the final fix, sometimes I still get
allocation warnings. But the proper fix will probably involve changing
some more allocation machinery around popovers. This is good enough for
now.

Merge branch 'window-signals' into 'main'

window: Disconnect the ::compute-size handler

See merge request GNOME/gtk!5606

window: Disconnect the ::compute-size handler

All the other signal handlers are connected in
realize and disconnected in unrealize, but the
::compute-size handler was forgotten.

This was notices in !5597.

Merge branch 'expose_landmark_role' into 'main'

a11y: Expose GTK_ACCESSIBLE_ROLE_LANDMARK to AtSPI2

See merge request GNOME/gtk!5601

a11y: When an accessible has GTK_STATE_CHECKED, set its ATSPI_STATE_CHECKABLE as well

This allows, for example, checkable list itemss.

a11y: Expose GTK_ACCESSIBLE_ROLE_LANDMARK to AtSPI2

Update Czech translation

Merge branch 'wip/exalm/template' into 'main'

widget: Use the correct template in dispose_template()

See merge request GNOME/gtk!5600

Merge branch 'main' into 'main'

removed duplicate function call

See merge request GNOME/gtk!5545

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

macos: Fix type func generation

See merge request GNOME/gtk!5596

Merge branch 'wip/fix-wl-destruct-order' into 'main'

wayland: Destroy xdg_surface after role

See merge request GNOME/gtk!5598

widget: Use the correct template in dispose_template()

In derivable classes, the widget's class can be different from the one
dispose_template() was called for, which can lead to failing the
template != NULL check at best, undefined behavior at worst.

Since we already pass the correct GType into the function, just use that
instead.

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

inspector: Make really sure we don't inspect ourselves

See merge request GNOME/gtk!5599

inspector: Make really sure we don't inspect ourselves

The problem here is that new windows appear in the list before the
window's dispay gets set and we don't update the filter when the
display changes (would need watches support for the filtermodel).

So add this somewhat hacky method.

wayland: Destroy xdg_surface after role

The split-up of gdksurface-wayland.c introduced a protocol violation
when it didn't make sure xdg_surface was destroyed after the role
objects (xdg_popup / xdg_toplevel). Fix that.

Fixes: 2a463baed0 ("wayland: Rearrange the surface code")

macos: Fix type func generation

We were checking for gdk_quartz even though
the sysbols are now all called gdk_macos. Oops.

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

cssnode: Don't crash the inspector

See merge request GNOME/gtk!5595

testsuite: Add tests for the unknown enums

See commit 40e7a265a7ccb1385a3af0931c591201b0ce063a

testsuite: Add empty test for new node

cssnode: Don't crash the inspector

Don't misinform the observing listmodel that CSS nodes were removed that
weren't actually removed, but just moved. Otherwise the observer would
think it has run out of items when it really hasn't.

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

meson: Mention testsuite in summary

See merge request GNOME/gtk!5594

meson: Mention testsuite in summary

This was broken out from the 'tests' toggle,
so list it separately.

Merge branch 'expose_error_message_relation' into 'main'

a11y: Expose GTK_ACCESSIBLE_RELATION_ERROR_MESSAGE to AtSPI2

See merge request GNOME/gtk!5588

Merge branch 'wip/otte/listview2' into 'main'

listview: GtkListTile

See merge request GNOME/gtk!5584

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

ci: Disable the testsuite in flatpak builds

See merge request GNOME/gtk!5593

ci: Disable the testsuite in flatpak builds

No need to build the testsuite when we are
just producing flatpaks.

listbase: Remove nonexisting function from header

gridview: Add a filler tile for empty space

That stupid space in the bottom right when n_items isn't a multiple of
n_columns needs its own tile, or we'll get errors about not finding a
tile.

So make one.

columnview: Clear the sorter first thing in dispose

Otherwise, when removing the columns, each column will trigger a
sorter::changed signal emission.

And because sorters are often still connected to a sortlistmodel, we
can't skip that emission and need to do it.
But we only need to do it once.

listview: Handle emptying of views

The previous check does not longer work.

When a model gets all items deleted, there will still be existing tiles
until the next time garbage collection is run.
So do that before checking if the list is empty.

listview: Remove an unused member variable

It's only used in the allocate function, so make it a local varaible
there.

listview: Simplify a vfunc

Instead of making it 2 vfuncs for getting horizontal and vertical area,
make it one vfunc to get the area.

Also rewrite the implementations to use the tile's area instead of
trying to deduce things with fancy math.

listview: Simplify allocation

With the Tile changes, a lot of stuff does no longer need to be
duplicated between listview and gridview. Move it to ListBase instead.

gridview: Remove an unused member variable

It's only used during size_allocate(), so make it a local variable
there.

listitemmanager: Add a split vfunc and use it

Instead of randomly changing tiles, the listitemmanager gains a split
vfunc that listview and gridview implement so they can keep their tile
areas intact. The listitemmanager will now conform to these rules:

1. Never delete a tile.
   This ensures that all areas stay intact.

2. Never change the n_items of a tile other than setting them to 0.
   This causes "empty" areas to appear, but listview/gridview can
   easily check for them by checking for tile->n_items == 0.
   gtk_list_tile_gc() will get rid of them.

3. Adding items always creates new tiles that are added with empty area.
   That way they don't interrupt any existing machinery until the next
   allocation.

4. Adding/removing widgets has no effect on areas
   This is useful in particular when scrolling where new widgets are
   moving between tiles. When the manager moves the widgets, it may
   split some areas, but will not remove any existing tiles, so the
   whole area stays intact and the list can deal with further scroll
   events before an allocation.

This improve the situation for #3334

listview: Add gc'ing

This is in preparation for the following reorg of the listitemmanager,
it should not have any effect now.

listitemmanager: Remove unused functionality

Both ListView and GridView use GtkListTile now, so no need anymore for
custom machinery.

gridview: Redo tile management

Instead of the custom size property, use the new tile size.

Also introduce the ability to split tiles, so that gridview can split a
layout that would look like (question mark denoting cells without a
widget, which in this case would be a single tile)

█ █ █ ? ?
? ? ? ? ?
? ? ? ? ?
? ? ?

into 3 rectangular tiles like so:

█ █ █ A A
B B B B B
B B B B B
C C C

This of course also means we need to be able to merge those tiles again
when cells got added/deleted or the gridview was resized. For that job,
gtk_list_tile_gc() exists now, which removes tiles without items and
merges adjacent tiles without widgets.

listview: Get rid of ListRow and its height

Use the tiles' area instead.

list: Allow storing size in the ListTile

... and use it to handle ListView allocations.

Nothing spectacular, just proof of concept.

The code introduces the idea that every tile stores its area (others
would call it "allocation", but I avoided that because tiles aren't
widgets). This should allow moving lots of code into gtklistbase.c and
not require special handling inside ListView and GridView.

And that in turn hopefully makes it easier to add more features (like
sections and so on.)

list: Make GtkListTile more prominent

* Instead of using a gpointer to refer to it, use the GtkListTile type.

* Use gtk_list_tile_get_foo() instead of
  gtk_list_item_manager_get_tile_foo() naming.

list: Rename GtkListItemManagerItem => GtkListTile

That's a good description for what the job of those elements is:
They're a tile in the lists scrollable area.

rendernodepaintable: Allow the node to be NULL

This can happen when creating paintables from GtkSnapshot when nothing
was drawn.

Update Catalan translation

Post-release version bump

4.10.0

Merge branch 'ebassi/file-info-attributes' into 'main'

Check for attributes being available before querying them

See merge request GNOME/gtk!5592

Merge branch 'wip/otte/dont-scale-or-it-breaks' into 'main'

Add a test for lots of texture scaling and fix the fallout

See merge request GNOME/gtk!5509

Check for attributes being available before querying them

GLib 2.75 started checking if a GFileInfo was created with the attribute
we're querying, instead of failing silently and leaving us in an
inconsistent state.

Turns out that GtkFileChooserWidget, GtkFileSystemModel, and GtkPathBar
trip the newly introduced check.

Merge branch 'wip/chergert/fix-action-activation-crash' into 'main'

inspector: be defensive against out parameters

See merge request GNOME/gtk!5591

Merge branch 'ebassi/stack-page-fix' into 'main'

a11y: Fix GtkStackPage first accessible child

See merge request GNOME/gtk!5590

Update Russian translation

Update Polish translation

testsuite: Add a test for stripes

The GL renderer was creating sripes for nodes that were scaled in
particular ways, probably due to rounding errors.

This testsuite focuses on one of those stripes to make sure they are
gone.

a11y: Use weak references for GtkATContext cached accessibles

The accessible objects already own the GtkATContext, let's avoid a
reference cycle.

stack: Don't recreate at contexts in dispose

This is the same protection we have in
GtkWidgetAccessible.

a11y: Plug reference leaks

The gtk_accessible_get_at_context() getter is now transfer full, which
means we need to drop the reference when getting the GtkATContext.

inspector: be defensive against out parameters

Set initial state to NULL so that we don't risk accessing an unset out
parameter.

Fixes a crash when activating certain actions.

a11y: Simplify bookkeeping while iterating accessible children

a11y: Fix GtkStackPage first accessible child

The first accessible child is the child widget of the GtkStackPage, not
the first child of the widget.

testsuite: Add a test for large scale nodes

This test fails if we naively create fullscale
intermediate offscreens. This was fixed in the
previous commits.

This tests the fixes in 22ba6b1f33dcbcab8d (for
cairo) and 3a0152b65fd75f631b76 (for GL).

gl: Respect clip wehn drawing scale nodes

Use the same approach and only create an offscreen
that is big enough for the clipped part of the scaled
texture.

If the clipped part is still too large for a single
texture, we give up and just render the texture without
filters (using the regular texture rendering code path
which supports slicing).

The following commit will add the texture-scale-magnify-10000x
test which fails without this fix.

rendernode: Respect clip when drawing scale nodes

Scale nodes can use large scale factors and we don't want to create
insanely huge Cairo surfaces.

A subsequent commit will add the texture-scale-magnify-10000x
test which fails without this fix.

rendernode: Clarify some Cairo stuff

Split out a function to make it more obvious what's going on.

rendernode: Don't do unnecessary stuff

Cairo surfaces are created transparent.

And even if they weren't, overdrawing with transparency wouldn't erase
what's in the surface because it's a no-op.

It would require CAIRO_OPERATOR_CLEAR or CAIRO_OPERATOR_SOURCE.

Fixes for gdk_memory_texture_new_subtexture checks

There were several mistakes here.

Update Korean translation

a11y: Expose GTK_ACCESSIBLE_RELATION_ERROR_MESSAGE to AtSPI2

Also, update the AtSPI2 relations to match the 1.26 version of AtSPI2.

Merge branch 'ebassi/accessible-transfer-full' into 'main'

Mark Accessible getters as transfer full

Closes #5615

See merge request GNOME/gtk!5586

Mark Accessible getters as transfer full

GtkAccessible implementations in C can get away returning objects just
by shuffling pointers around, but higher level languages prefer using
full ownership transfer in virtual functions.

Fixes: #5615

Merge branch 'expose_details_relation' into 'main'

a11y: Expose GTK_ACCESSIBLE_RELATION_DETAILS to at-spi2

See merge request GNOME/gtk!5580

Merge branch 'fix_section_role' into 'main'

a11y: Fix mapping of GTK_ACCESSIBLE_ROLE_PRESENTATION

See merge request GNOME/gtk!5576

Merge branch 'ebassi/atcontext-dispose' into 'main'

a11y: Clear the accessible tree in GtkATContext

See merge request GNOME/gtk!5587

Revert "stack: clear accessible parent when removing child"

This reverts commit 40d4441fd85742837d85d8509dc89355d39a1cdc.

The accessible parent of the child widget in a GtkStackPage is cleared
when the GtkATContext gets disposed, so we don't need to unset it
ourselves. This also avoids a temporary vivification of the GtkATContext
during dispose.