Disable/enable echo for enterprise device.

chrome/browser/policy/policy_browsertest.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 <algorithm>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 84 matching lines @@
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_filter.h"
 #include "policy/policy_constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "webkit/plugins/npapi/plugin_utils.h"
 #include "webkit/plugins/plugin_constants.h"
 #include "webkit/plugins/webplugininfo.h"
-
+#include "chrome/browser/extensions/extension_apitest.h"
+#include "chrome/browser/extensions/component_loader.h"

[oscarpan, 2013/02/06 04:38:45]

Either remove or reorder the imports in future, as this version is not
finalized.

 #if defined(OS_CHROMEOS)
 #include "ash/accelerators/accelerator_controller.h"
 #include "ash/accelerators/accelerator_table.h"
 #include "ash/shell.h"
 #include "ash/shell_delegate.h"
 #include "chrome/browser/chromeos/audio/audio_handler.h"
 #endif
 
 using content::BrowserThread;
 using content::URLRequestMockHTTPJob;
@@ skipping 174 matching lines @@
   bool result = false;
   EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
       contents,
       "var canvas = document.createElement('canvas');"
       "var context = canvas.getContext('experimental-webgl');"
       "domAutomationController.send(context != null);",
       &result));
   return result;
 }
 
+#if defined(OS_CHROMEOS)
+// Checks if redeeming cros offers is allowed in the given WebContents.
+bool CheckAllowRedeemCrosOffers(content::WebContents* contents) {
+  bool result = false;
+  EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
+      contents,
+      "chrome.echoPrivate.allowRedeemOffers("
+      "  function(allow) {"
+      "    domAutomationController.send(allow);"
+      "  });",
+      &result));
+  return result;
+}
+#endif
+
 bool IsJavascriptEnabled(content::WebContents* contents) {
   scoped_ptr<base::Value> value = content::ExecuteScriptAndGetValue(
       contents->GetRenderViewHost(), "123");
   int result = 0;
   if (!value->GetAsInteger(&result))
     EXPECT_EQ(base::Value::TYPE_NULL, value->GetType());
   return result == 123;
 }
 
 void CopyPluginListAndQuit(std::vector<webkit::WebPluginInfo>* out,
@@ skipping 208 matching lines @@
 
     content::WindowedNotificationObserver observer(
         chrome::NOTIFICATION_CRX_INSTALLER_DONE,
         content::NotificationService::AllSources());
     installer->InstallCrx(extension_path);
     observer.Wait();
     content::Details<const extensions::Extension> details = observer.details();
     return details.ptr();
   }
 
+  const extensions::Extension* LoadExtensionAsComponent(
+      const FilePath& path) {
+    ExtensionService* service = extension_service();
+
+    std::string manifest;
+    if (!file_util::ReadFileToString(
+            path.Append(extensions::Extension::kManifestFilename),
+            &manifest))
+      return NULL;
+
+    std::string extension_id = service->component_loader()->Add(manifest, path);
+    return service->extensions()->GetByID(extension_id);
+  }
+

[oscarpan, 2013/02/06 04:38:45]

This seems a bit ugly. I'm trying to load my extension as a component extension.
This cannot be done by InstallExtension()

[Mattias Nissler (ping if slow), 2013/02/06 10:03:26]

That's fine. The alternative would be to add the test to echo_private_apitest.cc
and use RunComponentExtensionTest, but then you'll have to find a way to inject
policy into the extension testing framework - I'm not familiar enough with that
to know whether that's easily possible.

   void UninstallExtension(const std::string& id, bool expect_success) {
     content::WindowedNotificationObserver observer(
         expect_success ? chrome::NOTIFICATION_EXTENSION_UNINSTALLED
                        : chrome::NOTIFICATION_EXTENSION_UNINSTALL_NOT_ALLOWED,
         content::NotificationService::AllSources());
     extension_service()->UninstallExtension(id, false, NULL);
     observer.Wait();
   }
 
   void UpdateProviderPolicy(const PolicyMap& policy) {
@@ skipping 769 matching lines @@
 
   // Enable via policy and verify that incognito windows can be opened.
   policies.Set(key::kIncognitoEnabled, POLICY_LEVEL_MANDATORY,
                POLICY_SCOPE_USER, base::Value::CreateBooleanValue(true));
   UpdateProviderPolicy(policies);
   EXPECT_TRUE(chrome::ExecuteCommand(browser(), IDC_NEW_INCOGNITO_WINDOW));
   EXPECT_EQ(2u, BrowserList::size());
   EXPECT_TRUE(BrowserList::IsOffTheRecordSessionActive());
 }
 
