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

drop: Add GdkDrop:surface property

This replaces gdk_drag_context_get_dest_surface().

display: Don't wake up the main loop anymore

This is not needed because GTK must be run in the main thread these days,
which is the same one that runs the main loop. So when this function is
called, the main loop is awake.

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

x11: Don't sneakily allocate memory in initializers

dnd: Add gdk_event_get_drop()

And remove gdk_event_get_drag_context().

All GTK code now only uses GdkDrop for dropping.

wayland: Fix drop operations for new semantics

dnd: Remove GdkDragContext::commit_drag_status() vfunc

The idea behind the vfunc doesn't work as status updates can happen
asynchronously.

A better solution needs to be found.

dnd: Make drag-motion and drag-drop signals use GdkDrop

dnd: Make drag-data-received use a GdkDrop

drop: Add gdk_drop_get_drag()

For now, it'll always return NULL. We'll fix that later.

widget: Make GtkWidgetClass::drag_leave() take a GdkDrop

Drag Contexts are on their way out!

dnd: Add gdk_drop_finish()

and move the vfunc into GdkDrop.

dnd: Get rid of gtk_drag_finish()

It's just a wrapper around gdk_drag_finish(), so use that one instead.

dnd: Rename gdk_drop_finish() => gdk_drag_finish()

This is a simple rename, so that we can reintroduce gdk_drop_finish()
with the GdkDrop object.

dnd: Move the gdk_drag_status() function

It's now gdk_drop_status().

Also clarify the intended semantics.

dnd: Add gdk_drop_get_actions()

This uses the new method without GDK_ACTION_ASK:

Either it is a single action (queryable via gdk_drag_action_is_unique())
or it is not and then the drop target has to make a decision
(potentially by asking someone).

dnd: Make actions a private member variable

Use a setter in the backends.

x11: Remove unused variables

wayland: Set correct variable

This got messed up in commit 7f8a8f221d60bc5da23602e5e024eb91f8b0f758.

dnd: Add gdk_drag_action_is_unique()

This will be necessary once we remove the ASK action.

dnd: Remove GDK_ACTION_DEFAULT and GDK_ACTION_PRIVATE

They're unused and nobody knows what they're supposed to men anyway.

drop: Move gdk_drop_read_async() to GdkDrop class

dnd: Add GdkDrop base class for GdkDragContext

The ultimate goal of this patch series is to split GdkDragContext into
GdkDrop + GdkDrag classes for the destination and source side of a dnd
operation.

The refactoring is meant to work something like this:
1. Introduce GdkDrop as a base class
2. Make all drop related code (like GdkEvent) use GdkDrop instead of
   GdkDragContext. Move/duplicate APIs to allow that.
3. Port all drop contexts in the backends from GdkDragContext to GdkDrop
4. Delete all APIs in GdkDragContext that aren't needed anymore.
5. Make GdkDragContext no longer a GdkDrop subclass
6. Rename GdkDragContext to GdkDrag

Merge branch 'lrn/misc-gtk4-fixes' into 'master'

Misc GTK4 fixes

See merge request GNOME/gtk!195

Merge branch 'win32-filechooser-fix-file-list-order' into 'master'

win32 file chooser: add missing g_slist_reverse

See merge request GNOME/gtk!199

themes: Don't handle box.linked special

We don't reverse the css nodes anymore in RTL layouts.

filechooserbutton: Remove priv pointer

GtkStyleContext: Remove the frame clock

It's unused within GtkStyleContext.

dnd: Fix up a comment

The below function does not take a GtkIconHelper anymore.

colorscale: Remove priv pointer

fontbutton: Remove priv pointer

radiobutton: remove priv pointer

picture: Fix up some docs

Use nullable instead of allow-none consistently, remove trailing
whitespace and fix some c&p problems.

drawingarea: Clarify some comments

messagedialog: Remove priv pointer

paned: Allocation x/y are always 0

The allocation position we get passed to size_allocate is always 0/0
these days.

paned: Remove handle_pos member

