Remove ListValue::Append(new {Fundamental,String}Value(...)) pattern in //sync

sync/internal_api/js_sync_encryption_handler_observer_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 "sync/internal_api/js_sync_encryption_handler_observer.h"
 
 #include "base/location.h"
 #include "base/run_loop.h"
 #include "base/values.h"
 #include "sync/internal_api/public/base/model_type.h"
@@ skipping 101 matching lines @@
   base::DictionaryValue expected_details;
   base::ListValue* encrypted_type_values = new base::ListValue();
   const bool encrypt_everything = false;
   expected_details.Set("encryptedTypes", encrypted_type_values);
   expected_details.SetBoolean("encryptEverything", encrypt_everything);
   ModelTypeSet encrypted_types;
 
   for (int i = FIRST_REAL_MODEL_TYPE; i < MODEL_TYPE_COUNT; ++i) {
     ModelType type = ModelTypeFromInt(i);
     encrypted_types.Put(type);
-    encrypted_type_values->Append(new base::StringValue(
-        ModelTypeToString(type)));
+    encrypted_type_values->AppendString(ModelTypeToString(type));
   }
 
   EXPECT_CALL(mock_js_event_handler_,
               HandleJsEvent("onEncryptedTypesChanged",
                             HasDetailsAsDictionary(expected_details)));
 
   js_sync_encryption_handler_observer_.OnEncryptedTypesChanged(
       encrypted_types, encrypt_everything);
   PumpLoop();
 }
@@ skipping 29 matching lines @@
               HandleJsEvent("onPassphraseTypeChanged",
                             HasDetailsAsDictionary(passphrase_type_details)));
 
   js_sync_encryption_handler_observer_.OnPassphraseTypeChanged(
       IMPLICIT_PASSPHRASE, ProtoTimeToTime(10));
   PumpLoop();
 }
 
 }  // namespace
 }  // namespace syncer