Experimental Extension Keybinding (first cut).

chrome/common/extensions/extension.cc

 // Copyright (c) 2012 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/common/extensions/extension.h"
 
 #include <algorithm>
 
 #include "base/base64.h"
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/i18n/rtl.h"
 #include "base/logging.h"
 #include "base/memory/singleton.h"
 #include "base/stl_util.h"
 #include "base/string16.h"
 #include "base/string_number_conversions.h"
 #include "base/string_piece.h"
+#include "base/string_split.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "base/version.h"
 #include "crypto/sha2.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/extensions/csp_validator.h"
 #include "chrome/common/extensions/extension_action.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_error_utils.h"
 #include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "chrome/common/extensions/file_browser_handler.h"
 #include "chrome/common/extensions/manifest.h"
 #include "chrome/common/extensions/user_script.h"
 #include "chrome/common/url_constants.h"
 #include "googleurl/src/url_util.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "net/base/registry_controlled_domain.h"
 #include "third_party/skia/include/core/SkBitmap.h"
+#include "ui/base/keycodes/keyboard_codes.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "webkit/glue/image_decoder.h"
 #include "webkit/glue/web_intent_service_data.h"
 
 namespace keys = extension_manifest_keys;
 namespace values = extension_manifest_values;
 namespace errors = extension_manifest_errors;
 
 using extensions::csp_validator::ContentSecurityPolicyIsLegal;
@@ skipping 157 matching lines @@
       shortcut_alt(false),
       shortcut_ctrl(false),
       shortcut_shift(false) {
 }
 
 Extension::InputComponentInfo::~InputComponentInfo() {}
 
 Extension::TtsVoice::TtsVoice() {}
 Extension::TtsVoice::~TtsVoice() {}
 
