commit | 0e1cdeae24dd678a5fe27c840802582c0ca45ec0 | [log] [tgz] |
---|---|---|
author | Albert Chaulk <achaulk@chromium.org> | Wed Feb 27 15:32:55 2013 -0800 |
committer | ChromeBot <chrome-bot@google.com> | Fri Mar 15 11:02:48 2013 -0700 |
tree | e67c68107acebe1643b935da46a397fca3109cfd | |
parent | ba7aba2649e019000aa802021b0f61bfae6886c1 [diff] |
shill: Implement Service.Remove & delete DHCP leases Implement the Service.Remove dbus API and modify the unload sequence to remove the DHCP lease file. Add unit tests to DHCPProvider & WiFiService to check lease file deletion. Currently only WiFi does the DHCP lease delete from DBus - WiMax/Cell/VPN do their own things for IP configuration and Ethernet can't be removed as per the API spec. Also make sure FilePath is qualified as base::FilePath throughout the project. BUG=chromium-os:32756 TEST=added unittests to test the call path on Unload() to delete the lease file Change-Id: Ic6eee17b9d81cd0be8d09c683d85d6a4d19278c9 Reviewed-on: https://gerrit.chromium.org/gerrit/44339 Tested-by: Albert Chaulk <achaulk@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Commit-Queue: Albert Chaulk <achaulk@chromium.org>