Refactor following sync commit loop change

sync/engine/build_commit_command.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/engine/build_commit_command.h"
 
 #include <limits>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 106 matching lines @@
   // space of positions that are immediate successors of the item
   // whose ID is the map's key.
   std::map<Id, std::pair<int64, int64> > position_map;
 
   for (size_t i = 0; i < batch_commit_set_.Size(); i++) {
     Id id = batch_commit_set_.GetCommitIdAt(i);
     SyncEntity* sync_entry =
         static_cast<SyncEntity*>(commit_message->add_entries());
     sync_entry->set_id(id);
     MutableEntry meta_entry(session->write_transaction(),
-                            syncable::GET_BY_ID,
-                            id);
+                            syncable::GET_BY_ID, id);
     CHECK(meta_entry.good());
-    // This is the only change we make to the entry in this function.
-    meta_entry.Put(syncable::SYNCING, true);
 
     DCHECK(0 != session->routing_info().count(meta_entry.GetModelType()))
         << "Committing change to datatype that's not actively enabled.";
 
     string name = meta_entry.Get(syncable::NON_UNIQUE_NAME);
     CHECK(!name.empty());  // Make sure this isn't an update.
     TruncateUTF8ToByteSize(name, 255, &name);
     sync_entry->set_name(name);
 
     // Set the non_unique_name.  If we do, the server ignores
@@ skipping 114 matching lines @@
   if (delta <= static_cast<uint64>(GetGap()*2))
     return lo + (static_cast<int64>(delta) + 7) / 8;  // Interpolate.
   else if (lo == GetFirstPosition())
     return hi - GetGap();  // Extend range just before successor.
   else
     return lo + GetGap();  // Use or extend range just after predecessor.
 }
 
 
 }  // namespace browser_sync