commit fb46caf4f23a41e73dfd87547121b5fb0230295a
author Thieu Le <thieule@chromium.org> Thu Mar 08 11:57:15 2012 -0800
committer Gerrit <chrome-bot@google.com> Fri Mar 09 16:28:35 2012 -0800
tree d479c4f9028a6a49784c42d8118cf196398e7cc8
parent 8783e0d540631c7f0f445d2cc62f52702c07c607

shill: Delete existing routes when the device is started.

BUG=chromium-os:27358
TEST=Unit tests, network_WiFiManager, manually add foreign routes and
ensure they are deleted, also test to make sure default route remains
after DHCP renewal

Change-Id: Ic26b70d685491f31699ff4684ed289e76af8a9ca
Reviewed-on: https://gerrit.chromium.org/gerrit/17640
Commit-Ready: Thieu Le <thieule@chromium.org>
Reviewed-by: Thieu Le <thieule@chromium.org>
Tested-by: Thieu Le <thieule@chromium.org>

connection_unittest.cc

diff --git a/connection_unittest.cc b/connection_unittest.cc
index 16e0f78..08d85b9 100644
--- a/connection_unittest.cc
+++ b/connection_unittest.cc
@@ -198,14 +198,14 @@
     connection->ReleaseRouting();
 
     // The destructor will remove the routes and addresses.
-    EXPECT_CALL(routing_table_, FlushRoutes(kTestDeviceInterfaceIndex0, true));
+    EXPECT_CALL(routing_table_, FlushRoutes(kTestDeviceInterfaceIndex0));
     EXPECT_CALL(*device_info_.get(),
                 FlushAddresses(kTestDeviceInterfaceIndex0));
   }
 }
 
 TEST_F(ConnectionTest, Destructor) {
-  EXPECT_CALL(routing_table_, FlushRoutes(kTestDeviceInterfaceIndex1, true));
+  EXPECT_CALL(routing_table_, FlushRoutes(kTestDeviceInterfaceIndex1));
   EXPECT_CALL(*device_info_, FlushAddresses(kTestDeviceInterfaceIndex1));
   {
     ConnectionRefPtr connection(new Connection(kTestDeviceInterfaceIndex1,