Implement networkingPrivate.getDeviceStates

chrome/browser/extensions/api/networking_private/networking_private_apitest.cc

Index: chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
diff --git a/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc b/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
index 8d7b2665020c45b0354aee7fde78d7061bb76dfb..102ca79ff4a7e0428bad124b5fbf624002794cfb 100644
--- a/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
+++ b/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
@@ -146,6 +146,16 @@ class TestDelegate : public NetworkingPrivateDelegate {
     return result.Pass();
   }
 
+  scoped_ptr<DeviceStateMap> GetDeviceStates() override {
+    scoped_ptr<DeviceStateMap> result;
+    if (!fail_) {
+      result.reset(new DeviceStateMap);
+      (*result)[ ::onc::network_config::kEthernet] =

[pneubeck (no reviews), 2015/04/08 10:05:07]

nit: remove space after [

[stevenjb, 2015/04/08 20:27:30]

Blame clang :P

+          ::onc::device_state::kEnabled;
+    }
+    return result.Pass();
+  }
+
   bool EnableNetworkType(const std::string& type) override {
     enabled_[type] = true;
     return !fail_;
@@ -362,6 +372,10 @@ IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTest, GetEnabledNetworkTypes) {
   EXPECT_TRUE(RunNetworkingSubtest("getEnabledNetworkTypes")) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTest, GetDeviceStates) {
+  EXPECT_TRUE(RunNetworkingSubtest("getDeviceStates")) << message_;
+}
+
 IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTest, EnableNetworkType) {
   EXPECT_TRUE(RunNetworkingSubtest("enableNetworkType")) << message_;
   EXPECT_TRUE(GetEnabled(::onc::network_config::kEthernet));
@@ -459,6 +473,10 @@ IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTestFail, GetEnabledNetworkTypes) {
   EXPECT_FALSE(RunNetworkingSubtest("getEnabledNetworkTypes")) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTestFail, GetDeviceStates) {
+  EXPECT_FALSE(RunNetworkingSubtest("getDeviceStates")) << message_;
+}
+
 // Note: Synchronous methods never fail:
 // * disableNetworkType
 // * enableNetworkType