Query the handle bounds on demand instead.

paned: Chain up in snapshot

No need to snapshot all child widgets ourselves, the implementation in
GtkWidget can just do it for us.

paned: Remove unnecessary local variable

We can just pass the given allocation on.

paned: Don't unnecessarily redraw handle in size_allocate

We size_allocate it after all, which will redraw it.

paned: Store GParamSpecs

So we can use the more efficient g_object_notify_by_pspec everywhere.

Remove gtk_css_node_reverse_children

Not needed anymore.

toolbar: Stop reordering css nodes based on text direction

headerbar: Stop reordering css nodes depending on text direction

notebook: Stop reversing tabs based on text direction

builder: Fix g-i annotations of _get_translation_domain

The return value can be null and is (transfer none).

builder: Use TRUE/FALSE for error return values

Instead of guint and 0/1.

builder: Remove priv pointer

box: Don't reorder children based on text direction

Make :first-child always be the first child, i.e. the leftest one in LTR
and the rightest one in RTL.

paned: Don't reorder css nodes based on text direction

center box: Don't reorder css nodes in RTL

So widget order matches css order. We will do the same thing with GtkBox
eventually.

scrolledwindow: Remove priv pointer

Merge branch 'window-activate-grab-4-again' into 'master'

gdk: activate surface on keyboard grabs

Closes #85

See merge request GNOME/gtk!174

win32 file chooser: add missing g_slist_reverse

the list is prepended to so we need to reverse the list
before returning

gdk: activate surface on keyboard grabs

In 01455399e83a ("gdk: do not deactivate surface on keyboard grabs"), we
made gdk avoid deactivating surfaces when another application takes a
keyboard grab, by using has_focus_window instead of has_focus. That however
broke activating surfaces when the gdk application acquired a grab itself,
in which case has_focus_window is false but has_focus is true.

We thus actually need to use both: surfaces should be activated either
because we have normal keyboard focus, or because we grabbed the keyboard.

This also renames HAS_FOCUS to APPEARS_FOCUSED to better reflect its
role.

Fixes #85

Update Polish translation

GDK W32: don't unref window iconlist textures

We do not own these textures, nor do we ref them, so it's inappropriate
to unref them.

W32: Use correct hinstance values

According to the old new thing[0], we should use the instance handle
of the GDK/GTK DLL when registering GDK-specific types in the system.
Using the instance handle for the whole application in these circumstances
is not an error, but can potentially clash with the types registered
by the application itself.

Also, extract window class icons from the GDK/GTK DLL, not from the
application executable.

[0]: https://blogs.msdn.microsoft.com/oldnewthing/20050418-59/?p=35873

Pass the correct data to gdk_content_register_serializer()

The argument is eventually passed to g_conv(), so it should
be the charset, not the mime/type. Without this change the
contentype converter will fail to convert UTF-8 strings to, say,
CP-1251 later on.

W32: don't put a string literal into gtk_libdir

In all other instances gtk_libdir is a dynamically-allocated string,
so dup the literal in this case as well.

Add forward declaration for DllMain() in gtkwin32.c

Use const for media type in devmode_to_settings()

Add foward declarations to gtk_print_operation_run_without_dialog

search bar example: remove unused function

center box: Remove snapshot implementation

gl renderer: call glViewport directly

button: Remove measure implementation

This is already done by GtkBin.

combobox: Remove priv pointer

Merge branch 'wip/lantw/fix-gtkdoc-build-without-wayland' into 'master'

docs: Fix gtk-doc build when wayland is disabled

See merge request GNOME/gtk!193

docs: Fix gtk-doc build when wayland is disabled

Unconditionally putting 'gdkwayland_inc' in src_dir argument of gtkdoc
call tells gtkdoc-scan to scan source files in a non-existent build
directory, gdk/wayland. To avoid causing build failure when a specific
backend is disabled, we should include directories conditionally.

Merge branch 'design_by' into 'master'

aboutbox: use a more fitting 'design by" role

Closes #1153

See merge request GNOME/gtk!192