[Sync] Made some sync session member functions const

chrome/browser/sync/engine/process_updates_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/process_updates_command.h"
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/location.h"
@@ skipping 10 matching lines @@
 namespace browser_sync {
 
 using sessions::SyncSession;
 using sessions::StatusController;
 
 ProcessUpdatesCommand::ProcessUpdatesCommand() {}
 ProcessUpdatesCommand::~ProcessUpdatesCommand() {}
 
 bool ProcessUpdatesCommand::ModelNeutralExecuteImpl(SyncSession* session) {
   const GetUpdatesResponse& updates =
-      session->status_controller()->updates_response().get_updates();
+      session->status_controller().updates_response().get_updates();
   const int update_count = updates.entries_size();
 
   // Don't bother processing updates if there were none.
   return update_count != 0;
 }
 
 void ProcessUpdatesCommand::ModelChangingExecuteImpl(SyncSession* session) {
   syncable::ScopedDirLookup dir(session->context()->directory_manager(),
                                 session->context()->account_name());
   if (!dir.good()) {
     LOG(ERROR) << "Scoped dir lookup failed!";
     return;
   }
 
-  StatusController* status = session->status_controller();
+  const sessions::UpdateProgress* progress =
+      session->status_controller().update_progress();
+  if (!progress)
+    return;  // Nothing to do.
 
   syncable::WriteTransaction trans(FROM_HERE, syncable::SYNCER, dir);
-  const sessions::UpdateProgress& progress(status->update_progress());
   vector<sessions::VerifiedUpdate>::const_iterator it;
-  for (it = progress.VerifiedUpdatesBegin();
-       it != progress.VerifiedUpdatesEnd();
+  for (it = progress->VerifiedUpdatesBegin();
+       it != progress->VerifiedUpdatesEnd();
        ++it) {
     const sync_pb::SyncEntity& update = it->second;
 
     if (it->first != VERIFY_SUCCESS && it->first != VERIFY_UNDELETE)
       continue;
     switch (ProcessUpdate(dir, update, &trans)) {
       case SUCCESS_PROCESSED:
       case SUCCESS_STORED:
         break;
       default:
         NOTREACHED();
         break;
     }
   }
 
+  StatusController* status = session->mutable_status_controller();
   status->set_num_consecutive_errors(0);
-
   status->mutable_update_progress()->ClearVerifiedUpdates();
-  return;
 }
 
 namespace {
 // Returns true if the entry is still ok to process.
 bool ReverifyEntry(syncable::WriteTransaction* trans, const SyncEntity& entry,
                    syncable::MutableEntry* same_id) {
 
   const bool deleted = entry.has_deleted() && entry.deleted();
   const bool is_directory = entry.IsFolder();
   const syncable::ModelType model_type = entry.GetModelType();
@@ skipping 63 matching lines @@
     // Force application of this update, no matter what.
     target_entry.Put(syncable::IS_UNAPPLIED_UPDATE, true);
   }
 
   SyncerUtil::UpdateServerFieldsFromUpdate(&target_entry, update, name);
 
   return SUCCESS_PROCESSED;
 }
 
 }  // namespace browser_sync