[Sync] Made some sync session member functions const

chrome/browser/sync/engine/build_commit_command.cc

 // Copyright (c) 2011 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 "chrome/browser/sync/engine/build_commit_command.h"
 
 #include <limits>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 39 matching lines @@
 }
 
 BuildCommitCommand::BuildCommitCommand() {}
 BuildCommitCommand::~BuildCommitCommand() {}
 
 void BuildCommitCommand::AddExtensionsActivityToMessage(
     SyncSession* session, CommitMessage* message) {
   // We only send ExtensionsActivity to the server if bookmarks are being
   // committed.
   ExtensionsActivityMonitor* monitor = session->context()->extensions_monitor();
-  if (!session->status_controller()->HasBookmarkCommitActivity()) {
+  if (!session->status_controller().HasBookmarkCommitActivity()) {
     // Return the records to the activity monitor.
     monitor->PutRecords(session->extensions_activity());
     session->mutable_extensions_activity()->clear();
     return;
   }
   const ExtensionsActivityMonitor::Records& records =
       session->extensions_activity();
   for (ExtensionsActivityMonitor::Records::const_iterator it = records.begin();
        it != records.end(); ++it) {
     sync_pb::ChromiumExtensionsActivity* activity_message =
@@ skipping 27 matching lines @@
       session->write_transaction()->directory(), &message);
 
   // Cache previously computed position values.  Because |commit_ids|
   // is already in sibling order, we should always hit this map after
   // the first sibling in a consecutive run of commit items.  The
   // entries in this map are (low, high) values describing the
   // 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;
 
-  const vector<Id>& commit_ids = session->status_controller()->commit_ids();
+  const vector<Id>& commit_ids = session->status_controller().commit_ids();
   for (size_t i = 0; i < commit_ids.size(); i++) {
     Id id = commit_ids[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);
     CHECK(meta_entry.good());
     // This is the only change we make to the entry in this function.
@@ skipping 82 matching lines @@
         }
         position_block.first = InterpolatePosition(position_block.first,
                                                    position_block.second);
 
         position_map[id] = position_block;
         sync_entry->set_position_in_parent(position_block.first);
       }
       SetEntrySpecifics(&meta_entry, sync_entry);
     }
   }
-  session->status_controller()->mutable_commit_message()->CopyFrom(message);
+  session->mutable_status_controller()->
+      mutable_commit_message()->CopyFrom(message);
 }
 
 int64 BuildCommitCommand::FindAnchorPosition(syncable::IdField direction,
                                              const syncable::Entry& entry) {
   Id next_id = entry.Get(direction);
   while (!next_id.IsRoot()) {
     Entry next_entry(entry.trans(),
                      syncable::GET_BY_ID,
                      next_id);
     if (!next_entry.Get(IS_UNSYNCED) && !next_entry.Get(IS_UNAPPLIED_UPDATE)) {
@@ skipping 21 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