sync: Start moving away from PREV_ID and NEXT_ID

sync/syncable/mutable_entry.cc

 // 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.
 
 #include "sync/syncable/mutable_entry.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "sync/internal_api/public/base/node_ordinal.h"
 #include "sync/syncable/directory.h"
 #include "sync/syncable/scoped_index_updater.h"
@@ skipping 371 matching lines @@
   Id successor_id;
   if (!predecessor_id.IsRoot()) {
     MutableEntry predecessor(write_transaction(), GET_BY_ID, predecessor_id);
     if (!predecessor.good()) {
       LOG(ERROR) << "Predecessor is not good : "
                  << predecessor_id.GetServerId();
       return false;
     }
     if (predecessor.Get(PARENT_ID) != Get(PARENT_ID))
       return false;
-    successor_id = predecessor.Get(NEXT_ID);
+    successor_id = predecessor.GetSuccessorId();
     predecessor.Put(NEXT_ID, Get(ID));
   } else {
     syncable::Directory* dir = trans()->directory();
     if (!dir->GetFirstChildId(trans(), Get(PARENT_ID), &successor_id)) {
       return false;
     }
   }
   if (!successor_id.IsRoot()) {
     MutableEntry successor(write_transaction(), GET_BY_ID, successor_id);
     if (!successor.good()) {
@@ skipping 23 matching lines @@
   DCHECK_NE(static_cast<MutableEntry*>(NULL), e);
   DCHECK(!e->IsRoot()) << "We shouldn't mark a permanent object for syncing.";
   if (!(e->Put(IS_UNSYNCED, true)))
     return false;
   e->Put(SYNCING, false);
   return true;
 }
 
 }  // namespace syncable
 }  // namespace syncer