glcontext: assert all contexts set the version on realize()

Now that all contexts do that, insist that they keep doing it.

And because they keep doing it, we can support querying the GL version
from gdk_gl_context_get_version() without requiring the context to be
made current.

mac: Try all different OpenGL profiles

.. and pick the best one that is supported.

Also make sure that the resulting context has at least the desired
version, otherwise bail.

glcontext: Compute matching version the simple way

Do it all in one function instead of requiring two different ones.

win32: Make sure highest possible GL version is created

Mirror EGL here.

win32: Pass the version properly through the creation stack

We were using major/minor instead of GdkGLVersion.

And we were resetting back to 0 and ignoring the required min version
which we should not do.

glx: Make sure highest possible GL version is created

Mirror EGL here.

egl: Make sure highest possible GL version is created

The EGL spec states:

    The context returned must be the specified version, or a later
    version which is backwards compatible with that version.
    Even if a later version is returned, the specified version
    must correspond to a defined version of the client API.

GTK has so far been relying on EGL implementations returning a
later version, because that is what Mesa does.
But ANGLE does not do that and only provides the minimum version, which
means Windows EGL has been forced to use a lower EGL version for no
reason.

So fix this and try versions in order from highest to lowest.

gdk: Move GdkGLAPI enum into gdkenums.h

glcontext: Use GdkGLVersion elsewhere

... and add a convenience API to generate GL versions from strings to
make the gdk_gl_context_check() API nicer.

gdk: Introduce GdkGLVersion

... and use it.

Makes the code simpler.

win32: Remove an outdated check

We require GL 3.0, so checking for less than 2.0 makes no sense anymore.

Merge branch 'wip/otte/treelistmodel-madness' into 'main'

treelistmodel: Be safer during collapsing

See merge request GNOME/gtk!5875

treelistmodel: Delay notifies from TreeListRow

Don't notify during destruction, notify afterwards.
This way we don't call into user code from a half-destructed node.

Note that this changes the order in which those notifies happen when
collapsing a large tree: From parent node before child nodes to child
nodes before parent node.

No actual use case for this, just thought it would be safer.

treelistmodel: Be safer during collapsing

While we are collapsing a subtree, some signal handlers may not be
disconnected while we are doing this. By adding this check and not
giving those nodes no longer access to the model, we can stop it from
modifying it while we are trying to collapse stuff.

Fixes some crashes in gnome-builder.

Merge branch 'wip/otte/surface-surgery' into 'main'

refactor internal surface API

See merge request GNOME/gtk!5846

3to4: Fix we said we replace doubles with doubles,

when what we are replacing are ints!

display: Remove ::create_surface() vfunc

Instead, have a toplevel_type and popup_type in GdkDisplay and
call g_object_new() with those types.

wayland: Set default title in toplevel_init()

macos: Add surface from ::constructed()

gdk: Remove GDK_SURFACE_DRAG enum member

I want to remove the whole enum, but this value was used all over the
place. So removing it first ensures I didn't forget anything.

wayland: Create drag surface directly

... instead of going through create_surface().

wayland: Remove struct member

It's unused.

x11: Add private gdk_x11_drag_surface_new()

... and use it.

x11: Set frame clock in ::constructed

x11: Move window construction to ::constructed()

broadway: Create gdk_broadway_drag_surface_new()

... and use it.

win32: Create gdk_win32_draw_surface_new()

... and use it.

macos: Set frame clock in ::constructed

broadway: Set frame clock in ::constructed

... instead of passing it to g_object_new().

broadway: Move surface init code into ::constructed()

win32: Set frame clock in constructed()

... instead of passing it as construct argument.

wayland: Construct the frame clock in the surface

... instead of passing it to g_object_new().

surface: Make gdk_surface_set_frame_clock() available

... to backends.

That way, frame clocks can be constructed by the backends' surface
implementations and dont need to be passed in as construct arguments.

Also add an assertion that they are indeed constructed.

macos: Move native window creation to ::constructed()

That way, it doesn't need a specific init function.

Also chain up last, so that the generic initialization code in
GdkSurface::constructed can access a fully initialized macos surface.

macos: Don't pass sizes to macos_surface_new()

They're 0, 0, 100, 100 always, so just use those values everywhere.

macos: Create drag surface directly

Do not go via macos_surface_new().

macos: Move construction stuff into ::constructed

macos: toplevels have no parent

win32: Move surface init code into ::constructed()

win32: Abort on error

Returning NULL from a function that must not return NULL is not a good
idea.

win32: Remove outdated debug messages

They are not needed anymore as they print stuff that's no decided in
that function anymore.

win32: Move toplevel signal into the toplevel class

