Fix Precise build.

chrome/browser/sync/test_profile_sync_service.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 "chrome/browser/sync/test_profile_sync_service.h"
 
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/sync/abstract_profile_sync_service_test.h"
 #include "chrome/browser/sync/glue/data_type_controller.h"
 #include "chrome/browser/sync/glue/sync_backend_host.h"
@@ skipping 87 matching lines @@
   FinishConfigureDataTypesOnFrontendLoop(failed_configuration_types,
                                          ready_task);
 }
 
 void SyncBackendHostForProfileSyncTest
         ::HandleSyncManagerInitializationOnFrontendLoop(
     const syncer::WeakHandle<syncer::JsBackend>& js_backend, bool success,
     syncer::ModelTypeSet restored_types) {
   // Here's our opportunity to pretend to do things that the SyncManager would
   // normally do during initialization, but can't because this is a test.
-  bool send_passphrase_required = false;
   if (success) {
     // Set up any nodes the test wants around before model association.
     if (!callback_.is_null()) {
       callback_.Run();
     }
 
     // Pretend we downloaded initial updates and set initial sync ended bits
     // if we were asked to.
     if (set_initial_sync_ended_on_init_) {
       UserShare* user_share = GetUserShare();
       Directory* directory = user_share->directory.get();
 
       if (!directory->initial_sync_ended_for_type(syncer::NIGORI)) {

[rlarocque, 2012/07/30 20:55:33]

It's possible this entire if block could be removed.  I'd have to try it to be
sure, though.

[dcheng, 2012/07/30 20:58:19]

What would I need to do to verify that removing this block is safe? Would
running sync_integration_tests be enough?

         ProfileSyncServiceTestHelper::CreateRoot(
             syncer::NIGORI, user_share, &id_factory_);
 
         // A side effect of adding the NIGORI mode (normally done by the
         // syncer) is a decryption attempt, which will fail the first time.
-        send_passphrase_required = true;
       }
 
       SetInitialSyncEndedForAllTypes();
       restored_types = syncer::ModelTypeSet::All();
     }
   }
 
   SyncBackendHost::HandleSyncManagerInitializationOnFrontendLoop(
       js_backend, success, restored_types);
 }
@@ skipping 89 matching lines @@
       profile(),
       sync_prefs_.AsWeakPtr(),
       invalidator_storage_.AsWeakPtr(),
       id_factory_,
       callback_,
       set_initial_sync_ended_on_init_,
       synchronous_backend_initialization_,
       fail_initial_download_,
       use_real_database_));
 }