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) {

[Mattias Nissler (ping if slow), 2014/04/24 12:20:36]

side note: The name of this function is misleading, should be Base64UrlEncode

[Joao da Silva, 2014/04/24 13:22:24]

Done.

   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 222 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