Fix memory leak in Sync FakeServerEntity

sync/test/fake_server/unique_client_entity.cc

 // Copyright 2014 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/test/fake_server/unique_client_entity.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/guid.h"
@@ skipping 50 matching lines @@
       specifics_(specifics),
       creation_time_(creation_time),
       last_modified_time_(last_modified_time) { }
 
 string UniqueClientEntity::GetParentId() const {
   // The parent ID for this type of entity should always be its ModelType's
   // root node.
   return FakeServerEntity::GetTopLevelId(model_type_);
 }
 
-sync_pb::SyncEntity* UniqueClientEntity::SerializeAsProto() {
-  sync_pb::SyncEntity* sync_entity = new sync_pb::SyncEntity();
-  FakeServerEntity::SerializeBaseProtoFields(sync_entity);
+void UniqueClientEntity::SerializeAsProto(sync_pb::SyncEntity* proto) {
+  FakeServerEntity::SerializeBaseProtoFields(proto);
 
-  sync_pb::EntitySpecifics* specifics = sync_entity->mutable_specifics();
+  sync_pb::EntitySpecifics* specifics = proto->mutable_specifics();
   specifics->CopyFrom(specifics_);
 
-  sync_entity->set_parent_id_string(GetParentId());
-  sync_entity->set_client_defined_unique_tag(client_defined_unique_tag_);
-  sync_entity->set_ctime(creation_time_);
-  sync_entity->set_mtime(last_modified_time_);
-
-  return sync_entity;
+  proto->set_parent_id_string(GetParentId());
+  proto->set_client_defined_unique_tag(client_defined_unique_tag_);
+  proto->set_ctime(creation_time_);
+  proto->set_mtime(last_modified_time_);
 }
 
 bool UniqueClientEntity::IsDeleted() const {
   return false;
 }
 
 bool UniqueClientEntity::IsFolder() const {
   return false;
 }
 
 }  // namespace fake_server