Change the modem reset command on cellular activation completion.

Change the modem reset command on cellular activation completion.
Instead of calling ModemReset on completing activation for an LTE modem, instead call the new CompleteCellularActivation shill service method.


R=gspencer@chromium.org,benchan@chromium.org
BUG=172526
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182956

[Greg Spencer (Chromium), 2013-02-15 22:32:14]

LGTM w/ a nit

https://codereview.chromium.org/12279012/diff/1/chrome/browser/chromeos/mobil...
File chrome/browser/chromeos/mobile/mobile_activator.cc (right):

https://codereview.chromium.org/12279012/diff/1/chrome/browser/chromeos/mobil...
chrome/browser/chromeos/mobile/mobile_activator.cc:282: // We're done with
activation, reset the cellular device.
Might want to update this comment, since you're not calling ResetModem here
anymore.

[rkc, 2013-02-15 22:53:13]

https://codereview.chromium.org/12279012/diff/1/chrome/browser/chromeos/mobil...
File chrome/browser/chromeos/mobile/mobile_activator.cc (right):

https://codereview.chromium.org/12279012/diff/1/chrome/browser/chromeos/mobil...
chrome/browser/chromeos/mobile/mobile_activator.cc:282: // We're done with
activation, reset the cellular device.
On 2013/02/15 22:32:14, Greg Spencer (Chromium) wrote:
> Might want to update this comment, since you're not calling ResetModem here
> anymore.
The comment isn't actually needed anymore since the function name is pretty
self-documenting - removing.

Done.

[zel, 2013-02-15 22:55:21]

lgtm

[commit-bot: I haz the power, 2013-02-15 22:57:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rkc@chromium.org/12279012/3001

[commit-bot: I haz the power, 2013-02-16 07:54:20]

Change committed as 182956