Fix wxSetCursorEvent handling

For compatibility with wxMSW, send event up the parent chain.
Properly handle setting the cursor for a wxSetCursorEvent, and don't overwrite the window cursor.
see #15801


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/branches/WX_3_0_BRANCH@75821 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/gtk/window.cpp

@@ -1528,6 +1528,31 @@ gtk_window_button_release_callback( GtkWidget *WXUNUSED(widget),
     return ret;
 }
 
+//-----------------------------------------------------------------------------
+
+WX_DECLARE_VOIDPTR_HASH_MAP(bool, wxVoidPtrBoolMap);
+static wxVoidPtrBoolMap gs_needCursorResetMap;
+
+static const wxCursor* gs_overrideCursor;
+
+static void SendSetCursorEvent(wxWindow* win, int x, int y)
+{
+    wxSetCursorEvent event(x, y);
+    wxWindow* w = win;
+    do {
+        if (w->GTKProcessEvent(event))
+        {
+            gs_overrideCursor = &event.GetCursor();
+            win->GTKUpdateCursor();
+            gs_needCursorResetMap[win] = true;
+            return;
+        }
+        w = w->GetParent();
+    } while (w);
+    if (gs_needCursorResetMap[win])
+        win->GTKUpdateCursor();
+}
+
 //-----------------------------------------------------------------------------
 // "motion_notify_event"
 //-----------------------------------------------------------------------------
@@ -1595,13 +1620,7 @@ gtk_window_motion_notify_callback( GtkWidget * WXUNUSED(widget),
     }
 
     if ( !g_captureWindow )
-    {
+        SendSetCursorEvent(win, event.m_x, event.m_y);
-        wxSetCursorEvent cevent( event.m_x, event.m_y );
-        if (win->GTKProcessEvent( cevent ))
-        {
-            win->SetCursor( cevent.GetCursor() );
-        }
-    }
 
     bool ret = win->GTKProcessEvent(event);
 
@@ -1809,13 +1828,7 @@ gtk_window_enter_callback( GtkWidget*,
     InitMouseEvent(win, event, gdk_event);
 
     if ( !g_captureWindow )
-    {
+        SendSetCursorEvent(win, event.m_x, event.m_y);
-        wxSetCursorEvent cevent( event.m_x, event.m_y );
-        if (win->GTKProcessEvent( cevent ))
-        {
-            win->SetCursor( cevent.GetCursor() );
-        }
-    }
 
     return win->GTKProcessEvent(event);
 }
@@ -1831,6 +1844,9 @@ gtk_window_leave_callback( GtkWidget*,
 {
     wxCOMMON_CALLBACK_PROLOGUE(gdk_event, win);
 
+    if (gs_needCursorResetMap[win])
+        win->GTKUpdateCursor();
+
     // Event was emitted after an ungrab
     if (gdk_event->mode != GDK_CROSSING_NORMAL) return FALSE;
 
@@ -2465,6 +2481,8 @@ wxWindowGTK::~wxWindowGTK()
         g_object_unref(m_styleProvider);
 #endif
 
+    gs_needCursorResetMap.erase(this);
+
     if (m_widget)
     {
         // Note that gtk_widget_destroy() does not destroy the widget, it just
@@ -3671,6 +3689,8 @@ bool wxWindowGTK::SetCursor( const wxCursor &cursor )
 
 void wxWindowGTK::GTKUpdateCursor(bool isBusyOrGlobalCursor, bool isRealize)
 {
+    gs_needCursorResetMap[this] = false;
+
     if (m_widget == NULL || !gtk_widget_get_realized(m_widget))
         return;
 
@@ -3688,7 +3708,11 @@ void wxWindowGTK::GTKUpdateCursor(bool isBusyOrGlobalCursor, bool isRealize)
     }
     GdkCursor* cursor = NULL;
     if (!isBusyOrGlobalCursor)
-        cursor = m_cursor.GetCursor();
+    {
+        const wxCursor* overrideCursor = gs_overrideCursor;
+        gs_overrideCursor = NULL;
+        cursor = (overrideCursor ? *overrideCursor : m_cursor).GetCursor();
+    }
 
     GdkWindow* window = NULL;
     if (cursor || isBusyOrGlobalCursor || !isRealize)