Change the modem reset command on cellular activation completion.

chrome/browser/chromeos/mobile/mobile_activator.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/mobile/mobile_activator.h"
 
 #include <algorithm>
 #include <map>
 #include <string>
 
@@ skipping 261 matching lines @@
 void MobileActivator::HandleSetTransactionStatus(bool success) {
   // The payment is received, try to reconnect and check the status all over
   // again.
   if (success && state_ == PLAN_ACTIVATION_SHOWING_PAYMENT) {
     NetworkLibrary* lib = GetNetworkLibrary();
     lib->SignalCellularPlanPayment();
     UMA_HISTOGRAM_COUNTS("Cellular.PaymentReceived", 1);
     CellularNetwork* network = FindMatchingCellularNetwork(true);
     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 not calling ResetModem here
anymore.

[rkc, 2013/02/15 22:53:14]

The comment isn't actually needed anymore since the function name is pretty
self-documenting - removing.

Done.

-      lib->ResetModem();
+      network->CompleteActivation();
       EvaluateCellularNetwork(network);
     } else {
       StartOTASP();
     }
   } else {
     UMA_HISTOGRAM_COUNTS("Cellular.PaymentFailed", 1);
   }
 }
 
 void MobileActivator::OnPortalLoaded(bool success) {
@@ skipping 660 matching lines @@
 
 std::string MobileActivator::GetErrorMessage(const std::string& code) const {
   return cellular_config_->GetErrorMessage(code);
 }
 
 NetworkLibrary* MobileActivator::GetNetworkLibrary() const {
   return CrosLibrary::Get()->GetNetworkLibrary();
 }
 
 }  // namespace chromeos