[Sync] Rework SharedValue<T> into Immutable<T>

chrome/browser/sync/api/sync_data.cc

 // Copyright (c) 2011 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/sync/api/sync_data.h"
 
 #include "chrome/browser/sync/protocol/sync.pb.h"
 
-SyncData::SharedSyncEntity::SharedSyncEntity(
-    sync_pb::SyncEntity* sync_entity)
-    : sync_entity_(new sync_pb::SyncEntity()){
-  sync_entity_->Swap(sync_entity);
+void SyncData::ImmutableSyncEntityTraits::CreateAndDefaultInitialize(
+    Container* container) {
+  *container = new sync_pb::SyncEntity();
 }
 
-const sync_pb::SyncEntity& SyncData::SharedSyncEntity::sync_entity() const {
-  return *sync_entity_;
+void SyncData::ImmutableSyncEntityTraits::Destroy(Container* container) {
+  delete *container;
 }
 
-SyncData::SharedSyncEntity::~SharedSyncEntity() {}
-
-
-SyncData::SyncData()
-    : is_local_(true) {
+const sync_pb::SyncEntity& SyncData::ImmutableSyncEntityTraits::Unwrap(
+    const Container& container) {
+  return *container;
 }
 
-SyncData::~SyncData() {
+sync_pb::SyncEntity* SyncData::ImmutableSyncEntityTraits::UnwrapMutable(
+    Container* container) {
+  return *container;
 }
 
+void SyncData::ImmutableSyncEntityTraits::Swap(sync_pb::SyncEntity* t1,
+                                               sync_pb::SyncEntity* t2) {
+  t1->Swap(t2);
+}
+
+SyncData::SyncData() : is_valid_(false), is_local_(true) {}
+
+SyncData::SyncData(sync_pb::SyncEntity* entity, bool is_local)
+    : is_valid_(false), is_local_(is_local), immutable_entity_(entity) {}
+
+SyncData::~SyncData() {}
+
 // Static.
 SyncData SyncData::CreateLocalData(const std::string& sync_tag) {
   sync_pb::SyncEntity entity;
   entity.set_client_defined_unique_tag(sync_tag);
-  SyncData a;
-  a.shared_entity_ = new SharedSyncEntity(&entity);
-  a.is_local_ = true;
-  return a;
+  return SyncData(&entity, true);
 }
 
 // Static.
 SyncData SyncData::CreateLocalData(
     const std::string& sync_tag,
     const std::string& non_unique_title,
     const sync_pb::EntitySpecifics& specifics) {
   sync_pb::SyncEntity entity;
   entity.set_client_defined_unique_tag(sync_tag);
   entity.set_non_unique_name(non_unique_title);
   entity.mutable_specifics()->CopyFrom(specifics);
-  SyncData a;
-  a.shared_entity_ = new SharedSyncEntity(&entity);
-  a.is_local_ = true;
-  return a;
+  return SyncData(&entity, true);
 }
 
 // Static.
 SyncData SyncData::CreateRemoteData(const sync_pb::SyncEntity& entity) {
   // TODO(zea): eventually use this for building changes from the original sync
   // entities if possible.
   NOTIMPLEMENTED();
   return SyncData();
 }
 
 // Static.
 SyncData SyncData::CreateRemoteData(
     const sync_pb::EntitySpecifics& specifics) {
   sync_pb::SyncEntity entity;
   entity.mutable_specifics()->CopyFrom(specifics);
-  SyncData a;
-  a.shared_entity_ = new SharedSyncEntity(&entity);
-  a.is_local_ = false;
-  return a;
+  return SyncData(&entity, false);
 }
 
 bool SyncData::IsValid() const {
-  return (shared_entity_.get() != NULL);
+  return is_valid_;
 }
 
 const sync_pb::EntitySpecifics& SyncData::GetSpecifics() const {
-  return shared_entity_->sync_entity().specifics();
+  return immutable_entity_.Get().specifics();
 }
 
 syncable::ModelType SyncData::GetDataType() const {
   return syncable::GetModelTypeFromSpecifics(GetSpecifics());
 }
 
 const std::string& SyncData::GetTag() const {
   DCHECK(is_local_);
-  return shared_entity_->sync_entity().client_defined_unique_tag();
+  return immutable_entity_.Get().client_defined_unique_tag();
 }
 
 const std::string& SyncData::GetTitle() const {
   // TODO(zea): set this for data coming from the syncer too.
-  DCHECK(shared_entity_->sync_entity().has_non_unique_name());
-  return shared_entity_->sync_entity().non_unique_name();
+  DCHECK(immutable_entity_.Get().has_non_unique_name());
+  return immutable_entity_.Get().non_unique_name();
 }
 
 bool SyncData::IsLocal() const {
   return is_local_;
 }