Rewrite simple uses of base::ListValue::Append(base::Value*) on CrOS.

chrome/browser/chromeos/policy/device_policy_decoder_chromeos.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 "chrome/browser/chromeos/policy/device_policy_decoder_chromeos.h"
 
 #include <limits>
 #include <string>
 
 #include "base/callback.h"
@@ skipping 143 matching lines @@
           POLICY_SCOPE_MACHINE, POLICY_SOURCE_CLOUD,
           base::MakeUnique<base::FundamentalValue>(container.allow_new_users()),
           nullptr);
     }
   }
 
   if (policy.has_user_whitelist()) {
     const em::UserWhitelistProto& container(policy.user_whitelist());
     std::unique_ptr<base::ListValue> whitelist(new base::ListValue);
     for (const auto& entry : container.user_whitelist())
-      whitelist->Append(new base::StringValue(entry));
+      whitelist->AppendString(entry);
     policies->Set(key::kDeviceUserWhitelist, POLICY_LEVEL_MANDATORY,
                   POLICY_SCOPE_MACHINE, POLICY_SOURCE_CLOUD,
                   std::move(whitelist), nullptr);
   }
 
   if (policy.has_ephemeral_users_enabled()) {
     const em::EphemeralUsersEnabledProto& container(
         policy.ephemeral_users_enabled());
     if (container.has_ephemeral_users_enabled()) {
       policies->Set(key::kDeviceEphemeralUsersEnabled, POLICY_LEVEL_MANDATORY,
@@ skipping 120 matching lines @@
           base::MakeUnique<base::FundamentalValue>(container.allow_bluetooth()),
           nullptr);
     }
   }
 
   if (policy.has_login_video_capture_allowed_urls()) {
     const em::LoginVideoCaptureAllowedUrlsProto& container(
         policy.login_video_capture_allowed_urls());
     std::unique_ptr<base::ListValue> urls(new base::ListValue());
     for (const auto& entry : container.urls()) {
-      urls->Append(new base::StringValue(entry));
+      urls->AppendString(entry);
     }
     policies->Set(key::kLoginVideoCaptureAllowedUrls, POLICY_LEVEL_MANDATORY,
                   POLICY_SCOPE_MACHINE, POLICY_SOURCE_CLOUD, std::move(urls),
                   nullptr);
   }
 
   if (policy.has_login_apps()) {
     const em::LoginAppsProto& login_apps_proto(policy.login_apps());
     std::unique_ptr<base::ListValue> login_apps(new base::ListValue);
     for (const auto& login_app : login_apps_proto.login_apps())
-      login_apps->Append(new base::StringValue(login_app));
+      login_apps->AppendString(login_app);
     policies->Set(key::kLoginApps, POLICY_LEVEL_MANDATORY, POLICY_SCOPE_MACHINE,
                   POLICY_SOURCE_CLOUD, std::move(login_apps), nullptr);
   }
 }
 
 void DecodeNetworkPolicies(const em::ChromeDeviceSettingsProto& policy,
                            PolicyMap* policies) {
   if (policy.has_data_roaming_enabled()) {
     const em::DataRoamingEnabledProto& container(policy.data_roaming_enabled());
     if (container.has_data_roaming_enabled()) {
@@ skipping 487 matching lines @@
   // Decode the various groups of policies.
   DecodeLoginPolicies(policy, policies);
   DecodeNetworkPolicies(policy, policies);
   DecodeReportingPolicies(policy, policies);
   DecodeAutoUpdatePolicies(policy, policies);
   DecodeAccessibilityPolicies(policy, policies);
   DecodeGenericPolicies(policy, policies);
 }
 
 }  // namespace policy