sync: Start moving away from PREV_ID and NEXT_ID

This change replaces serveral instances of Get(PREV_ID) and Get(NEXT_ID)
with calls to the newly introduced GetPredecessorId() and
GetSuccessorId().

This is done in anticipation of the change that will remove PREV_ID and
NEXT_ID entirely.  By making this change in advance, that patch will be
smaller and easier to review.

BUG=145412,126505


Review URL: https://chromiumcodereview.appspot.com/11637053

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175599 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 3 deletions

diff --git a/sync/syncable/syncable_util.cc b/sync/syncable/syncable_util.cc
index 0597109..857fc85 100644
--- a/sync/syncable/syncable_util.cc
+++ b/sync/syncable/syncable_util.cc
@@ -78,14 +78,14 @@ void ChangeEntryIDAndUpdateChildren(
   // order.  Do this by reinserting into the linked list; the first
   // step in PutPredecessor is to Unlink from the existing order, which
   // will overwrite the stale Id value from the adjacent nodes.
-  if (entry->Get(PREV_ID) == entry->Get(NEXT_ID) &&
-      entry->Get(PREV_ID) == old_id) {
+  if (entry->GetPredecessorId() == entry->GetSuccessorId() &&
+      entry->GetPredecessorId() == old_id) {
     // We just need a shallow update to |entry|'s fields since it is already
     // self looped.
     entry->Put(NEXT_ID, new_id);
     entry->Put(PREV_ID, new_id);
   } else {
-    entry->PutPredecessor(entry->Get(PREV_ID));
+    entry->PutPredecessor(entry->GetPredecessorId());
   }
 }