commit | 151d447d1f3a252a3f2ecbca0f75792e693f1d47 | [log] [tgz] |
---|---|---|
author | Ben Chan <benchan@chromium.org> | Fri Sep 06 13:29:46 2013 -0700 |
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Wed Sep 11 18:42:42 2013 +0000 |
tree | 3c3ad444e4862cdb2c9d77709a1ecf857935b571 | |
parent | 3fe66041142135b0f40820bee17376a77bf3bec1 [diff] |
shill: cellular: Simplify modem enabling operation. Instead of first checking the current modem state, this CL changes CellularCapabilityUniversal::StartModem() to always try to issue the org.freedesktop.ModemManager1.Modem.Enable DBus call to enable the modem, regardless of the current modem state. If the Enable call fails with an org.freedesktop.ModemManager1.Error.Core.WrongState DBus error, the Enable call is deferred until the modem goes into the Disabled state. BUG=chromium:287667 TEST=Tested the following: 1. Build and run unit tests. 2. Run network_3GSmokeTest and network_3GStressEnable tests with E362, ALT3100 and MU736. Change-Id: I20349e9ef923473274a0abe2b1b61a3d8223e6af Reviewed-on: https://chromium-review.googlesource.com/168663 Reviewed-by: Thieu Le <thieule@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>