Attachment 371835 Details for Bug 198750 – Patch

[patch] Patch

bug-198750-20190611113503.patch (text/plain), 3.63 KB, created by Ludovico de Nittis on 2019-06-11 02:35:04 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ludovico de Nittis

Created: 2019-06-11 02:35:04 PDT

Size: 3.63 KB

patch

obsolete

>Subversion Revision: 246303
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 8ac22ea5266483e510a3f98146279ec52f13b2d4..a0f6186a075da7d2b0d1d67707fb5a4a957f45dd 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2019-06-11  Ludovico de Nittis  <ludovico.denittis@collabora.com>
>+
>+        [GTK] Replace gdk_screen_get_monitor_geometry and gdk_screen_get_monitor_workarea
>+        https://bugs.webkit.org/show_bug.cgi?id=198750
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Since GTK 3.22 gdk_screen_get_monitor_geometry and
>+        gdk_screen_get_monitor_workarea has been deprecated. 
>+
>+        No behavior change.
>+
>+        * platform/gtk/PlatformScreenGtk.cpp:
>+        (WebCore::screenRect):
>+        (WebCore::screenAvailableRect):
>+
> 2019-06-10  Fujii Hironori  <Hironori.Fujii@sony.com>
> 
>         [WinCairo][MediaFoundation] Remove the unused video window
>diff --git a/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp b/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp
>index 1129b4bb944f6269219f71be406fc1e9365c8d33..998f3a647ab085b2203f4bbee14d5acdeb201fa4 100644
>--- a/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp
>+++ b/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp
>@@ -150,10 +150,12 @@ void setScreenDPIObserverHandler(Function<void()>&& handler, void* context)
>     }
> }
> 
>+#if !GTK_CHECK_VERSION(3, 22, 0)
> static GdkScreen* getScreen(GtkWidget* widget)
> {
>     return gtk_widget_has_screen(widget) ? gtk_widget_get_screen(widget) : gdk_screen_get_default();
> }
>+#endif
> 
> FloatRect screenRect(Widget* widget)
> {
>@@ -161,14 +163,24 @@ FloatRect screenRect(Widget* widget)
>     if (container)
>         container = getToplevel(container);
> 
>+    GdkRectangle geometry;
>+#if GTK_CHECK_VERSION(3, 22, 0)
>+    GdkDisplay* display = container ? gtk_widget_get_display(container) : gdk_display_get_default();
>+    if (!display)
>+        return FloatRect();
>+
>+    GdkMonitor* monitor = container ? gdk_display_get_monitor_at_window(display, gtk_widget_get_window(container)) : gdk_display_get_monitor(display, 0);
>+
>+    gdk_monitor_get_geometry(monitor, &geometry);
>+#else
>     GdkScreen* screen = container ? getScreen(container) : gdk_screen_get_default();
>     if (!screen)
>         return FloatRect();
> 
>     gint monitor = container ? gdk_screen_get_monitor_at_window(screen, gtk_widget_get_window(container)) : 0;
> 
>-    GdkRectangle geometry;
>     gdk_screen_get_monitor_geometry(screen, monitor, &geometry);
>+#endif // !GTK_CHECK_VERSION(3, 22, 0)
> 
>     return FloatRect(geometry.x, geometry.y, geometry.width, geometry.height);
> }
>@@ -179,14 +191,25 @@ FloatRect screenAvailableRect(Widget* widget)
>     if (container && !gtk_widget_get_realized(container))
>         return screenRect(widget);
> 
>+    GdkRectangle workArea;
>+#if GTK_CHECK_VERSION(3, 22, 0)
>+    GdkDisplay* display = container ? gtk_widget_get_display(container) : gdk_display_get_default();
>+    if (!display)
>+        return FloatRect();
>+
>+    GdkMonitor* monitor = container ? gdk_display_get_monitor_at_window(display, gtk_widget_get_window(container)) : gdk_display_get_monitor(display, 0);
>+
>+    gdk_monitor_get_workarea(monitor, &workArea);
>+    GdkMonitor* monitor = 0;
>+#else
>     GdkScreen* screen = container ? getScreen(container) : gdk_screen_get_default();
>     if (!screen)
>         return FloatRect();
> 
>     gint monitor = container ? gdk_screen_get_monitor_at_window(screen, gtk_widget_get_window(container)) : 0;
> 
>-    GdkRectangle workArea;
>     gdk_screen_get_monitor_workarea(screen, monitor, &workArea);
>+#endif // !GTK_CHECK_VERSION(3, 22, 0)
> 
>     return FloatRect(workArea.x, workArea.y, workArea.width, workArea.height);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 198750: 371835 | 371836