Title: [103067] trunk/Source/WebKit2
Revision
103067
Author
carlo...@webkit.org
Date
2011-12-16 06:41:41 -0800 (Fri, 16 Dec 2011)

Log Message

[GTK] Window frame should be 0x0 when the toplevel window is not visible
https://bugs.webkit.org/show_bug.cgi?id=74709

Reviewed by Gustavo Noronha Silva.

* UIProcess/API/gtk/WebKitUIClient.cpp:
(getWindowFrame): Check also whether the toplevel is visible
before getting its size and position.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (103066 => 103067)


--- trunk/Source/WebKit2/ChangeLog	2011-12-16 14:19:49 UTC (rev 103066)
+++ trunk/Source/WebKit2/ChangeLog	2011-12-16 14:41:41 UTC (rev 103067)
@@ -1,3 +1,14 @@
+2011-12-16  Carlos Garcia Campos  <cgar...@igalia.com>
+
+        [GTK] Window frame should be 0x0 when the toplevel window is not visible
+        https://bugs.webkit.org/show_bug.cgi?id=74709
+
+        Reviewed by Gustavo Noronha Silva.
+
+        * UIProcess/API/gtk/WebKitUIClient.cpp:
+        (getWindowFrame): Check also whether the toplevel is visible
+        before getting its size and position.
+
 2011-12-16  Michael Bruning  <michael.brun...@nokia.com>
 
         [qt][wk2] Viewport info panel shows wrong current scale

Modified: trunk/Source/WebKit2/UIProcess/API/gtk/WebKitUIClient.cpp (103066 => 103067)


--- trunk/Source/WebKit2/UIProcess/API/gtk/WebKitUIClient.cpp	2011-12-16 14:19:49 UTC (rev 103066)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/WebKitUIClient.cpp	2011-12-16 14:41:41 UTC (rev 103067)
@@ -112,7 +112,7 @@
 {
     GdkRectangle geometry = { 0, 0, 0, 0 };
     GtkWidget* window = gtk_widget_get_toplevel(toImpl(page)->viewWidget());
-    if (gtk_widget_is_toplevel(window)) {
+    if (gtk_widget_is_toplevel(window) && gtk_widget_get_visible(window)) {
         gtk_window_get_position(GTK_WINDOW(window), &geometry.x, &geometry.y);
         gtk_window_get_size(GTK_WINDOW(window), &geometry.width, &geometry.height);
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to