+Extension::ExtensionKeybinding::ExtensionKeybinding() {}
+Extension::ExtensionKeybinding::~ExtensionKeybinding() {}
+
+bool Extension::ExtensionKeybinding::Parse(DictionaryValue* command,
+                                           const std::string& command_name,
+                                           int index,
+                                           string16* error) {
+  DCHECK(!command_name.empty());
+  std::string key_binding;
+  if (!command->GetString(keys::kKey, &key_binding) ||
+      key_binding.empty()) {
+    *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+        errors::kInvalidKeyBinding,
+        base::IntToString(index),
+        "Missing");
+    return false;
+  }
+
+  std::string original_keybinding = key_binding;
+  // Normalize '-' to '+'.
+  ReplaceSubstringsAfterOffset(&key_binding, 0, "-", "+");
+  // Remove all spaces.
+  ReplaceSubstringsAfterOffset(&key_binding, 0, " ", "");
+  // And finally, lower-case it.
+  key_binding = StringToLowerASCII(key_binding);
+
+  std::vector<std::string> tokens;
+  base::SplitString(key_binding, '+', &tokens);
+  if (tokens.size() < 2 || tokens.size() > 3) {
+    *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+        errors::kInvalidKeyBinding,
+        base::IntToString(index),
+        original_keybinding);
+    return false;
+  }
+
+  // Now, parse it into an accelerator.
+  bool ctrl = false;
+  bool alt = false;
+  bool shift = false;
+  ui::KeyboardCode key = ui::VKEY_UNKNOWN;
+  for (size_t i = 0; i < tokens.size(); i++) {
+    if (tokens[i] == "ctrl") {
+      ctrl = true;
+    } else if (tokens[i] == "alt") {
+      alt = true;
+    } else if (tokens[i] == "shift") {
+      shift = true;
+    } else if (tokens[i].size() == 1 &&
+               base::ToUpperASCII(tokens[i][0]) >= ui::VKEY_A &&

[Matt Perry, 2012/02/22 23:59:42]

Just use 'a' so you don't have to uppercase it. Also, you're actually looking at
a character rather than a KeyboardCode, so it's safer to compare to the ASCII
val in case the KeyboardCode isn't the same.

+               base::ToUpperASCII(tokens[i][0]) <= ui::VKEY_Z) {
+      if (key != ui::VKEY_UNKNOWN) {
+        // Multiple key assignments.
+        key = ui::VKEY_UNKNOWN;
+        break;
+      }
+
+      key = static_cast<ui::KeyboardCode>(
+          ui::VKEY_A + (base::ToUpperASCII(tokens[i][0]) - ui::VKEY_A));

[Matt Perry, 2012/02/22 23:59:42]

see above, second VKEY_A => 'a'

+    } else {
+      *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+          errors::kInvalidKeyBinding,
+          base::IntToString(index),
+          original_keybinding);
+      return false;
+    }
+  }
+
+  // We support Ctrl+foo, Alt+foo, Ctrl+Shift+foo, Alt+Shift+foo, but not
+  // Ctrl+Alt+foo. For a more detailed reason why we don't support Ctrl+Alt+foo:
+  // http://blogs.msdn.com/b/oldnewthing/archive/2004/03/29/101121.aspx.
+  if (key == ui::VKEY_UNKNOWN || ctrl == true && alt == true) {

[Matt Perry, 2012/02/22 23:59:42]

Please add parens around (ctrl == true && alt == true) since it's always
confusing what has higher precedence.

Also, just test (ctrl && alt) since they're bools.

+    *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+        errors::kInvalidKeyBinding,
+        base::IntToString(index),
+        original_keybinding);
+    return false;
+  }
+
+  accelerator_ = ui::Accelerator(key, shift, ctrl, alt);
+
+  if (command_name !=
+          extension_manifest_values::kPageActionKeybindingEvent &&
+      command_name !=
+          extension_manifest_values::kBrowserActionKeybindingEvent) {
+    if (!command->GetString(keys::kDescription, &description_) ||
+        description_.empty()) {
+      *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+          errors::kInvalidKeyBindingDescription,
+          base::IntToString(index));
+      return false;
+    }
+  }
+
+  command_name_ = command_name;
+  return true;
+}
+
 //
 // Extension
 //
 
 // static
 scoped_refptr<Extension> Extension::Create(const FilePath& path,
                                            Location location,
                                            const DictionaryValue& value,
                                            int flags,
                                            std::string* utf8_error) {
@@ skipping 619 matching lines @@
           errors::kInvalidURLPatternError, filter);
       return NULL;
     }
     result->AddPattern(pattern);
   }
 
   std::string default_icon;
   // Read the file browser action |default_icon| (optional).
   if (file_browser_handler->HasKey(keys::kPageActionDefaultIcon)) {
     if (!file_browser_handler->GetString(
-            keys::kPageActionDefaultIcon,&default_icon) ||
+            keys::kPageActionDefaultIcon, &default_icon) ||
         default_icon.empty()) {
       *error = ASCIIToUTF16(errors::kInvalidPageActionIconPath);
       return NULL;
     }
     result->set_icon_path(default_icon);
   }
 
   return result.release();
 }
 
@@ skipping 283 matching lines @@
       *error = ASCIIToUTF16(errors::kInvalidIntent);
       return false;
     }
     service.action = UTF8ToUTF16(*iter);
 
     ListValue* mime_types = NULL;
     if (!one_service->HasKey(keys::kIntentType) ||
         !one_service->GetList(keys::kIntentType, &mime_types) ||
         mime_types->GetSize() == 0) {
       *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
-          errors::kInvalidIntentType,*iter);
+          errors::kInvalidIntentType, *iter);
       return false;
     }
 
     if (one_service->HasKey(keys::kIntentPath)) {
       if (!one_service->GetString(keys::kIntentPath, &value)) {
         *error = ASCIIToUTF16(errors::kInvalidIntentPath);
         return false;
       }
       if (is_hosted_app()) {
         // Hosted apps require an absolute URL for intents.
         GURL service_url(value);
         if (!service_url.is_valid() ||
             !(web_extent().MatchesURL(service_url))) {
           *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
-              errors::kInvalidIntentPageInHostedApp,*iter);
+              errors::kInvalidIntentPageInHostedApp, *iter);
           return false;
         }
         service.service_url = service_url;
       } else {
         // We do not allow absolute intent URLs in non-hosted apps.
         if (GURL(value).is_valid()) {
           *error =ExtensionErrorUtils::FormatErrorMessageUTF16(
-              errors::kCannotAccessPage,value.c_str());
+              errors::kCannotAccessPage, value.c_str());
           return false;
         }
         service.service_url = GetResourceURL(value);
       }
     }
 
     if (one_service->HasKey(keys::kIntentTitle) &&
         !one_service->GetString(keys::kIntentTitle, &service.title)) {
       *error = ASCIIToUTF16(errors::kInvalidIntentTitle);
       return false;
@@ skipping 513 matching lines @@
           minimum_version_string);
       return false;
     }
   }
 
   // Initialize converted_from_user_script (if present)
   if (manifest->HasKey(keys::kConvertedFromUserScript))
     manifest->GetBoolean(keys::kConvertedFromUserScript,
                         &converted_from_user_script_);
 
