| commit | 11567d0439c33ad4ed5165ce4a4eb791f06a4d24 | [log] [tgz] |
|---|---|---|
| author | Eric Shienbrood <ers@chromium.org> | Tue Apr 10 18:08:49 2012 -0400 |
| committer | Gerrit <chrome-bot@google.com> | Wed Apr 11 08:16:20 2012 -0700 |
| tree | fea801f5ba0b1ab73f06e53cdd0bf98b2b342b97 | |
| parent | 6681533daa896254189d1061ede8aab242a82ab9 [diff] |
shill: Create DBusPropertiesProxy for Modem. Since the ModemManagerClassic/ModemManager1 split happened, modems had not been creating a DBusPropertiesProxy, so they have not been getting PropertyChanged or MmPropertiesChanged signals. As a result, shill hasn't been handling updates to network technology or SIM lock status. This change adds creation of the proxy to the Modem intialization path, and moves the inheritance from DBusPropertiesProxyDelegate up to the base Modem class from ModemClassic. BUG=chromium-os:29230 TEST=Disable and enable the modem. Use list-services and mm-status to check whether shill's view of the network technology matches ModemManager's AccessTechnology property. Also enable and disable the SIM lock PIN, and make sure that the device SIMLockStatus property gets updated properly. Change-Id: I1f03935cb05e1ad0edf057b5bb64e993b9ccd977 Reviewed-on: https://gerrit.chromium.org/gerrit/19928 Reviewed-by: Jason Glasgow <jglasgow@chromium.org> Commit-Ready: Eric Shienbrood <ers@chromium.org> Tested-by: Eric Shienbrood <ers@chromium.org>