commit | 5db19692534f5ce992c5b28cacb00c3189045273 | [log] [tgz] |
---|---|---|
author | Ben Chan <benchan@chromium.org> | Wed Jul 10 06:07:03 2013 |
committer | ChromeBot <chrome-bot@google.com> | Fri Jul 12 05:32:53 2013 |
tree | a112bf958ed79092fe5cca3e1ca35e9ecfb78502 | |
parent | 5d851b17501ac27d28e7ced3fe72f3b4fdd37260 [diff] |
shill: cellular: Handle vanish of bearer when getting its properties. This CL changes CellularCapabilityUniversal::OnListBearersReply() to get the properties of a bearer via a single invocation of DBusPropertiesProxy::GetAll() instead of multiple invocations of different property getters in mm1::BearerProxy. This allows us to handle the vanish of a bearer more gracefully when getting its properties over DBus. BUG=chromium:244496 TEST=Tested the following: 1. Build and run unit tests. 2. Run network_3GSmokeTest test on platforms with Y3400 and E362 modem. 3. Run network_3GModemControl test with pseudomodem. 4. Test a few 3G dongles that use PPP. Change-Id: I7d594d59dae5191bc0db1defcdaf4663d929a529 Reviewed-on: https://gerrit.chromium.org/gerrit/61587 Commit-Queue: Ben Chan <benchan@chromium.org> Reviewed-by: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>