Populate versions on individual nodes in sync model and native bookmark model.

sync/syncable/write_transaction.h

 // 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.
 
 #ifndef SYNC_SYNCABLE_WRITE_TRANSACTION_H_
 #define SYNC_SYNCABLE_WRITE_TRANSACTION_H_
 
 #include "sync/syncable/base_transaction.h"
 #include "sync/syncable/entry_kernel.h"
 
 namespace syncer {
 namespace syncable {
 
 // Locks db in constructor, unlocks in destructor.
 class WriteTransaction : public BaseTransaction {
  public:
   WriteTransaction(const tracked_objects::Location& from_here,
                    WriterTag writer, Directory* directory);
 
+  // Constructor used for getting back model version after making sync
+  // API changes to one model. After the transaction is destroyed,

[Nicolas Zea, 2012/11/02 21:02:25]

nit: "After the transaction is destroyed" -> "During destruction, "

[haitaol1, 2012/11/02 22:58:18]

Done.

+  // |new_model_version| stores the new model version if model is changed,
+  // or -1 if model is not changed.
+  WriteTransaction(const tracked_objects::Location& from_here,
+                   Directory* directory, int64* new_model_version);
+
   virtual ~WriteTransaction();
 
   void SaveOriginal(const EntryKernel* entry);
 
  protected:
   // Overridden by tests.
   virtual void NotifyTransactionComplete(ModelTypeSet models_with_changes);
 
  private:
   friend class MutableEntry;
 
   // Clears |mutations_|.
   ImmutableEntryKernelMutationMap RecordMutations();
 
   void UnlockAndNotify(const ImmutableEntryKernelMutationMap& mutations);
 
   ModelTypeSet NotifyTransactionChangingAndEnding(
       const ImmutableEntryKernelMutationMap& mutations);
 
+  // Increment versions of the models whose entries are modified and set the
+  // version on the changed entries.
+  void UpdateTransactionVersion(const std::vector<int64>& entry_changed);
+
   // Only the original fields are filled in until |RecordMutations()|.
   // We use a mutation map instead of a kernel set to avoid copying.
   EntryKernelMutationMap mutations_;
 
+  int64* new_model_version_;

[Nicolas Zea, 2012/11/02 21:02:25]

Comment that you don't have ownership of this.

[haitaol1, 2012/11/02 22:58:18]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(WriteTransaction);
 };
 
 }  // namespace syncable
 }  // namespace syncer
 
 #endif //  SYNC_SYNCABLE_WRITE_TRANSACTION_H_