vulkan: Always render clip extents

Now that we have the full render nodes available, there is not much
benefit in fine-grained control over multiple rectangles.
In particular, it's causing pain with complex regions.

There might be a benefit in clipping to the region's rectangles in cases
like widget-factory where the whole diff is made up of the 2 rectangles
of spinner and the pulsing progress bar, but it needs a good heuristic
for where this is useful.

snapshot: Remove clip argument from gtk_snapshot_new()

It's not used anymore. And anybody who wants to have a clip on a
newly created snapshot can achieve that using gtk_snapshot_push_clip().

gtk: Remove gtk_widget_queue_draw_region()

... and gtk_widget_queue_draw_area().

They don't doi anything anymore.

widget: Stop tracking invalidations

Instead of calling gdk_surface_invalidate_region(), just
gdk_surface_queue_expose() and rely on the renderer computing the diff
from the previous rendering.

gtk: Remove debug updates

With the soon-to-arrive automatic updates (aka computing invalid region
from render nodes), this will no longer be necessary.

renderer: Track the previous node

... and diff the previous node with the current one to determine the
clip region.

This doubles the work necessary to track clip regions, but the following
commits will clean that up.

gsk: move begin/end_frame vfuncs into the renderers

gsk: Fold gsk_renderer_begin_draw_frame() into renderer

It doesn't need to be exported anymore.

As a side effect, the inspector no longer has any information about the
render region, so remove the code that was taking care of that.

widget: Reorder gtk_widget_render()

Now that we don't clip the created render nodes anymore, we don't have
to compute the clip region beforehand.

So snapshot the render nodes before initializing the renderer.

gsk: Implement gsk_render_node_diff()

This includes a copy of the diff(1) algorithm used by git diff by Davide
Libenzi.

It's used for the common case ofcontainer nodes having only very few
changes for the few nodes of child widgets that changed (like a button
lighting up when hilighted or a spinning spinner).

magnifier: Redo with GtkWidgetPaintable

gtk: Add GtkWidgetPaintable

A GtkWidgetPaintable is a paintable that observes a given GtkWidget and
renders that widget into a paintable.

widget: Cache the render node

This requires a bunch of refactorings:

1. Don't pass the current clip region to gtk_widget_snapshot()
   so we don't create full widget contents
3. Have a widget->priv->draw_needed that we invalidate on every
   queue_draw() call and set on every snapshot()
2. In queue_draw(), walk the widget chain to invalidate the
   render nodes of all parents

rendernode: Add gsk_render_node_diff()

... and gsk_render_node_can_diff(). Those are vfuncs to compute a region
containing all the pixels that differ between the two nodes.

This is just the plumbing that chains into node classes. No node
implements it yet.

font button: fix compiler warnings

A PangoLanguage is not a string.

Merge branch 'text-changed-delete-4' into 'master'

gtkentryaccessible: Fix text-changed::delete length

See merge request GNOME/gtk!100

Merge branch 'textview-delete-4' into 'master'

gtktextviewaccessible: update cursor position after text suppression

See merge request GNOME/gtk!102

gtktextviewaccessible: update cursor position after text suppression

delete_range_cb is set to be called before the text suppression done by
the gtktextlayout (otherwise it does not work properly). But at that
point the cursor position is not yet up to date.  We thus need to move
the accessibility cursor notification to after the actual text
suppression, by using another callback.

This fixes cursor position in brltty screen reading.

(cherry picked from commit fa6994d0331a219e85dce01007b2d6f3f7fd43c9)

gtkentryaccessible: Fix text-changed::delete length

The second parameter of the text-changed::delete event is to be the length,
not the end position.  This fixes spurious text removals in brltty
screen reading.

(cherry picked from commit 209f908a03a9d27508704425a57dcd533fab4c7d)

GLArea: Remove wrong transfer annotation on new()

Like other widgets, this returns a floating reference, so
(transfer full) is wrong. Just omit the annotation as others do,
thus implying (transfer none).

Close https://gitlab.gnome.org/GNOME/gtk/issues/156

imwayland: destroy objects only if it matches our manager object

Should fix one of the crash from #129.

font chooser: Remove some dead code

This is leftover code from already removed debug spew.

font button: Fix a crash

Since we are now storing the language as a PangoLanguage,
we must not free it anymore.

font chooser: Make the language property writable

This is meant as an input to the font chooser.
We don't want the user to select a language, but
rather have fonts presented as they would work for
the current language. Therefore, do away with the
lang/script combo on the tweak page.

gtk, demos: Fix builds without HarfBuzz and PangoFT

