Add getGlobalPolicy to networkingPrivate API.

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 02d6d1abc93783764fdd3aaecb5aa73ac1396b11..459c374f1d37e0d4c4e7c560a373acfb5dd0ba5d 100644
--- a/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
+++ b/chrome/browser/extensions/api/networking_private/networking_private_apitest.cc
@@ -183,6 +183,16 @@ class TestDelegate : public NetworkingPrivateDelegate {
     return result;
   }
 
+  std::unique_ptr<base::DictionaryValue> GetGlobalPolicy() override {
+    auto result = base::MakeUnique<base::DictionaryValue>();
+    result->SetBooleanWithoutPathExpansion(

[tbarzic, 2017/01/06 22:43:17]

I'd add an extra property to the policy dict - to ensure API handles that case
well.

[stevenjb, 2017/01/09 19:30:37]

Good call, but I will actually do that in the chromeos specific test. I am going
to make this just return an empty dictionary and remove the trivial api test for
consistency. We should really either make the rest of these delegate methods
trivial, or get assertEq working and actually test the return values for the
entire API.

+        onc::global_network_config::kAllowOnlyPolicyNetworksToAutoconnect,
+        true);
+    result->SetBooleanWithoutPathExpansion(
+        onc::global_network_config::kAllowOnlyPolicyNetworksToConnect, false);
+    return result;
+  }
+
   bool EnableNetworkType(const std::string& type) override {
     enabled_[type] = true;
     return !fail_;
@@ -462,6 +472,10 @@ IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTest, SetCellularSimState) {
   EXPECT_TRUE(RunNetworkingSubtest("setCellularSimState")) << message_;
 }
 
+IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTest, GetGlobalPolicy) {
+  EXPECT_TRUE(RunNetworkingSubtest("getGlobalPolicy")) << message_;
+}
+
 // Test failure case
 
 class NetworkingPrivateApiTestFail : public NetworkingPrivateApiTest {
@@ -516,6 +530,7 @@ IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTestFail, GetDeviceStates) {
 // * disableNetworkType
 // * enableNetworkType
 // * requestNetworkScan
+// * getGlobalPolicy
 
 IN_PROC_BROWSER_TEST_F(NetworkingPrivateApiTestFail, StartConnect) {
   EXPECT_FALSE(RunNetworkingSubtest("startConnect")) << message_;