Add 2 Extension APIs for handwriting: experimental.input.sendHandritingStroke and cancelHandWriting

chrome/browser/extensions/extension_input_api.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/extension_input_api.h"
 
 #include <string>
 
 #include "base/string_util.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/key_identifier_conversion_views.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "views/events/event.h"
 #include "views/ime/input_method.h"
 #include "views/widget/widget.h"
 
+#if defined(OS_CHROMEOS) && defined(TOUCH_UI)
+#include "chrome/browser/chromeos/cros/cros_library.h"
+#include "chrome/browser/chromeos/cros/input_method_library.h"
+#endif
+
 namespace {
 
 // Keys.
 const char kType[] = "type";
 const char kKeyIdentifier[] = "keyIdentifier";
 const char kAlt[] = "altKey";
 const char kCtrl[] = "ctrlKey";
 const char kMeta[] = "metaKey";
 const char kShift[] = "shiftKey";
 const char kKeyDown[] = "keydown";
 const char kKeyUp[] = "keyup";
 
 // Errors.
 const char kUnknownEventTypeError[] = "Unknown event type.";
 const char kUnknownOrUnsupportedKeyIdentiferError[] = "Unknown or unsupported "
     "key identifier.";
 const char kUnsupportedModifier[] = "Unsupported modifier.";
 const char kNoValidRecipientError[] = "No valid recipient for event.";
 const char kKeyEventUnprocessedError[] = "Event was not handled.";
+#if defined(OS_CHROMEOS) && defined(TOUCH_UI)
+const char kCrosLibraryNotLoadedError[] = "Cros shared library not loaded.";
+#endif
 
 ui::EventType GetTypeFromString(const std::string& type) {
   if (type == kKeyDown) {
     return ui::ET_KEY_PRESSED;
   } else if (type == kKeyUp) {
     return ui::ET_KEY_RELEASED;
   }
   return ui::ET_UNKNOWN;
 }
 
@@ skipping 61 matching lines @@
   views::InputMethod* ime = widget->GetInputMethod();
   if (ime) {
     ime->DispatchKeyEvent(event);
   } else if (!widget->OnKeyEvent(event)) {
     error_ = kKeyEventUnprocessedError;
     return false;
   }
 
   return true;
 }
+
+#if defined(OS_CHROMEOS) && defined(TOUCH_UI)
+bool SendHandwritingStrokeFunction::RunImpl() {
+  // TODO(yusukes): Add a parameter for an input context ID.
+  ListValue* value = NULL;
+  EXTENSION_FUNCTION_VALIDATE(args_->GetList(0, &value));
+
+  chromeos::HandwritingStroke stroke;
+  for (size_t i = 0; i < value->GetSize(); ++i) {
+    DictionaryValue* dict;
+    double x = 0.0;
+    double y = 0.0;
+    EXTENSION_FUNCTION_VALIDATE(value->GetDictionary(i, &dict));
+    EXTENSION_FUNCTION_VALIDATE(dict->GetDouble("x", &x));
+    EXTENSION_FUNCTION_VALIDATE(dict->GetDouble("y", &y));
+    stroke.push_back(std::make_pair(x, y));
+  }
+  chromeos::CrosLibrary* cros_library = chromeos::CrosLibrary::Get();

[Mihai Parparita -not on Chrome, 2011/05/05 22:47:49]

Shouldn't this be at the start of the function?

[Yusuke Sato, 2011/05/16 15:47:38]

Done.

+  if (!cros_library->EnsureLoaded()) {
+    error_ = kCrosLibraryNotLoadedError;
+    return false;
+  }
+  cros_library->GetInputMethodLibrary()->SendHandwritingStroke(stroke);
+  return true;
+}
+
+bool CancelHandwritingFunction::RunImpl() {
+  // TODO(yusukes): Add a parameter for an input context ID.
+  int n_strokes = 0;  // zero means 'clear all strokes'.
+  if (HasOptionalArgument(0)) {
+    EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &n_strokes));
+  }
+  chromeos::CrosLibrary* cros_library = chromeos::CrosLibrary::Get();

[Mihai Parparita -not on Chrome, 2011/05/05 22:47:49]

Ditto.

[Yusuke Sato, 2011/05/16 15:47:38]

Done.

+  if (!cros_library->EnsureLoaded()) {
+    error_ = kCrosLibraryNotLoadedError;
+    return false;
+  }
+  cros_library->GetInputMethodLibrary()->CancelHandwriting(n_strokes);

[Erik does not do reviews, 2011/05/06 00:51:03]

is passing a negative number (or a large number) bad here?  If so, you should
guard that with EXTENSION_FUNCTION_VALIDATE.

[Yusuke Sato, 2011/05/16 15:47:38]

Done.

+  return true;
+}
+#endif