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

Unified Diff: chrome/browser/chromeos/mobile/mobile_activator.cc

Issue 12279012: Change the modem reset command on cellular activation completion. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
« no previous file with comments | « chrome/browser/chromeos/cros/network_library.cc ('k') | chromeos/dbus/mock_shill_service_client.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/mobile/mobile_activator.cc
diff --git a/chrome/browser/chromeos/mobile/mobile_activator.cc b/chrome/browser/chromeos/mobile/mobile_activator.cc
index 0c4357f4daf51a3b31f903f0204743149986a2b9..4d0d5b75e62abc8a248075ad334e43caf4c7e510 100644
--- a/chrome/browser/chromeos/mobile/mobile_activator.cc
+++ b/chrome/browser/chromeos/mobile/mobile_activator.cc
@@ -280,7 +280,7 @@ void MobileActivator::HandleSetTransactionStatus(bool success) {
if (network && network->activate_over_non_cellular_network()) {
state_ = PLAN_ACTIVATION_DONE;
// We're done with activation, reset the cellular device.
Greg Spencer (Chromium) 2013/02/15 22:32:14 Might want to update this comment, since you're no
rkc 2013/02/15 22:53:14 The comment isn't actually needed anymore since th
- lib->ResetModem();
+ network->CompleteActivation();
EvaluateCellularNetwork(network);
} else {
StartOTASP();
« no previous file with comments | « chrome/browser/chromeos/cros/network_library.cc ('k') | chromeos/dbus/mock_shill_service_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698