sync: Refactor MutablEntry internals

sync/syncable/mutable_entry.h

 // Copyright 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.
 
 #ifndef SYNC_SYNCABLE_MUTABLE_ENTRY_H_
 #define SYNC_SYNCABLE_MUTABLE_ENTRY_H_
 
 #include "sync/base/sync_export.h"
 #include "sync/internal_api/public/base/model_type.h"
 #include "sync/syncable/entry.h"
@@ skipping 44 matching lines @@
   void PutServerMtime(base::Time value);
   void PutCtime(base::Time value);
   void PutServerCtime(base::Time value);
   bool PutId(const Id& value);
   void PutParentId(const Id& value);
   void PutServerParentId(const Id& value);
   bool PutIsUnsynced(bool value);
   bool PutIsUnappliedUpdate(bool value);
   void PutIsDir(bool value);
   void PutServerIsDir(bool value);
-  bool PutIsDel(bool value);
+  void PutIsDel(bool value);
   void PutServerIsDel(bool value);
   void PutNonUniqueName(const std::string& value);
   void PutServerNonUniqueName(const std::string& value);
   bool PutUniqueServerTag(const std::string& value);
   bool PutUniqueClientTag(const std::string& value);
   void PutUniqueBookmarkTag(const std::string& tag);
   void PutSpecifics(const sync_pb::EntitySpecifics& value);
   void PutServerSpecifics(const sync_pb::EntitySpecifics& value);
   void PutBaseServerSpecifics(const sync_pb::EntitySpecifics& value);
   void PutUniquePosition(const UniquePosition& value);
@@ skipping 24 matching lines @@
   void UpdateTransactionVersion(int64 version);
 
  protected:
   syncable::MetahandleSet* GetDirtyIndexHelper();
 
  private:
   friend class Directory;
   friend class WriteTransaction;
   friend class syncer::WriteNode;
 
-  bool Put(Int64Field field, const int64& value);
-  bool Put(TimeField field, const base::Time& value);
-  bool Put(IdField field, const Id& value);
-  bool Put(StringField field, const std::string& value);
-  bool Put(BaseVersion field, int64 value);
-  bool Put(ProtoField field, const sync_pb::EntitySpecifics& value);
-  bool Put(BitTemp field, bool value);
-  bool Put(BitField field, bool value);
-  bool Put(IndexedBitField field, bool value);
-  bool Put(UniquePositionField field, const UniquePosition& value);
-
   // Don't allow creation on heap, except by sync API wrappers.
   void* operator new(size_t size) { return (::operator new)(size); }
 
   // Adjusts the successor and predecessor entries so that they no longer
   // refer to this entry.
   bool UnlinkFromOrder();
 
   // Kind of redundant. We should reduce the number of pointers
   // floating around if at all possible. Could we store this in Directory?
   // Scope: Set on construction, never changed after that.
   WriteTransaction* const write_transaction_;
 
  protected:
   MutableEntry();
 
   DISALLOW_COPY_AND_ASSIGN(MutableEntry);
 };
 
 // This function sets only the flags needed to get this entry to sync.
 bool MarkForSyncing(syncable::MutableEntry* e);
 
 }  // namespace syncable
 }  // namespace syncer
 
 #endif  // SYNC_SYNCABLE_MUTABLE_ENTRY_H_