Enabling Commands API for Apps

chrome/common/extensions/api/commands/commands_manifest_unittest.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/manifest_tests/extension_manifest_test.h"
 
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/api/commands/commands_handler.h"
+#include "chrome/common/extensions/features/feature_channel.h"
 #include "extensions/common/manifest_constants.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace extensions {
 
 namespace errors = manifest_errors;
 
 class CommandsManifestTest : public ExtensionManifestTest {
 };
 
@@ skipping 65 matching lines @@
   scoped_refptr<Extension> extension =
       LoadAndExpectSuccess("browser_action_synthesizes_command.json");
   // An extension with a browser action but no extension command specified
   // should get a command assigned to it.
   const extensions::Command* command =
       CommandsInfo::GetBrowserActionCommand(extension.get());
   ASSERT_TRUE(command != NULL);
   ASSERT_EQ(ui::VKEY_UNKNOWN, command->accelerator().key_code());
 }
 
+// This test makes sure that the "commands" feature and the "commands.global"
+// property behave as expected on dev and stable (enabled and working on dev,
+// not working on stable).
+TEST_F(CommandsManifestTest, ChannelTests) {
+  // This tests the following combinations.
+  // Ext+Command         Stable : OK.
+  // Ext+Command+Global  Stable : Property is silently ignored (expect success).
+  // App+Command         Stable : NOT OK.
+  // App+Command+Global  Stable : NOT OK.
+  {
+    std::string warning = "'commands' requires Google Chrome dev channel or "
+                          "newer, but this is the stable channel.";
+    ScopedCurrentChannel channel(chrome::VersionInfo::CHANNEL_STABLE);
+    scoped_refptr<Extension> extension1 =
+        LoadAndExpectSuccess("command_ext.json");
+    scoped_refptr<Extension> extension2 =
+        LoadAndExpectSuccess("command_ext_global.json");
+    LoadAndExpectWarning("command_app.json", warning);
+    LoadAndExpectWarning("command_app_global.json", warning);
+  }
+
+  // Ext+Command         Dev    : OK.
+  // App+Command         Dev    : OK.
+  // Ext+Command+Global  Dev    : OK.
+  // App+Command+Global  Dev    : OK.
+  {
+    ScopedCurrentChannel channel(chrome::VersionInfo::CHANNEL_DEV);
+    scoped_refptr<Extension> extension1 =
+        LoadAndExpectSuccess("command_ext.json");
+    scoped_refptr<Extension> extension2 =
+        LoadAndExpectSuccess("command_app.json");
+    scoped_refptr<Extension> extension3 =
+        LoadAndExpectSuccess("command_ext_global.json");
+    scoped_refptr<Extension> extension4 =
+        LoadAndExpectSuccess("command_app_global.json");
+  }
+}
+
 }  // namespace extensions