Don't upload extension IDs in the cloud policy protocol.

chrome/browser/policy/cloud/component_cloud_policy_browsertest.cc

 // Copyright (c) 2013 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 <string>
 
 #include "base/base64.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
@@ skipping 36 matching lines @@
 
 using testing::InvokeWithoutArgs;
 using testing::Mock;
 using testing::Return;
 using testing::_;
 
 namespace em = enterprise_management;
 
 namespace policy {
 
+namespace {
+
 const char kDMToken[] = "dmtoken";
 const char kDeviceID[] = "deviceid";
 
 const char kTestExtension[] = "kjmkgkdkpedkejedfhmfcenooemhbpbo";
 
 const base::FilePath::CharType kTestExtensionPath[] =
     FILE_PATH_LITERAL("extensions/managed_extension");
 
 const char kTestPolicy[] =
     "{"
@@ skipping 10 matching lines @@
 
 const char kTestPolicy2[] =
     "{"
     "  \"Another\": {"
     "    \"Value\": \"turn_it_off\""
     "  }"
     "}";
 
 const char kTestPolicy2JSON[] = "{\"Another\":\"turn_it_off\"}";
 
+#if !defined(OS_CHROMEOS)
 // Same encoding as ResourceCache does for its keys.
-bool Base64Encode(const std::string& value, std::string* encoded) {
+bool Base64UrlEncode(const std::string& value, std::string* encoded) {
   if (value.empty())
     return false;
   base::Base64Encode(value, encoded);
   base::ReplaceChars(*encoded, "+", "-", encoded);
   base::ReplaceChars(*encoded, "/", "_", encoded);
   return true;
 }
+#endif
+
+}  // namespace
 
 class ComponentCloudPolicyTest : public ExtensionBrowserTest {
  protected:
   ComponentCloudPolicyTest() {}
   virtual ~ComponentCloudPolicyTest() {}
 
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     ExtensionBrowserTest::SetUpCommandLine(command_line);
 #if defined(OS_CHROMEOS)
     // ExtensionBrowserTest sets the login users to a non-managed value;
@@ skipping 181 matching lines @@
 // get policy for components working again.
 #if !defined(OS_CHROMEOS)
 IN_PROC_BROWSER_TEST_F(ComponentCloudPolicyTest, SignOutAndBackIn) {
   // Read the initial policy.
   ExtensionTestMessageListener initial_policy_listener(kTestPolicyJSON, true);
   event_listener_->Reply("get-policy-Name");
   EXPECT_TRUE(initial_policy_listener.WaitUntilSatisfied());
 
   // Verify that the policy cache exists.
   std::string cache_key;
-  ASSERT_TRUE(Base64Encode("extension-policy", &cache_key));
+  ASSERT_TRUE(Base64UrlEncode("extension-policy", &cache_key));
   std::string cache_subkey;
-  ASSERT_TRUE(Base64Encode(kTestExtension, &cache_subkey));
+  ASSERT_TRUE(Base64UrlEncode(kTestExtension, &cache_subkey));
   base::FilePath cache_path = browser()->profile()->GetPath()
       .Append(FILE_PATH_LITERAL("Policy"))
       .Append(FILE_PATH_LITERAL("Components"))
       .AppendASCII(cache_key)
       .AppendASCII(cache_subkey);
   EXPECT_TRUE(base::PathExists(cache_path));
 
   // Now sign-out. The policy cache should be removed, and the extension should
   // get an empty policy update.
   ExtensionTestMessageListener event_listener("event", true);
@@ skipping 18 matching lines @@
   ExtensionTestMessageListener signin_policy_listener(kTestPolicyJSON, true);
   event_listener2.Reply("get-policy-Name");
   EXPECT_TRUE(signin_policy_listener.WaitUntilSatisfied());
 
   // And the cache is back.
   EXPECT_TRUE(base::PathExists(cache_path));
 }
 #endif
 
 }  // namespace policy