gtkentrycompletion: Properly remove controller from entry

When disconnecting the GtkEntry from the GtkEntryCompletion, we
must remove the controller in order to avoid dangling callbacks.

gtkentrycompletion: Move focus out handling to key controller

We can use the already present key controller to handle focus out.

gtkentrycompletion: Perform event handling on popup through controllers

Instead of a ::event signal handler.

gtkplacessidebar: Replace ::event signal handlers with gestures

Set up a gesture on the sidebar rows to detect pointer clicks on
it. The row DnD management has been moved to the row widget itself,
it makes more sense even if the drag is began from the sidebar widget.

gtkwindow: Move map/delete/configure event management to gtkmain

Toplevels are about the only widgets interested in these events,
just handle those without going through the capture/bubble handling
code.

gtkwindow: Simplify WM drag/resize handling

We still need a drag gesture both on front (capture) and back (bubble)
to handle dragging from both the GtkWindow widget and chrome in the
headerbar. But we can do it through 2 drag gestures, instead of special
event handling code.

gtkmenushell: Port to GtkGesture

We still need to poke the current event at places, but this is
better than the ::event vfunc.

gtkpopover: Fix key navigation

This has been broken since we switched key event delivery to follow
the same semantics than pointer/touch. There, GTK+ grabs will influence
the topmost widget during event delivery, rendering the toplevel
unable to handle key navigation. The toplevel must handle those key
events in an explicit manner then.

We don't render the keyboard focus rectangle yet, but I assume that's
something else.

gtkpopover: Port to GtkEventController/GtkGesture

Use GtkEventControllerKey and GtkGestureMultiPress to replace key/button
event handling.

gtkwindow: Handle focus in/out through key controller

One less use of ::event in this widget, now mostly left to
map/delete/configure, those puny events.

gtkspinbutton: Use GtkEventControllerKey

And drop the ::event vfunc implementation with it.

gtkfontchooser: Use GtkEventControllerKey

Use an event controller on GtkFontChooserDialog, a nice side effect
is that we can use gtk_event_controller_key_forward() and
gtk_search_entry_set_key_capture_widget() instead of passing events
around for dialog search.

gtkfilechooserwidget: Drop usage of ::event vmethod

Use a key controller set up in the UI file instead.

gtkfilechooserentry: Use GtkEntry key controller for focus-out handling

Expose the GtkEntry key controller in private API, so we don't have to
create yet another one just to handle focus-out.

gtkentry: Use gestures for entry icons

Instead of doing all handling manually in the ::event vfunc,
set up drag/multipress gestures on icon images, and implement
emission of ::icon-press/release and DnD there.

As a side effect, the GdkEvent field in ::icon-press/release
signals has been dropped. Callers that might be interested on it
may still use gtk_get_current_event*().

gtkbutton: remove keyboard device GTK+ grab

This isn't really necessary, if keyboard focus forcibly goes somewhere
else we will get ::grab-notify, which is sufficient to deactivate the
button again.

gtkbutton: Use key controller for button (de)activation through keybindings

And stop using GtkWidget::event for good.

x11: Fix dnd action handling

We were mistakenly assigning an Atom to a flags value.

EmojiCompletion: Avoid another un/signed warning

The cherry-pick missed this, I guess.

EmojiCompletion: Avoid un/signed compare warnings

(A) Use gsize to match the result of g_variant_n_children
(B) Use guint for n_matches, like the struct (and all other n_matches)

themes: Remove leftover bit of .linked workaround

Now that .linked acts like the .path-bar already did, there is no point
in applying the same styles again (with higher specificity).

container: Remove SPECIAL_CONTAINER hack

As discussed in !129

Closes !129

stackswitcher: Remove icon-size property

This ought to be controlled via css these days.

infobar: Remove priv pointer

Remove some unnecessary snapshot implementations

fixed: Remove priv pointer

listbox: Use widget insert API instead of fiddling with CSS nodes

button: Remove ICON_SIZE_BUTTON mention in docs

The icon will have the size given by css, not whatever
GTK_ICON_SIZE_BUTTON was.

adjustment: Only emit ::changed from dispatch_properties_changed

adjustment: Remove adjustment_changed_stamp

testsuite: Print tested property name in notify test

adjustment: Use public setters in set_property

themes: Regenerate CSS for recent commits

e.g. the box.linked one, which didn't update the compiled CSS.

HC: Avoid excessive selectors from prev commit

The new rule does not need to affect all those other pseudoclasses. I
just put it in the wrong scope.

HC: Avoid same BG/FG colors in flat treeview entry

Selected rows in tree views in HighContrast have a background colour the
same or nearly as the normal text colour, so we cannot let entries in
such rows have transparent backgrounds, or the text inside the entry
becomes nearly or totally impossible to see.

Dodge this by giving entry.flat inside treeview and with :focus the
$base_color, which is different from the text & so lets that be seen.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/125

searchenginetracker: Don't leak the hits' GFiles

https://gitlab.gnome.org/GNOME/gtk/merge_requests/206

Merge branch 'issue1084' into 'master'

application: Complete the startup sequence before quitting remote instances