Really exclude the portions in the gtkfontchooserwidget.c that are built
when HarfBuzz and PangoFT2 are built, and update the Meson files to
exclude such sources as well from the main GTK SO/DLL and from the
gtk4-demo program.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

testsuite/gsk/test-render-nodes.c: Avoid VLA usage

Just #define N as 5, instead of using a const int, which is not enough
to justify that as non-VLA usage.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

[gtk|demos/gtk-demo]/language-names.c: Fix build on non-GCC/CLang

Remove g_auto*() usage from these sources and use the traditional
g_free(), as g_auto*() are GCCisms (or CLangisms).

Also, don't include unistd.h unconditionally and stop including
langinfo.h and dirent.h, since they seem to be unused.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

build: Fix linking demos on Visual Studio

The demos are now built as GUI programs, which will require the presence
of WinMain() on Visual Studio builds, unless we specify the entry point.

Pass the /entry:mainCRTStartup linker flag on Visual Studio builds for
the demo programs so that they can link properly.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

font chooser: Add examples for font features

For some font features, we can figure out affected
glyphs, and show before/after. For some others, we
hardcode typical sequences.

Still to do: figure out how to find ligatures and
show them.

widget-factory: Enable all details in the font chooser

We want to see this here.

Update POTFILES.in

W32: Link GTK to pangowin32

Needed for pango_win32_font_logfont() from gtkimcontextime.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Merge branch 'lrn/issue-147' into 'master'

Fix cursor handling in GTK4 on W32

Closes #147

See merge request GNOME/gtk!92

Fix some g-i annotation warnings

Enable variations and features for font explorer

font chooser: Align level value names

In the backport, I used the plural for variations, and that
is a better name, so use it here too.

font chooser: Align property defaults

Set the default font chooser level to the same value we use
in the backports, to preserve font chooser behavior.

Font chooser: Hide tweak button sometimes

When there is no chance of tweaking anything, don't
show a tweak button.

Defeat compiler optimization

We always want to call both update functions here, and
the compiler unhelpfully optimizes out the second call
if the first one returns TRUE.

GskRenderer: Remove outdated documentation comment

All render node trees are immutable

snapshot: Plug memory leak when collecting rounded clip nodes

gl renderer: Don't highlight cairo nodes as fallback

ProgressBar: Drop leftover "superimposed" mention

This was missed from the commit that dropped other mentions to the
previous ability to superimpose text.

https://bugzilla.gnome.org/show_bug.cgi?id=748784

appchooserwidget: search text using g_str_match_string()

so we match on diacritics (eg. accented letters) and also
search on text tokens. As a plus we also reduce code size.

https://bugzilla.gnome.org/show_bug.cgi?id=745128

togglebutton: Remove priv pointer

popover: Remove priv pointer

shaderbuilder: Insert newlines directly after defines

Instead of doing that in the loop in the last interation, we can just do
it after the loop.

gl renderer: Remover unused mask uniform

shader builder: Remove unused members

shader builder: Reuse GString for shader code

widget: Fix snapshot condition

Of course, we only need to snapshot if both width and height > 0,
otherwise the result is invisible.

entrybuffer: Remove priv pointer

menubar: Don't crash when cycling through menubars

Merge branch 'im-wayland-reset' into 'master'

wayland: Don't emit signals if nothing changed

See merge request GNOME/gtk!83

fishbowl: Make stats tracking work somewhat with broken X servers

Broken X servers being the ones that don't support NET_WM_TIMINGS (ie
every WM but mutter and XWayland).

demos: Improve frame rate tracking for fishbowl demo

We now properly use GdkFrameTimings and can therefor check for dropping
even a single frame in the history (of 16 frames or 1/4 of a second).
Once that happens, we immediately stop adding new items to the bowl.

A side effect is that the number of icons is now a *lot* smaller.

x11: Always request frame updates

This ensures that the frame clock gets updated with correct presentation
times even if nothing was drawn.

This is necessary for benchmarking but would also be relevant for videos
that want to sync to the frame clock but draw frames a lot less.

x11: Make the frame clock work in GL and Vulkan

GDK W32: Remember surface cursor, implicit surface grab

This commit ensures that each GdkSurface impl remembers the
cursor that GDK sets for it, and that this cursor is set
each time WM_SETCURSOR is called for that sufrace's HWND.
This is needed because W32, unlike X, has no per-window cursors -
the cursor on W32 is a global resource, and we need to keep track
of which cursor should be set when pointer is over which surface
ourselves (WM_SETCURSOR exists exactly for this reason).

This commit also makes GDK remember the surface that has an implicit
grab (since implicit grabs are gone from the upper levels of the toolkit),
and ensures that crossing events are correctly synthesized and the grab
is broken when surface focus changes. This fixes a bug where opening
a new window (by clicking something in some other, pre-existing window)
will make that new window not get any mouse input due to the fact
that the mouse-button-down event from that click caused an implicit
grab on the pre-existing window, and that grab was not released afterward.

