2007-12-04 Luca Bruno <lethalman88@gmail.com>
authoralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Dec 2007 14:12:16 +0000 (14:12 +0000)
committeralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Dec 2007 14:12:16 +0000 (14:12 +0000)
        Reviewed by Alp Toker.

        Fix a regression in key press propagation in r28386.

        Fix indentation (was off by two spaces).

        * WebView/webkitwebview.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28388 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/gtk/ChangeLog
WebKit/gtk/WebView/webkitwebview.cpp

index 94275bb292759798718362ef080c573611315cde..b75164c5b0cde123d0184c1ce933c977e54b05da 100644 (file)
@@ -1,3 +1,13 @@
+2007-12-04  Luca Bruno  <lethalman88@gmail.com>
+
+        Reviewed by Alp Toker.
+
+        Fix a regression in key press propagation in r28386.
+
+        Fix indentation (was off by two spaces).
+
+        * WebView/webkitwebview.cpp:
+
 2007-12-04  Luca Bruno  <lethalman88@gmail.com>
 
         Reviewed by Alp Toker.
index 7feb2efe4e21858a8c6b71fe3d9afa3a9a931c09..7972d1591749c3591facb71b5b7febc239bc712b 100644 (file)
@@ -108,31 +108,33 @@ static gboolean webkit_web_view_key_event(GtkWidget* widget, GdkEventKey* event)
     Frame* frame = core(getFrameFromView(WEBKIT_WEB_VIEW(widget)));
     PlatformKeyboardEvent keyboardEvent(event);
 
-    if (!frame->eventHandler()->keyEvent(keyboardEvent) && event->type == GDK_KEY_PRESS)
-      {
-          FrameView* view = frame->view();
-
-          /* FIXME: at the very least we should be using the same code than the
-             Windows port here, but our ScrollView file diverges enough to make
-             that impossible. A short term solution would be to unify ScrollViewWin
-             and ScrollViewGtk. Long-term ScrollView and FrameView should be
-             unified and used everywhere for scrollbars */
-
-          switch (event->keyval) {
-          case GDK_Down:
-              view->scrollBy(0, LINE_STEP);
-              return TRUE;
-          case GDK_Up:
-              view->scrollBy(0, -LINE_STEP);
-              return TRUE;
-          case GDK_Right:
-              view->scrollBy(LINE_STEP, 0);
-              return TRUE;
-          case GDK_Left:
-              view->scrollBy(-LINE_STEP, 0);
-              return TRUE;
-          }
-      }
+    if (frame->eventHandler()->keyEvent(keyboardEvent))
+        return TRUE;
+
+    if (event->type == GDK_KEY_PRESS) {
+        FrameView* view = frame->view();
+
+        /* FIXME: at the very least we should be using the same code than the
+           Windows port here, but our ScrollView file diverges enough to make
+           that impossible. A short term solution would be to unify ScrollViewWin
+           and ScrollViewGtk. Long-term ScrollView and FrameView should be
+           unified and used everywhere for scrollbars */
+
+        switch (event->keyval) {
+        case GDK_Down:
+            view->scrollBy(0, LINE_STEP);
+            return TRUE;
+        case GDK_Up:
+            view->scrollBy(0, -LINE_STEP);
+            return TRUE;
+        case GDK_Right:
+            view->scrollBy(LINE_STEP, 0);
+            return TRUE;
+        case GDK_Left:
+            view->scrollBy(-LINE_STEP, 0);
+            return TRUE;
+        }
+    }
 
     return gtk_bindings_activate_event(GTK_OBJECT(widget), event);
 }