Closes #979

See merge request GNOME/gtk!156

Merge branch 'master' into 'issue1084'

# Conflicts:
#   gdk/gdk.c

window: Remove priv pointer

pathbar: Remove unused define

docs: Remove a duplicate line in gdk4-sections.txt

This was causing another docs build break.

docs: fix a typo

Invalid syntax in gdk4-sections.txt was causing docs builds to fail.

Merge branch '1160-crash-clearing-gtkentry-icon-tooltip-master' into 'master'

Resolve "Crash clearing GtkEntry icon tooltip"

Closes #1160

See merge request GNOME/gtk!205

win32: Fix typo

This should fix gtk-doc erroring out.

Revert "Add aligned allocator functions to GSK"

This reverts commit 8e74eb382f0c617b7b65a6850952d6acc9fd3018.

This code is not necessary. It worked around a bug in graphene where
graphene was requiring stricter alignment than glib allocators could
guarantee.

Revert "Use aligned allocators for GtkSnapshot"

This reverts commit c02bc22cc575470aab870599f2781c3781fb75e6.

This code is not necessary.
The bug causing this problem ws prsent in the graphene library.

rendernodepaintable: Fix rendering position

When the given size had an offset, we were moving that offset in the
wrong direction.

dnd: Use a GtkPicture in gtk_dnd_set_icon_paintable()

We don't want to get the paintable scaled down to icon size.

Also set can-shrink = FALSE to replicate GTK3 behavior with size
requests.

snapshot: Compute correct size in gtk_snapshot_to_paintable()

Also, do actually respect the passed in size argument if it isn't NULL.

Fixes text being cut off inside DND icons.

dnd: gdk_drag_action_is_unique() should return a boolean

a11y/entry: Fix copy-pasteo re 2ndary icon tooltip

The else case was wrongly resetting the accessible description on the
primary icon, which might not exist and can therefore cause a crash.

https://gitlab.gnome.org/GNOME/gtk/issues/1160

Merge branch 'wip/carlosg/issue-1159' into 'master'

Fix issue 1159

Closes #1159

See merge request GNOME/gtk!202

Update POTFILES.in

wayland: Use shm_open(SHM_ANON) on FreeBSD

This functionality is similar to Linux's memfd. It creates anonymous shared memory without touching the filesystem, which allows it to work in Capsicum capability mode (sandbox).

gtkgesture: Check claimed status after event delivery

The claimed status check should happen after ::end is emitted,
as the gesture may deny the sequence that much late. In this
case the event should keep propagating.

https://gitlab.gnome.org/GNOME/gtk/issues/1159

Closes: #1159

gtkwidget: Avoid doubly coordinate conversion when emulating press

We are poking again into the event propagation machinery, which
expects events in toplevel coordinates. Since we can't fetch the
original event back at this point, translate the coordinates
back to the toplevel so the emulated press ends up in the right
place.

https://gitlab.gnome.org/GNOME/gtk/issues/1159

Closes: #1159

x11: Set background as transparent as possible

Instead of making the background black, make it transparent black for
RGBA visuals and set a None background otherwise.

widget: Remove time argument from drag_data_get() vfunc

dnd: Remove unused time arguments

dnd: Remove GdkDragContext.dest_surface

The variable is now unused

GDK W32: Remove gdk_win32_surface_foreign_new_for_display

That function is now unused.

GDK W32: Adapt to dest_surface removal, misc changes

* There's no GdkDragContext->dest_surface anymore.
  Add dest_window field to GdkWin32DragContext,
  and use that instead.
* Remove unused function prototypes
* Add more comments
* Rename variables and fields from 'window' to 'surface'
  where appropriate
* Fix header indentation a bit
* Try to ensure that uninitialized/unknown handle variables
  and fields are set to INVALID_HANDLE_VALUE instead of NULL,
  as there may be cases where NULL is a valid handle value.

dnd: Remove GdkDragContext.is_source

All drag contexts are sources these days, the other ones are GdkDrop
now.

x11: Remove gdk_x11_surface_foreign_new_for_display()

People who want to use foreign windows should use X directly.

x11: Store the XID instead of a GdkSurface

This might be foreign Windows and we don't want to create surfaces for
those.

Also, stop using GdkDragContext.dest_surface, that variable is meant to
go away.

x11: Simplify test

Now that we have gdk_drag_context_get_display(), use it.

x11: xdnd_send_event() always returns TRUE

Make it return nothing instead and delete the code that handled the
FALSE case.

x11: Don't multiply X values by the scale factor

This is fallout from a too eager change in
bdb442be21471a91820219347b1345ed7b84d034.

dnd: Make the base Drag/Drop types abstract

This way, they'll be ignored by the testsuite.

And we don't want to instantiate them anyway, the backends have
their own subclasses.

dnd: GdkDragContext is no longer a GdkDrop subclass

This includes a bunch of header cleanup

dnd: Remove gdk_drag_status() and gdk_drag_finish()

Those functions are unused and have been replaced by their equivalents
gdk_drop_status() and gdk_drop_finish().

surface: Kill event masks