GDK W32: Use the new cursor class

This makes all the code use the new cursor class instead of
raw HCURSOR handles.

GDK W32: New cursor class

Instead of now-unused GdkWin32Cursor class (a subclass of GdkCursor),
add a stand-alone GdkWin32HCursor class that is a wrapper around
HCURSOR handle.

On creation it's given a display instance, a HCURSOR handle and a boolean
that indicates whether the HCURSOR handle can or cannot be destroyed
(this depends on how the handle was obtained).
That information is stored in a hash table inside the GdkWin32Display
singleton, each entry of that table has reference count.
When the GdkWin32HCursor object is finalized, it reduces the reference
count on the table entry in the GdkWin32Display. When it's created,
it either adds such an entry or refs an existing one.
This way two pieces of code (or the same piece of code called
multiple times) that independently obtain the same HCURSOR from the OS
will get to different GdkWin32HCursor instances, but GdkWin32Display
will know that both use the same handle.

Once the reference count reaches 0 on the table entry, it is freed
and the handle (if destroyable) is put on the destruction list,
and an idle destruction function is queued.

If the same handle is once again registered for use before the
idle destructior is invoked (this happens, for example, when
an old cursor is destroyed and then replaced with a new one),
the handle gets removed from the destruction list.

The destructor just calls DestroyCursor() on each handle, calling
SetCursor(NULL) before doing that when the handle is in use.
This ensures that SetCursor(NULL) (which will cause cursor to disappear,
which is bad by itself, and which will also cause flickering if the
cursor is set to a non-NULL again shortly afterward)
is almost never called, unless GTK messes up and keeps using a cursor
beyond its lifetime.

This scheme also ensures that non-destructable cursors are not destroyed.

It's also possible to call _gdk_win32_display_hcursor_ref()
and _gdk_win32_display_hcursor_unref() manually instead of creating
GdkWin32HCursor objects, but that is not recommended.

themes: Make disabled Expander arrow look disabled

If GtkExpander:sensitive was FALSE, the arrow still got the normal fg
colour, which made it look clickable, in contrast to the adjacent label.
Fix this by adding selectors to catch the applicable :disabled states.

Note: Needing these may indicate an oops in generic styles elsewhere,
but I couldn’t see any, so let’s just get it looking right for now.

Close https://gitlab.gnome.org/GNOME/gtk/issues/146

emojichooser.ui: Add i18n context to all tooltips…

…since, unlike in gtk-3-22, it's not just Objects that already had that
here in the title labels. Thanks to Piotr for spotting this!

Update POTFILES.in

emojichooser.ui: Reuse headings as button tooltips

https://bugzilla.gnome.org/show_bug.cgi?id=792632

Widget: Fix doc of default ::keynav-failed handler

https://bugzilla.gnome.org/show_bug.cgi?id=793089

GDK W32: the .area member of the expose event is gone

The .area and .count members were removed in commit 5c7ee3a483f1fc59b60da0e8fafb33681bab507d

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: Adapt to the window->surface change

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: gdk_content_formats_builder_free{,_to_formats}

The function was renamed in commit 2cbe094b919442471b54685a0fef0d6f83ce44b3

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: Don't use gdk_threads_add_timeout_full()

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: don't use gdk_drag_find_surface() and gdk_drag_motion()

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: Adapt to event filter removal

Add a new W32 backend-specific message filtering mechanism.
Works roughly the same way old event filtering did, but without
events (events are GDK/X11 concept that never really made sense
on W32), so there's no functionality for 'altering' events being
emitted. If an event needs to be emitted in response to a message
do it yourself.