broadway: Pass parent as a construct argument

x11: Get rid of sizes when constructing surfaces

Just call XCreateWindow with 0, 0, 1, 1 size.

x11: Remove unused variable

wayland: Move surface initialization into constructed()

That way, it doesn't ned a specific init function.

Also chain up last, so that the generic initialization code can access a
fully initialized wayland surface.

wayland: Move toplevel-specific code into the toplevel

The display->toplevels tracking belongs to GdkWAylandToplevel.

display: Remove x/y/w/h from create_surface()

Merge branch 'application-signal-cleanup' into 'main'

application: Clean up signal handlers

See merge request GNOME/gtk!5872

application: Clean up signal handlers

This is the right thing to do and might help for #5775.

Merge branch 'win32-warnings' into 'main'

Fix some win32 warnings

See merge request GNOME/gtk!5868

Merge branch 'pathbar-webdav-root' into 'main'

pathbar: Handle webdav where is the root is a path

Closes #2866

See merge request GNOME/gtk!5830

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

gsk: Don't misuse bitwise operators

See merge request GNOME/gtk!5869

Merge branch 'modelbutton-no-click-outside' into 'main'

gtkmodelbutton: Ignore releases outside of the button

Closes #5760

See merge request GNOME/gtk!5852

Update Galician translation

(cherry picked from commit 7470bc01f25123ed57e36eaf4e2a5c98203a03da)

gtkmodelbutton: Ignore releases outside of the button

This is also how regular buttons behave. Otherwise releasing on a
different menu item would register a click on the item that was
originally pressed. In these cases it is better to not register a click
at all.

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

gsk: Don't misuse bitwise operators

We should use && for booleans, not &=.

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

gsk: Don't misuse bitwise operators

See merge request GNOME/gtk!5867

gtk/win32: fix usage of deprecated function

[106/939] Compiling C object gtk/libgtk.a.p/gtkimcontextime.c.obj
../gtk/gtkimcontextime.c: In function 'gtk_im_context_ime_set_preedit_font':
../gtk/gtkimcontextime.c:780:3: warning: 'gtk_widget_get_style_context' is deprecated [-Wdeprecated-declarations]
  780 |   font_desc = gtk_css_style_get_pango_font (gtk_style_context_lookup_style (gtk_widget_get_style_context (context_ime->client_widget)));

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix defined but not used warnings

[30/1038] Compiling C object gdk/win32/libgdk-win32.a.p/gdkmain-win32.c.obj
../gdk/win32/gdkmain-win32.c:146:1: warning: 'gdk_win32_finalize_ole' defined but not used [-Wunused-function]
  146 | gdk_win32_finalize_ole (void)
      | ^~~~~~~~~~~~~~~~~~~~~~
../gdk/win32/gdkmain-win32.c:113:1: warning: 'gdk_win32_finalize_com' defined but not used [-Wunused-function]
  113 | gdk_win32_finalize_com (void)

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix `hr` set but not used

A number of warnings are produced:

[23/1038] Compiling C object gdk/win32/libgdk-win32.a.p/gdkinput-dmanipulation.c.obj
../gdk/win32/gdkinput-dmanipulation.c: In function 'reset_viewport':
../gdk/win32/gdkinput-dmanipulation.c:354:11: warning: variable 'hr' set but not used [-Wunused-but-set-variable]
  354 |   HRESULT hr;
      |           ^~

Try to do something sensible instead.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix g_string_free warning

../gdk/win32/gdkclipdrop-win32.c: In function 'transmute_cf_shell_id_list_to_text_uri_list':
C:/msys64/ucrt64/include/glib-2.0/glib/gstring.h:72:5: warning: ignoring return value of 'g_string_free_and_steal' declared with attribute 'warn_unused_result' [-Wunused-result]

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gdk/win32: fix GDK_NOTE redefined warning

In file included from ../gdk/win32/gdkdrag-win32.c:201:
../gdk/win32/gdkprivate-win32.h:45: warning: "GDK_NOTE" redefined
   45 | #define GDK_NOTE(type,action)                             \
      |
../gdk/win32/gdkdrag-win32.c:40: note: this is the location of the previous definition
   40 | #define GDK_NOTE(a,b)

Fixes: bc159207bd0 ("gdk: Drop old debug macros")
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gsk: Don't misuse bitwise operators

We should use || for booleans, not |=.

Merge branch 'label_get_active_uri' into 'main'

Fix hovered link URI not returned by gtk_label_get_current_uri

See merge request GNOME/gtk!5864

Merge branch 'icontheme-has-icon-consistency' into 'main'

icontheme: make has_(g)icon consistent with lookup

Closes #5709

See merge request GNOME/gtk!5865