In particular, this patch removes:
  gdk_surface_get_events()
  gdk_surface_set_events()
  gdk_surface_get_device_events()
  gdk_surface_set_device_events()

Event masks so far still exist for grabs.

x11: Store the source surface as an XID

Don't create a foreign GDK surface, just store the XID.

With this, we can avoid GDK APIs that we want to get rid of and just
use the X counterpart.

x11: Split drag and drop contexts

While doing so, turn the drop context into a GdkDrop subclass and no
longer pretend to be a GdkDragContext.

x11: Handle events on drag side differently

Instead of using the filters that the drop side uses, handle events in
the event filter installed by the DragContext.

x11: Change the way local Drop => Drag is shortcut

This is one step further towards untangling drag and drop parts of X11
DND.

x11: Remove unused argument

The propagate argument is always FALSE, so just use FALSE everywhere.

x11: switch a bunch of variable types

GdkDragContext => GdkDrop

This is all in preparation of separation of the drag and drop.

Also, don't check for GDK_DRAG_PROTO_XDND anymore - it's the only
possible value for the protocol on the target side.

broadway: Remove unused variable

dnd: Remove gdk_drag_context_get_dest_surface()

... and gdk_drag_context_get_source_surface().

In the backends, use direct access to the variables instead.

dnd: Implement gtk_drag_get_source_widget() differently

Instead of keeping a list of source contexts, just use the current
drag's info.

x11: Replace the hardcoded DND filter

Use the new method of connecting to the xevent signal instead.

Also, don't consume the xevent, there might be other code listening for
it. And we don't use PropertyNotify in the generic code path anymore, so
it'll just be ignored there.

GDK W32: Remove unused files

GDK W32: Adapt to GdkDrop and GdkDragContext changes

* Remove clipdrop->dnd_target_state, it's not used anymore
* Remove non-functioning _gdk_dropfiles_store(), store dropfiles
  list in GdkWin32Drop instead
* Fix multiple comment typos
* Fix _gdk_win32_get_clipboard_format_name_as_interned_mimetype() to
  leave names that look like mime/types alone
* Refactor _gdk_win32_add_w32format_to_pairs() to populate
  GdkContentFormatsBuilder directly, instead of making a GList
* Rename context -> drag (still using GdkDragContext type,
  but [almost?] all variables and comments say "drag" now)
* Rename GdkDropContext -> GdkDrop
* Rename some parameter names for clarity
* Rewrite local protocol to look more like OLE2 protocol
  instead of mirroring the structure of the X11 API.
* Add handle_events field to GdkWin32DragContext,
  to shut off event handling (temporary fix until GTK is patched up)
* Remove _gdk_win32_drag_context_find() - the drag object is stored
  in GdkDrop instead. Use _gdk_win32_find_drag_for_dest_surface()
  to get it initially.
* Remove target_ctx_for_window, droptarget context is stored
  in the surface instead.
* Call gdk_drag_context_set_cursor() just like wayland backend does
  (slightly broken for now)
* Clean up the action choosing code (filter source actions by using
  keyboard state, pass that to GTK, get all actions supported by GTK in
  response, match them up with filtered source actions, return the
  result, falling back to COPY in case of multiple actions)
* Check drag_win32->protocol instead of the use_ole2_dnd variable where
  possible
* Remove protocol checks from functions that are only used by the local
  protocol
* Use event state to manufacture the keyboard state for WM_MOUSEMOVE
* Change function names printed by GDK_NOTE to name the actual
  functions, not their theoretical generic GDK stack ancestors
* Consistently use drag_win32 and drop_win32 variables instead of a mix
  of that and win32_drag/win32_drop
* Return FALSE from button handler to ensure that GTK gets the button
  event to break implicit grab
* Emit leave event on failed idroptarget_drop() calls

dnd: Add read_value() and read_text() functions to GdkDrop

These just copy what GdkClipboard does.

drop: Implement shortcut for local DND

Now that we have the drop->drag, we can use the same shortcut we use in
the clipboard code to shortcut DND operations.

x11: Pass the drag source when creating a drop context

x11: Find dest drops differently

Instead of looking at the list of contexts, just look at the current
drop context. There is only one, after all.

Then remove the is_source argument from gdk_drag_context_find().

wayland: Store the current drag operation

Use it to detect local drags when creating drop objects.

dnd: Move GdkDragAction to gdktypes.h

That way, both gdkdrag.h and gdkdrop.h can use it without one having to
include the other.

gdk: Remove leftover type from autocleanup

wayland: Get rid of GdkWaylandSelection

Move data source handling into the DND code instead.

wayland: Split drop context into GdkWaylandDrop

GdkWaylandDrop no longer inherits from GdkDragContext now.

wayland: Remove a bunch of unused functions

events: Make GdkEventDND have a GdkDrop member

... instead of a GdkDragContext.

x11: Refactor DND filter

The filters now return TRUE/FALSE and no longer a GdkFilterReturn. They
also don't conform to the GdkFilterFunc typedef anymore but instead take
the arguments that they need.

x11: Use new DND event emitters

win32: Use new DND event emitters

wayland: Use new DND event emitters

drop: Add private API for event emission