[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373

eric at webkit.org eric at webkit.org
Thu Apr 8 01:56:40 UTC 2010


The following commit has been merged in the webkit-1.2 branch:
commit d44d48a5848e295622d673c263580162cfc4f86a
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Feb 23 11:09:57 2010 +0000

    2010-02-23  Stephan Aßmus  <superstippi at gmx.de>
    
            Reviewed by Eric Seidel.
    
            [Haiku] Fix various issues in keyboard event generation.
            https://bugs.webkit.org/show_bug.cgi?id=34685
    
            Covered by existing tests.
    
            Fixed backspace keycode.
            Fixed using the appropriate fields from the Haiku event for mapping
            to the VK_* codes and added mapping a great deal more codes.
            Added extraction of modifier key flags.
            Completed implementation of disambiguateKeyDownEvent()
            Implemented currentCapsLockState().
    
            * platform/haiku/PlatformKeyboardEventHaiku.cpp:
            (WebCore::keyIdentifierForHaikuKeyCode):
            (WebCore::windowsKeyCodeForKeyEvent):
            (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
            (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):
            (WebCore::PlatformKeyboardEvent::currentCapsLockState):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@55138 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index ee08b35..12d8c38 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,26 @@
+2010-02-23  Stephan Aßmus  <superstippi at gmx.de>
+
+        Reviewed by Eric Seidel.
+
+        [Haiku] Fix various issues in keyboard event generation.
+        https://bugs.webkit.org/show_bug.cgi?id=34685
+
+        Covered by existing tests.
+
+        Fixed backspace keycode.
+        Fixed using the appropriate fields from the Haiku event for mapping
+        to the VK_* codes and added mapping a great deal more codes.
+        Added extraction of modifier key flags.
+        Completed implementation of disambiguateKeyDownEvent()
+        Implemented currentCapsLockState().
+
+        * platform/haiku/PlatformKeyboardEventHaiku.cpp:
+        (WebCore::keyIdentifierForHaikuKeyCode):
+        (WebCore::windowsKeyCodeForKeyEvent):
+        (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
+        (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):
+        (WebCore::PlatformKeyboardEvent::currentCapsLockState):
+
 2010-02-23  Steve Block  <steveblock at google.com>
 
         Reviewed by Darin Adler.
diff --git a/WebCore/platform/haiku/PlatformKeyboardEventHaiku.cpp b/WebCore/platform/haiku/PlatformKeyboardEventHaiku.cpp
index 1545dfb..053e30a 100644
--- a/WebCore/platform/haiku/PlatformKeyboardEventHaiku.cpp
+++ b/WebCore/platform/haiku/PlatformKeyboardEventHaiku.cpp
@@ -1,6 +1,7 @@
 /*
  * Copyright (C) 2007 Ryan Leavengood <leavengood at gmail.com>
  * Copyright (C) 2008 Andrea Anzani <andrea.anzani at gmail.com>
+ * Copyright (C) 2010 Stephan Aßmus <superstippi at gmx.de>
  *
  * All rights reserved.
  *
@@ -29,6 +30,7 @@
 #include "config.h"
 #include "PlatformKeyboardEvent.h"
 
+#include "CString.h"
 #include "KeyboardCodes.h"
 #include "NotImplemented.h"
 #include <InterfaceDefs.h>
@@ -78,7 +80,7 @@ static String keyIdentifierForHaikuKeyCode(char singleByte, int keyCode)
         break;
 
     case B_BACKSPACE:
-        return "U+0009";
+        return "U+0008";
     case B_LEFT_ARROW:
         return "Left";
     case B_RIGHT_ARROW:
@@ -105,12 +107,47 @@ static String keyIdentifierForHaikuKeyCode(char singleByte, int keyCode)
         return "U+0009";
     }
 
-    return String::format("U+%04X", toASCIIUpper(keyCode));
+    return String::format("U+%04X", toASCIIUpper(singleByte));
 }
 
-static int windowsKeyCodeForKeyEvent(char singleByte)
+static int windowsKeyCodeForKeyEvent(char singleByte, int keyCode)
 {
     switch (singleByte) {
+    case B_FUNCTION_KEY:
+        switch (keyCode) {
+        case B_F1_KEY:
+            return VK_F1;
+        case B_F2_KEY:
+            return VK_F2;
+        case B_F3_KEY:
+            return VK_F3;
+        case B_F4_KEY:
+            return VK_F4;
+        case B_F5_KEY:
+            return VK_F5;
+        case B_F6_KEY:
+            return VK_F6;
+        case B_F7_KEY:
+            return VK_F7;
+        case B_F8_KEY:
+            return VK_F8;
+        case B_F9_KEY:
+            return VK_F9;
+        case B_F10_KEY:
+            return VK_F10;
+        case B_F11_KEY:
+            return VK_F11;
+        case B_F12_KEY:
+            return VK_F12;
+        case B_PRINT_KEY:
+            return VK_PRINT;
+        case B_PAUSE_KEY:
+            return 0; // FIXME
+        case B_SCROLL_KEY:
+            return 0; // FIXME
+        }
+        break;
+
     case B_BACKSPACE:
         return VK_BACK; // (08) BACKSPACE key
     case B_TAB:
@@ -141,39 +178,191 @@ static int windowsKeyCodeForKeyEvent(char singleByte)
         return VK_INSERT; // (2D) INS key
     case B_DELETE:
         return VK_DELETE; // (2E) DEL key
-    case 0x2e:
-    default:
-        return 0;
+
+    case '0':
+    case ')':
+        return VK_0;
+    case '1':
+    case '!':
+        return VK_1;
+    case '2':
+    case '@':
+        return VK_2;
+    case '3':
+    case '#':
+        return VK_3;
+    case '4':
+    case '$':
+        return VK_4;
+    case '5':
+    case '%':
+        return VK_5;
+    case '6':
+    case '^':
+        return VK_6;
+    case '7':
+    case '&':
+        return VK_7;
+    case '8':
+    case '*':
+        return VK_8;
+    case '9':
+    case '(':
+        return VK_9;
+    case 'a':
+    case 'A':
+        return VK_A;
+    case 'b':
+    case 'B':
+        return VK_B;
+    case 'c':
+    case 'C':
+        return VK_C;
+    case 'd':
+    case 'D':
+        return VK_D;
+    case 'e':
+    case 'E':
+        return VK_E;
+    case 'f':
+    case 'F':
+        return VK_F;
+    case 'g':
+    case 'G':
+        return VK_G;
+    case 'h':
+    case 'H':
+        return VK_H;
+    case 'i':
+    case 'I':
+        return VK_I;
+    case 'j':
+    case 'J':
+        return VK_J;
+    case 'k':
+    case 'K':
+        return VK_K;
+    case 'l':
+    case 'L':
+        return VK_L;
+    case 'm':
+    case 'M':
+        return VK_M;
+    case 'n':
+    case 'N':
+        return VK_N;
+    case 'o':
+    case 'O':
+        return VK_O;
+    case 'p':
+    case 'P':
+        return VK_P;
+    case 'q':
+    case 'Q':
+        return VK_Q;
+    case 'r':
+    case 'R':
+        return VK_R;
+    case 's':
+    case 'S':
+        return VK_S;
+    case 't':
+    case 'T':
+        return VK_T;
+    case 'u':
+    case 'U':
+        return VK_U;
+    case 'v':
+    case 'V':
+        return VK_V;
+    case 'w':
+    case 'W':
+        return VK_W;
+    case 'x':
+    case 'X':
+        return VK_X;
+    case 'y':
+    case 'Y':
+        return VK_Y;
+    case 'z':
+    case 'Z':
+        return VK_Z;
+    case ';':
+    case ':':
+        return VK_OEM_1;
+    case '+':
+    case '=':
+        return VK_OEM_PLUS;
+    case ',':
+    case '<':
+        return VK_OEM_COMMA;
+    case '-':
+    case '_':
+        return VK_OEM_MINUS;
+    case '.':
+    case '>':
+        return VK_OEM_PERIOD;
+    case '/':
+    case '?':
+        return VK_OEM_2;
+    case '`':
+    case '~':
+        return VK_OEM_3;
+    case '[':
+    case '{':
+        return VK_OEM_4;
+    case '\\':
+    case '|':
+        return VK_OEM_5;
+    case ']':
+    case '}':
+        return VK_OEM_6;
+    case '\'':
+    case '"':
+        return VK_OEM_7;
     }
+    return singleByte;
 }
 
 PlatformKeyboardEvent::PlatformKeyboardEvent(BMessage* message)
     : m_autoRepeat(false)
+    , m_isKeypad(false)
+    , m_shiftKey(false)
     , m_ctrlKey(false)
     , m_altKey(false)
     , m_metaKey(false)
-    , m_isKeypad(false)
-    , m_shiftKey(false)
 {
     BString bytes = message->FindString("bytes");
 
+    m_nativeVirtualKeyCode = message->FindInt32("key");
+
     m_text = String::fromUTF8(bytes.String(), bytes.Length());
     m_unmodifiedText = String(bytes.String(), bytes.Length());
-    m_keyIdentifier = keyIdentifierForHaikuKeyCode(bytes.ByteAt(0), message->FindInt32("key"));
+    m_keyIdentifier = keyIdentifierForHaikuKeyCode(bytes.ByteAt(0), m_nativeVirtualKeyCode);
+
+    m_windowsVirtualKeyCode = windowsKeyCodeForKeyEvent(bytes.ByteAt(0), m_nativeVirtualKeyCode);
 
     if (message->what == B_KEY_UP)
         m_type = KeyUp;
     else if (message->what == B_KEY_DOWN)
         m_type = KeyDown;
 
-    m_windowsVirtualKeyCode = windowsKeyCodeForKeyEvent(bytes.ByteAt(0));
+    int32 modifiers = message->FindInt32("modifiers");
+    m_shiftKey = modifiers & B_SHIFT_KEY;
+    m_ctrlKey = modifiers & B_COMMAND_KEY;
+    m_altKey = modifiers & B_CONTROL_KEY;
+    m_metaKey = modifiers & B_OPTION_KEY;
 }
 
-void PlatformKeyboardEvent::disambiguateKeyDownEvent(Type type, bool)
+void PlatformKeyboardEvent::disambiguateKeyDownEvent(Type type, bool backwardCompatibilityMode)
 {
     // Can only change type from KeyDown to RawKeyDown or Char, as we lack information for other conversions.
     ASSERT(m_type == KeyDown);
     m_type = type;
+
+    if (backwardCompatibilityMode)
+        return;
+
     if (type == RawKeyDown) {
         m_text = String();
         m_unmodifiedText = String();
@@ -185,8 +374,7 @@ void PlatformKeyboardEvent::disambiguateKeyDownEvent(Type type, bool)
 
 bool PlatformKeyboardEvent::currentCapsLockState()
 {
-    notImplemented();
-    return false;
+    return ::modifiers() & B_CAPS_LOCK;
 }
 
 } // namespace WebCore

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list