Sync: fix for the code that checks whether the initial download has completed

sync/test/engine/mock_connection_manager.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.
 //
 // Mock ServerConnectionManager class for use in client regression tests.
 
 #include "sync/test/engine/mock_connection_manager.h"
 
 #include <map>
 
@@ skipping 481 matching lines @@
 
 void MockConnectionManager::SetLastUpdatePosition(int64 server_position) {
   GetMutableLastUpdate()->set_position_in_parent(server_position);
 }
 
 void MockConnectionManager::SetNewTimestamp(int ts) {
   next_token_ = base::StringPrintf("mock connection ts = %d", ts);
   ApplyToken();
 }
 
+sync_pb::DataTypeProgressMarker*
+MockConnectionManager::AddUpdateProgressMarker() {
+  return GetUpdateResponse()->add_new_progress_marker();
+}
+
 void MockConnectionManager::ApplyToken() {
   if (!update_queue_.empty()) {
     GetUpdateResponse()->clear_new_progress_marker();
-    sync_pb::DataTypeProgressMarker* new_marker =
-        GetUpdateResponse()->add_new_progress_marker();
+    sync_pb::DataTypeProgressMarker* new_marker = AddUpdateProgressMarker();
     new_marker->set_data_type_id(-1);  // Invalid -- clients shouldn't see.
     new_marker->set_token(next_token_);
   }
 }
 
 void MockConnectionManager::SetChangesRemaining(int64 timestamp) {
   GetUpdateResponse()->set_changes_remaining(timestamp);
 }
 
 void MockConnectionManager::ProcessGetUpdates(
@@ skipping 250 matching lines @@
     server_status_ = HttpResponse::SERVER_CONNECTION_OK;
   }
 }
 
 void MockConnectionManager::SetServerStatus(
     HttpResponse::ServerConnectionCode server_status) {
   server_status_ = server_status;
 }
 
 }  // namespace syncer