Fix log spew for "_comment..." policies.

components/policy/core/common/config_dir_policy_loader_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 "components/policy/core/common/config_dir_policy_loader.h"
 
 #include <utility>
 
 #include "base/compiler_specific.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequenced_task_runner.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 #include "components/policy/core/common/async_policy_provider.h"
 #include "components/policy/core/common/configuration_policy_provider_test.h"
 #include "components/policy/core/common/policy_bundle.h"
 #include "components/policy/core/common/policy_map.h"
 #include "components/policy/core/common/policy_types.h"
 
-// Use real existing policies as samples because ConfigDirPolicyLoader does
-// basic verification at load time.  These constants are specific to
-// ConfigDirPolicyLoader and thus are hidden in an anonymous namespace.  (They
-// cannot be applied generally because Windows tests depend on reading GPO
-// sample files containing hard-coded policy names.)
-namespace {
-
-const char kKeyString[] = "HomepageLocation";
-const char kKeyBoolean[] = "HomepageIsNewTabPage";
-const char kKeyInteger[] = "NetworkPredictionOptions";
-const char kKeyStringList[] = "DisabledSchemes";
-const char kKeyDictionary[] = "ProxySettings";
-
-const char kTestChromeSchema[] =
-    "{"
-    "  \"type\": \"object\","
-    "  \"properties\": {"
-    "    \"HomepageLocation\": { \"type\": \"string\" },"
-    "    \"HomepageIsNewTabPage\": { \"type\": \"boolean\" },"
-    "    \"NetworkPredictionOptions\": { \"type\": \"integer\" },"
-    "    \"DisabledSchemes\": {"
-    "      \"type\": \"array\","
-    "      \"items\": { \"type\": \"string\" }"
-    "    },"
-    "    \"ProxySettings\": {"
-    "      \"type\": \"object\","
-    "      \"properties\": {"
-    "        \"bool\": { \"type\": \"boolean\" },"
-    "        \"double\": { \"type\": \"number\" },"
-    "        \"int\": { \"type\": \"integer\" },"
-    "        \"string\": { \"type\": \"string\" },"
-    "        \"array\": {"
-    "          \"type\": \"array\","
-    "          \"items\": { \"type\": \"string\" }"
-    "        },"
-    "        \"dictionary\": {"
-    "          \"type\": \"object\","
-    "          \"properties\": {"
-    "            \"sub\": { \"type\": \"string\" },"
-    "            \"sublist\": {"
-    "              \"type\": \"array\","
-    "              \"items\": {"
-    "                \"type\": \"object\","
-    "                \"properties\": {"
-    "                  \"aaa\": { \"type\": \"integer\" },"
-    "                  \"bbb\": { \"type\": \"integer\" },"
-    "                  \"ccc\": { \"type\": \"string\" },"
-    "                  \"ddd\": { \"type\": \"string\" }"
-    "                }"
-    "              }"
-    "            }"
-    "          }"
-    "        },"
-    "        \"list\": {"
-    "          \"type\": \"array\","
-    "          \"items\": {"
-    "            \"type\": \"object\","
-    "            \"properties\": {"
-    "              \"subdictindex\": { \"type\": \"integer\" },"
-    "              \"subdict\": {"
-    "                \"type\": \"object\","
-    "                \"properties\": {"
-    "                  \"bool\": { \"type\": \"boolean\" },"
-    "                  \"double\": { \"type\": \"number\" },"
-    "                  \"int\": { \"type\": \"integer\" },"
-    "                  \"string\": { \"type\": \"string\" }"
-    "                }"
-    "              }"
-    "            }"
-    "          }"
-    "        },"
-    "        \"dict\": {"
-    "          \"type\": \"object\","
-    "          \"properties\": {"
-    "            \"bool\": { \"type\": \"boolean\" },"
-    "            \"double\": { \"type\": \"number\" },"
-    "            \"int\": { \"type\": \"integer\" },"
-    "            \"string\": { \"type\": \"string\" },"
-    "            \"list\": {"
-    "              \"type\": \"array\","
-    "              \"items\": {"
-    "                \"type\": \"object\","
-    "                \"properties\": {"
-    "                  \"subdictindex\": { \"type\": \"integer\" },"
-    "                  \"subdict\": {"
-    "                    \"type\": \"object\","
-    "                    \"properties\": {"
-    "                      \"bool\": { \"type\": \"boolean\" },"
-    "                      \"double\": { \"type\": \"number\" },"
-    "                      \"int\": { \"type\": \"integer\" },"
-    "                      \"string\": { \"type\": \"string\" }"
-    "                    }"
-    "                  }"
-    "                }"
-    "              }"
-    "            }"
-    "          }"
-    "        }"
-    "      }"
-    "    }"
-    "  }"
-    "}";
-
-}  // anonymous namespace
-
 namespace policy {
 
 namespace {
 
 // Subdirectory of the config dir that contains mandatory policies.
 const base::FilePath::CharType kMandatoryPath[] = FILE_PATH_LITERAL("managed");
 
 class TestHarness : public PolicyProviderTestHarness {
  public:
   TestHarness();
@@ skipping 35 matching lines @@
   base::ScopedTempDir test_dir_;
   int next_policy_file_index_;
 
   DISALLOW_COPY_AND_ASSIGN(TestHarness);
 };
 
 TestHarness::TestHarness()
     : PolicyProviderTestHarness(POLICY_LEVEL_MANDATORY,
                                 POLICY_SCOPE_MACHINE,
                                 POLICY_SOURCE_PLATFORM),
-      next_policy_file_index_(100) {
-    key_string_ = kKeyString;
-    key_boolean_ = kKeyBoolean;
-    key_integer_ = kKeyInteger;
-    key_stringlist_ = kKeyStringList;
-    key_dictionary_ = kKeyDictionary;
-    test_schema_ = kTestChromeSchema;
-}
+      next_policy_file_index_(100) {}
 
 TestHarness::~TestHarness() {}
 
 void TestHarness::SetUp() {
   ASSERT_TRUE(test_dir_.CreateUniqueTempDir());
 }
 
 ConfigurationPolicyProvider* TestHarness::CreateProvider(
     SchemaRegistry* registry,
     scoped_refptr<base::SequencedTaskRunner> task_runner) {
@@ skipping 141 matching lines @@
   std::unique_ptr<PolicyBundle> bundle(loader.Load());
   ASSERT_TRUE(bundle.get());
   PolicyBundle expected_bundle;
   expected_bundle.Get(PolicyNamespace(POLICY_DOMAIN_CHROME, std::string()))
       .LoadFrom(&test_dict_foo, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
                 POLICY_SOURCE_PLATFORM);
   EXPECT_TRUE(bundle->Equals(expected_bundle));
 }
 
 }  // namespace policy