Title: [108334] trunk/Source/WebKit/gtk
Revision
108334
Author
[email protected]
Date
2012-02-21 03:46:07 -0800 (Tue, 21 Feb 2012)

Log Message

Fix one more GTK+ unit test after r108278.

Reviewed by Alejandro G. Castro.

* tests/testkeyevents.c:
(map_event_cb): No longer grab focus here.
(setup_keyevent_test): Grabbing focus here seems to be late enough in the process.

Modified Paths

Diff

Modified: trunk/Source/WebKit/gtk/ChangeLog (108333 => 108334)


--- trunk/Source/WebKit/gtk/ChangeLog	2012-02-21 11:34:24 UTC (rev 108333)
+++ trunk/Source/WebKit/gtk/ChangeLog	2012-02-21 11:46:07 UTC (rev 108334)
@@ -1,3 +1,13 @@
+2012-02-21  Philippe Normand  <[email protected]>
+
+        Fix one more GTK+ unit test after r108278.
+
+        Reviewed by Alejandro G. Castro.
+
+        * tests/testkeyevents.c:
+        (map_event_cb): No longer grab focus here.
+        (setup_keyevent_test): Grabbing focus here seems to be late enough in the process.
+
 2012-02-20  Martin Robinson  <[email protected]>
 
         Fix GTK+ unit tests after r108281.

Modified: trunk/Source/WebKit/gtk/tests/testkeyevents.c (108333 => 108334)


--- trunk/Source/WebKit/gtk/tests/testkeyevents.c	2012-02-21 11:34:24 UTC (rev 108333)
+++ trunk/Source/WebKit/gtk/tests/testkeyevents.c	2012-02-21 11:46:07 UTC (rev 108334)
@@ -118,7 +118,6 @@
 
 gboolean map_event_cb(GtkWidget *widget, GdkEvent* event, gpointer data)
 {
-    gtk_widget_grab_focus(widget);
     KeyEventFixture* fixture = (KeyEventFixture*)data;
     webkit_web_view_load_string(fixture->webView, fixture->info->page,
                                 "text/html", "utf-8", "file://");
@@ -131,6 +130,7 @@
     g_signal_connect(fixture->window, "map-event",
                      G_CALLBACK(map_event_cb), fixture);
 
+    gtk_widget_grab_focus(GTK_WIDGET(fixture->webView));
     gtk_widget_show(fixture->window);
     gtk_widget_show(GTK_WIDGET(fixture->webView));
     gtk_window_present(GTK_WINDOW(fixture->window));
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to