Rename ArcApplicationPolicy to ArcPolicy

chrome/browser/chromeos/arc/arc_policy_bridge_unittest.cc

 // Copyright 2016 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 <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/arc/arc_policy_bridge.h"
@@ skipping 56 matching lines @@
   policy::MockPolicyService policy_service_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcPolicyBridgeTest);
 };
 
 TEST_F(ArcPolicyBridgeTest, EmptyPolicyTest) {
   // No policy is set, result should be empty.
   policy_bridge()->GetPolicies(PolicyStringCallback("{}"));
 }
 
-TEST_F(ArcPolicyBridgeTest, ArcApplicationPolicyTest) {
+TEST_F(ArcPolicyBridgeTest, ArcPolicyTest) {
   policy_map().Set(
-      "ArcApplicationPolicy", policy::POLICY_LEVEL_MANDATORY,
-      policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
+      "ArcPolicy", policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER,
+      policy::POLICY_SOURCE_CLOUD,
       new base::StringValue(
-          "{\"applications\":"
-              "[{\"packageName\":\"com.google.android.apps.youtube.kids\","
-                "\"installType\":\"REQUIRED\","
-                "\"lockTaskAllowed\":false,"
-                "\"permissionGrants\":[]"
-              "}],"
+          "{\"applications\": [{"
+          "    \"packageName\":\"com.google.android.apps.youtube.kids\","
+          "    \"installType\":\"REQUIRED\","
+          "    \"lockTaskAllowed\":false,"
+          "    \"permissionGrants\":[]"
+          "}],"

[hidehiko, 2016/04/12 06:55:42]

nit: indent looks not aligned. Ditto for below.

[bartfab (slow), 2016/04/12 13:28:15]

I had tried to reformat things to make clang-format happy. However, I then
realized that I actually cannot do that as the whole concatenated string is
compared verbatim to the value returned by the policy bridge. I cannot add any
whitespace or formatting inside the strings. I reverted them back to what they
were before.

           "\"defaultPermissionPolicy\":\"GRANT\""
           "}"),
       nullptr);
   policy_bridge()->GetPolicies(PolicyStringCallback(
-      "{\"applications\":"
-          "[{\"installType\":\"REQUIRED\","
-            "\"lockTaskAllowed\":false,"
-            "\"packageName\":\"com.google.android.apps.youtube.kids\","
-            "\"permissionGrants\":[]"
-          "}],"
+      "{\"applications\": [{"
+      "    \"installType\":\"REQUIRED\","
+      "    \"lockTaskAllowed\":false,"
+      "    \"packageName\":\"com.google.android.apps.youtube.kids\","
+      "    \"permissionGrants\":[]"
+      "}],"
       "\"defaultPermissionPolicy\":\"GRANT\""
       "}"));
 }
 
 TEST_F(ArcPolicyBridgeTest, HompageLocationTest) {
   // This policy will not be passed on, result should be empty.
   policy_map().Set("HomepageLocation", policy::POLICY_LEVEL_MANDATORY,
                    policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
                    new base::StringValue("http://chromium.org"), nullptr);
   policy_bridge()->GetPolicies(PolicyStringCallback("{}"));
 }
 
 TEST_F(ArcPolicyBridgeTest, VideoCaptureAllowedTest) {
   policy_map().Set("VideoCaptureAllowed", policy::POLICY_LEVEL_MANDATORY,
                    policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
                    new base::FundamentalValue(false), nullptr);
   policy_bridge()->GetPolicies(
       PolicyStringCallback("{\"cameraDisabled\":true}"));
 }
 
 TEST_F(ArcPolicyBridgeTest, AllPoliciesTest) {
   // Keep them in alphabetical order.
   policy_map().Set(
-      "ArcApplicationPolicy", policy::POLICY_LEVEL_MANDATORY,
-      policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
-      new base::StringValue("{\"applications\":"
-              "[{\"packageName\":\"com.google.android.apps.youtube.kids\","
-                "\"installType\":\"REQUIRED\","
-                "\"lockTaskAllowed\":false,"
-                "\"permissionGrants\":[]"
-              "}],"
+      "ArcPolicy", policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER,
+      policy::POLICY_SOURCE_CLOUD,
+      new base::StringValue(
+          "{\"applications\": [{"
+          "    \"packageName\":\"com.google.android.apps.youtube.kids\","
+          "    \"installType\":\"REQUIRED\","
+          "    \"lockTaskAllowed\":false,"
+          "    \"permissionGrants\":[]"
+          "}],"
           "\"defaultPermissionPolicy\":\"GRANT\"}"),
       nullptr);
   policy_map().Set("HomepageLocation", policy::POLICY_LEVEL_MANDATORY,
                    policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
                    new base::StringValue("http://chromium.org"), nullptr);
   policy_map().Set("VideoCaptureAllowed", policy::POLICY_LEVEL_MANDATORY,
                    policy::POLICY_SCOPE_USER, policy::POLICY_SOURCE_CLOUD,
                    new base::FundamentalValue(false), nullptr);
   policy_bridge()->GetPolicies(PolicyStringCallback(
-      "{\"applications\":"
-          "[{\"installType\":\"REQUIRED\","
-            "\"lockTaskAllowed\":false,"
-            "\"packageName\":\"com.google.android.apps.youtube.kids\","
-            "\"permissionGrants\":[]"
-          "}],"
-        "\"cameraDisabled\":true,"
-        "\"defaultPermissionPolicy\":\"GRANT\""
+      "{\"applications\": [{"
+      "    \"installType\":\"REQUIRED\","
+      "    \"lockTaskAllowed\":false,"
+      "    \"packageName\":\"com.google.android.apps.youtube.kids\","
+      "    \"permissionGrants\":[]"
+      "}],"
+      "\"cameraDisabled\":true,"
+      "\"defaultPermissionPolicy\":\"GRANT\""
       "}"));
 }
 
 }  // namespace arc