Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Unified Diff: chrome/browser/chromeos/login/wizard_controller.cc

Issue 2694923005: Appropriate handling for Cellular First devices. (Closed)
Patch Set: made a couple of tweaks Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/wizard_controller.cc
diff --git a/chrome/browser/chromeos/login/wizard_controller.cc b/chrome/browser/chromeos/login/wizard_controller.cc
index a5be676dd96b5d2a4fd8ebc160019636e0b73720..c6862a4c707481d0a57616ab6380ddbbb5fcd272 100644
--- a/chrome/browser/chromeos/login/wizard_controller.cc
+++ b/chrome/browser/chromeos/login/wizard_controller.cc
@@ -890,6 +890,20 @@ void WizardController::InitiateOOBEUpdate() {
return;
}
+ // If this is a Cellular First device, instruct UpdateEngine to allow
+ // updates over cellular data connections.
+ if (chromeos::switches::IsCellularFirstDevice()) {
+ DBusThreadManager::Get()
+ ->GetUpdateEngineClient()
+ ->SetUpdateOverCellularPermission(
+ true, base::Bind(&WizardController::InitiateOOBEUpdateHelper,
+ weak_factory_.GetWeakPtr()));
+ } else {
+ InitiateOOBEUpdateHelper();
+ }
+}
+
+void WizardController::InitiateOOBEUpdateHelper() {
VLOG(1) << "InitiateOOBEUpdate";
stevenjb 2017/03/01 23:11:56 We should make this method more descriptive (e.g.
kumarniranjan 2017/03/02 00:29:34 Done.
SetCurrentScreenSmooth(GetScreen(OobeScreen::SCREEN_OOBE_UPDATE), true);
UpdateScreen::Get(this)->StartNetworkCheck();

Powered by Google App Engine
This is Rietveld 408576698