add new behavior option for long keypress

Author: hekra01

Diff for: src/webdriver/extension_qt/q_key_converter.cc

@@ -299,6 +299,14 @@ bool QKeyConverter::ConvertKeysToWebKeyEvents(const string16& client_keys,
         if (i > 0 && key == keys[i - 1]) 
             autoPress = true;
 
+	bool sendRelease;
+#if WD_ENABLE_ONE_KEYRELEASE 
+        // Send only last key release
+        sendRelease = !autoRelease;
+ #else
+        // Send key release in all cases
+        sendRelease = true;
+#endif
         // Create the key events.
         bool necessary_modifiers[3];
         for (int i = 0; i < 3; ++i) {
@@ -313,18 +321,25 @@ bool QKeyConverter::ConvertKeysToWebKeyEvents(const string16& client_keys,
 
         if (unmodified_text.length() || modified_text.length()) {
             key_events.push_back(QKeyEvent(QEvent::KeyPress, key_code, all_modifiers, unmodified_text.c_str(), autoPress));
-            key_events.push_back(QKeyEvent(QEvent::KeyRelease, key_code, all_modifiers, unmodified_text.c_str(), autoRelease));      
+            if (sendRelease) {
+                key_events.push_back(QKeyEvent(QEvent::KeyRelease, key_code, all_modifiers, unmodified_text.c_str(), autoRelease));      
+            }
         }
         else
         {
             key_events.push_back(QKeyEvent(QEvent::KeyPress, key_code, all_modifiers, QString::null, autoPress));
-            key_events.push_back(QKeyEvent(QEvent::KeyRelease, key_code, all_modifiers, QString::null, autoRelease));
+            if (sendRelease) {
+                key_events.push_back(QKeyEvent(QEvent::KeyRelease, key_code, all_modifiers, QString::null, autoRelease));
+            }
         }
 
-        for (int i = 2; i > -1; --i) {
-            if (necessary_modifiers[i]) {
-                key_events.push_back(
-                    QKeyEvent(QEvent::KeyRelease, kModifiers[i].key_code, sticky_modifiers, QString::null, autoRelease));
+        
+        if (sendRelease) {
+            for (int i = 2; i > -1; --i) {
+                if (necessary_modifiers[i]) {
+                    key_events.push_back(
+                        QKeyEvent(QEvent::KeyRelease, kModifiers[i].key_code, sticky_modifiers, QString::null, autoRelease));
+                }
             }
         }
     }

Diff for: wd_build_options.gypi

@@ -131,6 +131,10 @@
 
     [ '<(WD_CONFIG_PLAYER) == 1', {
      'defines': [ 'WD_ENABLE_PLAYER=1' ],
-    } ]
+    }],
+
+    [ '<(WD_CONFIG_ONE_KEYRELEASE) == 1', {
+     'defines': [ 'WD_ENABLE_ONE_KEYRELEASE=1' ],
+    }]
   ],
 }

Diff for: wd_common.gypi

@@ -6,6 +6,7 @@
     'WD_CONFIG_QUICK%': '1',
     'WD_BUILD_MONGOOSE%': '0',
     'WD_CONFIG_PLAYER%': '1',
+    'WD_CONFIG_ONE_KEYRELEASE%': '0',
 
     'QT_BIN_PATH%': '/usr/lib/qt4/bin',
     'QT_INC_PATH%': '/usr/include',