Implemented like this, it should give better performance than
if we were to use GLib signals for this, since W32 sends a LOT
of messages (unlike X11, which doesn't send events as often)
all the time, and invoking the signal machinery on *each* message
would probably be bad.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: adapt to GdkDragProtocol removal

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: Another massive clipboard and DnD update

Rename GdkWin32Selection to GdkWin32Clipdrop, since GdkSelection
is mostly gone, and the word "selection" does not reflect the
functionality of this object too well.

Clipboard is now handled by a separate thread, most of the code for
it now lives in gdkclipdrop-win32.c, gdkclipboard-win32.c just uses
clipdrop as a backend.

The DnD source part is also put into a thread.
The DnD target part does not spin the main loop, it just
emits a GDK event and returns a default value if it doesn't get a reply
by the time the event is processed.

Both clipboard and DnD use a new GOutputStream subclass to get data
from GTK and put it into a HGLOBAL.

GdkWin32DragContext is split into GdkWin32DragContext and GdkWin32DropContext,
anticipating a similar change that slated to happen to GdkDragContext.

OLE2 DnD protocol is now used by default, set GDK_WIN32_OLE2_DND envvar to 0
to make GDK use the old LOCAL and DROPFILES protocols.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

HighContrast: Honour .monospace set by GtkTextView

Copy the line that sets a monospace family from Adwaita (same place too)

noticed at https://gitlab.gnome.org/GNOME/gtk/issues/141#note_91610

ci: Manually install Meson from PyPI

The Fedora base image we use for our CI doesn't always keep Meson up to
date with our requirements, so it's better if we just install Meson via
Python's pip.

Add an option for building examples

Examples are not like demos; the latter are installed, and provide a
Flatpak manifest for CI pipelines and GNOME Builder. We should not be
using a single configure time option to gate building both.

gdkglcontext-win32.c: Fix window->surface changes

Rename the GdkSurface variables to surface, and make sure that we are
creating the context GObject correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

builder: Allow named objects to be used in paintable/pixbuf properties

Instead of erroring if an object by a name already exists, use that
object.

gsk: Remove gsk_renderer_create_cairo_surface()

It's not used anymore.

renderer: Get rid of unused member variable

frameclockidle: Rename header so it's private now

frameclockidle: Remove nonexisting functions from header

vulkan: Reserve 4 images in the swapchain by default

With the previous approach we would spend most of the time waiting for
the swapchain to be filled again because it seems the compositor takes
care of 2 images at once from time to time.

This is not visible in profiles because waiting for a frame is a
read/poll/whatever operation that does not take CPU. It's only
noticeable because the app becomes less responsive.

Rename gtkkineticscrolling.h to gtkkineticscrollingprivate.h

To match the naming scheme we use for all private header files.

expander: Update css docs

testgtk: don't set a focus adjustment on a box

That will make the focus adjustment code in gtkcontainer.c scroll to the
wrong coordinates.

Revert "container: Fix scroll adjustment coordinates"

This reverts commit 7351848c75e8d6efee154fe77cbd90056ed993b5.

expander: Remove priv pointer

build: Defer defining HAVE_PANGOFT and HAVE_HARFBUZZ

We forgot to account for the case where we lookup for HarfBuzz manually
under Visual Studio builds, so only set HAVE_HARFBUZZ (and thus
HAVE_PANGOFT, since PangoFT2 depends on HarfBuzz) after we did the
fallback check for HarfBuzz.

Also, check for hb.h instead of harfbuzz/hb.h to be inline with the
pkg-config case, as the sources also include the HarfBuzz header by
using #include <hb.h>, not #include <harfbuzz/hb.h>

https://bugzilla.gnome.org/show_bug.cgi?id=773299

gsk/gl: Include cairo.h consistently

Follow the other sources that include cairo.h, by just doing #include
<cairo.h>, not <cairo/cairo.h>.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

gtkimcontextime: fix to compile again

This makes the code compile again, though obviously there have been
some substantial changes in how IM contexts work, so it's possible
that IME IM context doesn't work now.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: remove unused client_message

This seems to be a leftover from API that was removed in
commit c332ac207ab4cac8f4c32aa85cdf6406157340af back in 2011.
The code, as it is now, does not even make sense.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: drop cursor-related GdkWin32Display functions

These functions went away in commit 77bab4e027c06dff7726d96fa4e477bc11b23835

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: remove the use of GDK_SURFACE_STATE

GDK_WINDOW_STATE (or, after rename, GDK_SURFACE_STATE) got
removed from GDK in commit 11a946df39f29b3cc0ce902c0c3b49abede1ead4

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: move GdkWin32MonitorDpiType to a different header

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Check for freetype2 version when PangoFT is used

Check for freetype2 version, because pangoft works with any version
(pangoft availability does not indicate that ft2 is new enough), unlike
GTK.

On Windows, since pangoft is optional, we check for the presence of
freetype2 .pc file first after finding that we have pangoft, and then
check for FT_Get_Var_Design_Coordinates() manually by looking for the
freetype headers and .lib first, and then looking for the presence of
that symbol, since freetype2's Visual Studio build system does not
generate a .pc file for us.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: _gdk_surface_invalidate_{for_expose,region}

https://bugzilla.gnome.org/show_bug.cgi?id=773299

GDK W32: drop the use of gdk_keymap_get_default()

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Only use gtk_print_backends_init() on UNIX

It's from gtkprintbackend.c, which is in gtk_unix_print_sources
source list and thus only available on os_unix only.

https://bugzilla.gnome.org/show_bug.cgi?id=773299