+#if defined(OS_CHROMEOS)
+IN_PROC_BROWSER_TEST_F(PolicyTest, AllowRedeemChromeOsRegistrationOffers) {
+  FilePath extension_path(ui_test_utils::GetTestFilePath(
+        FilePath(kTestExtensionsDir), FilePath("echo_component_extension")));
+  LoadExtensionAsComponent(extension_path);
+  content::WebContents* contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+  contents->GetController().LoadURL(
+      GURL("chrome-extension://lbgjohhgghbkcgejgklgcmfijhbheflf/main.html"),
+      content::Referrer(),

[oscarpan, 2013/02/06 04:38:45]

Although it seems it can access chrome.echoPrivate api now, but i'm getting the
following error when running the test.

[17747:17747:0205/202052:ERROR:request_sender.cc(133)] Could not find specified
request id: 0
../../chrome/browser/policy/policy_browsertest.cc:1368: Failure
Value of: CheckAllowRedeemCrosOffers(contents)
  Actual: true
Expected: false
[17747:17747:0205/202052:ERROR:request_sender.cc(133)] Could not find specified
request id: 1
[17747:17747:0205/202052:ERROR:request_sender.cc(133)] Could not find specified
request id: 2
[  FAILED  ] PolicyTest.AllowRedeemChromeOsRegistrationOffers, where TypeParam =
 and GetParam() =  (3579 ms)
[----------] 1 test from PolicyTest (3580 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (3580 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] PolicyTest.AllowRedeemChromeOsRegistrationOffers, where TypeParam =
 and GetParam() = 

 1 FAILED TEST
[0205/202054:ERROR:process_util_posix.cc(297)] Unable to terminate process group
17702: No such process

[Mattias Nissler (ping if slow), 2013/02/06 10:03:26]

Seems like you want to attach a debugger and check why the policy value is not
taking effect in the extension function.

+      content::PAGE_TRANSITION_AUTO_TOPLEVEL,
+      std::string());
+  content::WaitForLoadStop(contents);
+  EXPECT_TRUE(CheckAllowRedeemCrosOffers(contents));
+  PolicyMap policies;
+
+  // Allow offer redemption and verify the flag value through js.
+  policies.Set(key::kDeviceAllowRedeemChromeOsRegistrationOffers,
+               POLICY_LEVEL_MANDATORY,
+               POLICY_SCOPE_MACHINE,
+               base::Value::CreateBooleanValue(false));
+  UpdateProviderPolicy(policies);
+  EXPECT_FALSE(CheckAllowRedeemCrosOffers(contents));
+
+  // Disallow offer redemption and verify the flag value through js.
+  policies.Set(key::kDeviceAllowRedeemChromeOsRegistrationOffers,
+               POLICY_LEVEL_MANDATORY,
+               POLICY_SCOPE_MACHINE,
+               base::Value::CreateBooleanValue(true));
+
+  UpdateProviderPolicy(policies);
+  EXPECT_TRUE(CheckAllowRedeemCrosOffers(contents));
+}
+#endif
+
 IN_PROC_BROWSER_TEST_F(PolicyTest, Javascript) {
   // Verifies that Javascript can be disabled.
   content::WebContents* contents =
       browser()->tab_strip_model()->GetActiveWebContents();
   EXPECT_TRUE(IsJavascriptEnabled(contents));
   EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_DEV_TOOLS));
   EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_DEV_TOOLS_CONSOLE));
 
   // Disable Javascript via policy.
   PolicyMap policies;
@@ skipping 545 matching lines @@
                  this));
 
   MessageLoop::current()->Run();
 }
 
 INSTANTIATE_TEST_CASE_P(MediaStreamDevicesControllerBrowserTestInstance,
                         MediaStreamDevicesControllerBrowserTest,
                         testing::Bool());
 
 }  // namespace policy