ChromeOS: Remove ThreadRestrictions::ScopedAllowIO from WizardController.

chrome/browser/chromeos/login/wizard_controller.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/chromeos/login/wizard_controller.h"
 
 #include <signal.h>
 #include <stdlib.h>
 #include <sys/types.h>
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_registry_simple.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_restrictions.h"

[Lei Zhang, 2015/02/09 19:34:45]

You can remove this now.

[Alexander Alekseev, 2015/02/10 12:04:35]

Done.

 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_process_platform_part.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
 #include "chrome/browser/chromeos/app_mode/kiosk_app_manager.h"
 #include "chrome/browser/chromeos/customization/customization_document.h"
 #include "chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.h"
 #include "chrome/browser/chromeos/login/enrollment/enrollment_screen.h"
 #include "chrome/browser/chromeos/login/existing_user_controller.h"
 #include "chrome/browser/chromeos/login/helper.h"
@@ skipping 99 matching lines @@
   histogram->AddTime(step_time);
 }
 
 bool IsRemoraRequisition() {
   return g_browser_process->platform_part()
       ->browser_policy_connector_chromeos()
       ->GetDeviceCloudPolicyManager()
       ->IsRemoraRequisition();
 }
 
+#if defined(GOOGLE_CHROME_BUILD)
+void InitializeCrashReporter() {
+  // The crash reporter initialization needs IO to complete.
+  DCHECK(BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
+  breakpad::InitCrashReporter(std::string());
+}
+#endif
+
 }  // namespace
 
 namespace chromeos {
 
 const char WizardController::kNetworkScreenName[] = "network";
 const char WizardController::kLoginScreenName[] = "login";
 const char WizardController::kUpdateScreenName[] = "update";
 const char WizardController::kUserImageScreenName[] = "image";
 const char WizardController::kEulaScreenName[] = "eula";
 const char WizardController::kEnableDebuggingScreenName[] = "debugging";
@@ skipping 437 matching lines @@
   } else {
     InitiateOOBEUpdate();
   }
 }
 
 void WizardController::InitiateMetricsReportingChangeCallback(bool enabled) {
   CrosSettings::Get()->SetBoolean(kStatsReportingPref, enabled);
   if (!enabled)
     return;
 #if defined(GOOGLE_CHROME_BUILD)
-  // The crash reporter initialization needs IO to complete.
-  base::ThreadRestrictions::ScopedAllowIO allow_io;
-  breakpad::InitCrashReporter(std::string());
+  if (!content::BrowserThread::PostBlockingPoolTask(
+          FROM_HERE, base::Bind(&InitializeCrashReporter))) {
+    LOG(ERROR) << "Failed to start crash reporter initialization.";
+  }
 #endif
-
 }
 
 void WizardController::OnUpdateErrorCheckingForUpdate() {
   // TODO(nkostylev): Update should be required during OOBE.
   // We do not want to block users from being able to proceed to the login
   // screen if there is any error checking for an update.
   // They could use "browse without sign-in" feature to set up the network to be
   // able to perform the update later.
   OnUpdateCompleted();
 }
@@ skipping 680 matching lines @@
   }
 
   EnrollmentScreen* screen = EnrollmentScreen::Get(this);
   screen->SetParameters(effective_config, shark_controller_.get(),
                         remora_controller_.get());
   SetStatusAreaVisible(true);
   SetCurrentScreen(screen);
 }
 
 }  // namespace chromeos