Modifying processing of pause actions in keyboard action sequences

The vast majority of use of pause actions in a keyboard action
sequence are used to synchronize the entire action chain with other
(mouse) action sequences. To allow the driver to properly synchronize
the input sequences, the driver will now ignore the duration property
of pause events in keyboard action sequences. This is deliberately in
violation of the W3C specification, and may be revisited in the future.

Author: jimevans

cpp/iedriver/InputManager.cpp

@@ -185,6 +185,16 @@ int InputManager::GetTicks(const Json::Value& sequences, Json::Value* ticks) {
       }
       Json::UInt action_index = static_cast<Json::UInt>(j);
       Json::Value action = actions[action_index];
+      if (device_type == "key" &&
+          action.isMember("type") &&
+          action["type"].isString() &&
+          action["type"].asString() == "pause") {
+        // HACK! Ignore the duration of pause events in keyboard action
+        // sequences. This is deliberately in violation of the W3C spec.
+        // This allows us to better synchronize mixed keyboard and mouse
+        // action sequences.
+        action["duration"] = 0;
+      }
       (*ticks)[action_index].append(action);
     }
   }