sync: Start moving away from PREV_ID and NEXT_ID

sync/internal_api/write_node.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/internal_api/public/write_node.h"
 
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "sync/internal_api/public/base_transaction.h"
 #include "sync/internal_api/public/write_transaction.h"
@@ skipping 443 matching lines @@
   // |predecessor| must be a child of |new_parent| or NULL.
   if (predecessor && predecessor->GetParentId() != new_parent.GetId()) {
     DCHECK(false);
     return false;
   }
 
   syncable::Id new_parent_id = new_parent.GetEntry()->Get(syncable::ID);
 
   // Filter out redundant changes if both the parent and the predecessor match.
   if (new_parent_id == entry_->Get(syncable::PARENT_ID)) {
-    const syncable::Id& old = entry_->Get(syncable::PREV_ID);
+    const syncable::Id& old = entry_->GetPredecessorId();
     if ((!predecessor && old.IsRoot()) ||
         (predecessor && (old == predecessor->GetEntry()->Get(syncable::ID)))) {
       return true;
     }
   }
 
   // Atomically change the parent. This will fail if it would
   // introduce a cycle in the hierarchy.
   if (!entry_->Put(syncable::PARENT_ID, new_parent_id))
     return false;
@@ skipping 31 matching lines @@
   MarkForSyncing();
 
   return true;
 }
 
 void WriteNode::MarkForSyncing() {
   syncable::MarkForSyncing(entry_);
 }
 
 }  // namespace syncer