tree 931729eb4a3f7cab847941fef790d8796cb75664
parent 4823183bd3a6e31868fb54972c7ef9bc49b52ec3
author mukesh agrawal <quiche@chromium.org> 1374270548 -0700
committer ChromeBot <chrome-bot@google.com> 1374560048 -0700

shill: PPP dongles: cleanly move Service from Cellular to PPPDevice

In CL:61602, I added a CHECK that the Cellular object doesn't have
a selected_service() when PPP comes up. The idea was to ensure that
we only have one Device trying to drive the state of the CellularService.

This CHECK fails on devices with an embryonic cdc_ether or cdc_ncm
interface, because we SelectService() in LinkEvent().

Resolve this by dropping the selected_service() in StartPPP().
We do this using DropConnection(), which may be what I should have
done all along. (DropConnection() also takes care of killing dhcpcd,
if it is running.)

While there: fix up a couple of buggy log statements in device_info.

BUG=chromium:261364
TEST=unit tests, manual

Manual testing
--------------
- plug in a dongle with an embryonic cdc_ether or cdc_ncm interface.
  e.g. the huawei UMG1691 dongle
- check that the cellular connection comes up (IP address is configured,
  can load web pages)
- check that shill did not crash in the process (grep crash /var/log/messages)
- repeat steps 1-3 with a ppp-only dongle (e.g. huawei e369)

Change-Id: Ia270f00746dc321b41a223e9f848ffc796dd8528
Reviewed-on: https://gerrit.chromium.org/gerrit/62712
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Queue: mukesh agrawal <quiche@chromium.org>