Merge branch 'null-display' into 'main'

gdkdisplaymanager: Add missing nullable to display name

See merge request GNOME/gtk!5866

gdkdisplaymanager: Add missing nullable to display name

icontheme: make has_(g)icon consistent with lookup

Make `gtk_icon_theme_has_icon` and `gtk_icon_theme_has_gicon` also
consider unthemed icons. This makes their behavior consistent with the
actual (documented) lookup behavior.

Fixes: #5709 and makes the workaround in nautilus@b643a00b obsolete

Fix hovered link URI not returned by gtk_label_get_current_uri

This is needed for a query-tooltip handler, as mentioned in the
documentation, when there has been a hover timeout.

Maintain the previous behaviour when the link is clicked and follow the
existing documentation regarding selectable labels.

A notify::cursor handler can now also be used to retrieve the URI of the
link under the cursor.

Merge branch 'ebassi/button-can-shrink' into 'main'

Add GtkButton:can-shrink

See merge request GNOME/gtk!5552

Add GtkMenuButton:can-shrink

Map the GtkMenuButton property to the underlying GtkButton widget.

Update Turkish translation

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

surface: Clean up drawing code

See merge request GNOME/gtk!5861

surface: reformat function

Make the function follow usual coding conventions.

And while doing that, remove duplicate functionality.

surface: Refactor code

Move the early exit conditions to the top and turn them into early exits
instead of nesting if statements.

surface: Fold function into its only caller

No other changes

surface: Stop maintaining an unused linked list

wat?

surface: Remove in_update tracking

It's 2023, we use frame clocks now and don't have nested surface drawing
anymore.

surface: Stop tracking the active update area

It's unused.

Update Russian translation

Update Polish translation

Merge branch 'gtkfilesystemmodel-emit-items-changed-on-modifying-attributes' into 'main'

filesystemmodel: Emit items-changed when modifying attributes

Closes #5758

See merge request GNOME/gtk!5853

filesystemmodel: Emit items-changed when modifying attributes

Merge branch 'wip/corey/file-chooser-signal' into 'main'

Port filechooserwidget to GtkSignalListItemFactory

See merge request GNOME/gtk!5858

filechoosercell: Drop show-time property

Now that we are using GtkSignalListItemFactory, the convoluted show-time
property is no longer necessary.

filechooserwidget: Port to GtkSignalListItemFactory

BuilderListItemFactory isn't quite suited for our purposes, primarily
because you can't pass user data to BuilderListItemFactory.  Because
we can't get the data we are using a workaround to get the
GtkFileChooserWidget ancestory, which used to work, but with the
recent list view changes no longer doesn't.  Use GtkSignalListItemFactory
with the GtkFileChooserWidget as the user data.

Revert "filechooser: Set date and time after cell is a child of filechooserwidget"

This reverts commit 2a70093a309481cf633086a5e8f0c8bc6d2c89f0.

This fix was incomplete, the actual fix is to use signal factory.

Revert "filechoosercell: Store type_format in filechoosercell"

This reverts commit dd407dab000dc4de7d078c6270c93d303a1c18c6.

This fix was incomplete, the actual fix is to use signal factory.

Update Portuguese translation

(cherry picked from commit 5bc3284dd873ae13ef5aa883cb324b4979c9762c)

Update Hebrew translation

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

testsuite: Be less verbose in accessor-apis test

Closes #5763

See merge request GNOME/gtk!5851

listbase: Don't warn on scroll in empty list

Empty lists can still be scrolled if the scroll happens in the same
frame as the emptying of the list.

Related: #5763

adjustment: sanity-check values when setting them

It's not enough to sanitize values when starting an animation, as the
adjustment can reconfigure itself while the animation runs.
So as a simple way to handle this, we sanitize every value right before
setting it, too.

In the future we might also want to look at sanitizing start/end values
of the animation.

Fixes #5763

adjustment: Split out a function

Merge branch 'dark_theme_fix_hover_toolbar_buttons' into 'main'

theme: fix 'hover' and 'check' colors for buttons under .toolbar

See merge request GNOME/gtk!5844

Merge branch 'dark_theme_increase_selection_color' into 'main'

Increase contrast for text entry selection color on dark theme

See merge request GNOME/gtk!5843

testsuite: Allow accessors in interfaces

There are a lot of cases where properties are implemented in classes but
the getters for these exist in an interface that class implements.

A common Example is g_list_model_get_n_items() being the getter for
GtkWhateverListModel::n-items.
But also property implementations that don't use override_property()
(usually because they have a different default) are handled by this.

testsuite: Be less verbose in accessor-apis test

Only print the tried names when --verbose is passed to the test.

Update Turkish translation