+  // Initialize commands (if present).
+  if (manifest->HasKey(keys::kCommands)) {
+    DictionaryValue* commands = NULL;
+    if (!manifest->GetDictionary(keys::kCommands, &commands)) {
+      *error = ASCIIToUTF16(errors::kInvalidCommandsKey);
+      return false;
+    }
+
+    int command_index = 0;
+    for (DictionaryValue::key_iterator iter = commands->begin_keys();
+         iter != commands->end_keys(); ++iter) {
+      ++command_index;
+
+      DictionaryValue* command = NULL;
+      if (!commands->GetDictionary(*iter, &command)) {
+        *error = ExtensionErrorUtils::FormatErrorMessageUTF16(
+            errors::kInvalidKeyBindingDictionary,
+            base::IntToString(command_index));
+        return false;
+      }
+
+      ExtensionKeybinding binding;
+      if (!binding.Parse(command, *iter, command_index, error))
+        return false;  // |error| already set.
+
+      commands_.push_back(binding);
+    }
+  }
+
   // Initialize icons (if present).
   if (manifest->HasKey(keys::kIcons)) {
     DictionaryValue* icons_value = NULL;
     if (!manifest->GetDictionary(keys::kIcons, &icons_value)) {
       *error = ASCIIToUTF16(errors::kInvalidIcons);
       return false;
     }
 
     for (size_t i = 0; i < arraysize(kIconSizes); ++i) {
       std::string key = base::IntToString(kIconSizes[i]);
@@ skipping 1143 matching lines @@
 
   if (permission->id() == ExtensionAPIPermission::kExperimental) {
     if (!CanSpecifyExperimentalPermission()) {
       *error = ASCIIToUTF16(errors::kExperimentalFlagRequired);
       return false;
     }
   }
 
   bool supports_type = false;
   switch (GetType()) {
-    case TYPE_USER_SCRIPT: // Pass through.
+    case TYPE_USER_SCRIPT:  // Pass through.
     case TYPE_EXTENSION:
       supports_type = permission->supports_extensions();
       break;
     case TYPE_HOSTED_APP:
       supports_type = permission->supports_hosted_apps();
       break;
     case TYPE_PACKAGED_APP:
       supports_type = permission->supports_packaged_apps();
       break;
     case TYPE_PLATFORM_APP:
@@ skipping 135 matching lines @@
   }
 }
 
 bool Extension::IsSyncable() const {
   // TODO(akalin): Figure out if we need to allow some other types.
 
   // We want to sync any extensions that are shown in the luancher because
   // their positions should sync.
   return location_ == Extension::INTERNAL ||
       ShouldDisplayInLauncher();
-
 }
 
 bool Extension::ShouldDisplayInLauncher() const {
   // All apps should be displayed on the NTP except for the Cloud Print App.
   return is_app() && id() != extension_misc::kCloudPrintAppId;
 }
 
 ExtensionInfo::ExtensionInfo(const DictionaryValue* manifest,
                              const std::string& id,
                              const FilePath& path,
@@ skipping 29 matching lines @@
     already_disabled(false),
     extension(extension) {}
 
 UpdatedExtensionPermissionsInfo::UpdatedExtensionPermissionsInfo(
     const Extension* extension,
     const ExtensionPermissionSet* permissions,
     Reason reason)
     : reason(reason),
       extension(extension),
       permissions(permissions) {}