Cast: Allow extension to control wifi options on windows

net/base/net_util_unittest.cc

Index: net/base/net_util_unittest.cc
diff --git a/net/base/net_util_unittest.cc b/net/base/net_util_unittest.cc
index 8306db53c2b86fc4c2687e88176d5fe74f5c9807..e10f4ffb762bf30402cb69351211930d26efb7f3 100644
--- a/net/base/net_util_unittest.cc
+++ b/net/base/net_util_unittest.cc
@@ -790,6 +790,16 @@ TEST(NetUtilTest, GetNetworkList) {
   }
 }
 
+// Test SetWifiOptions().
+TEST(NetUtilTest, SetWifiOptionsTest) {
+  SetWifiOptions(0);
+  SetWifiOptions(WIFI_OPTIONS_DISABLE_SCAN);
+  SetWifiOptions(WIFI_OPTIONS_MEDIA_STREAMING_MODE);
+  SetWifiOptions(WIFI_OPTIONS_DISABLE_SCAN |
+                 WIFI_OPTIONS_MEDIA_STREAMING_MODE);

[pauljensen, 2014/09/18 14:16:50]

I assume you can verify that these functions are taking effect and cleaning up
after themselves with WlanQueryInterface.  Can you please add that verification
here?

[hubbe, 2014/09/18 18:21:05]

It's possible, but is it worth it?

I'll have to expose WlanQueryInterface from inside net_util_win.h, or duplicate
the code here, and then it will only ever do something if the computer you run
the test on actually has a wifi interface.
(Which I don't think any of our test bots have.)

AND, to top it all off, it's possible to configure your window ACLs such that
calling SetWifiOptions always fails unless you're administrator.... (Although
I'm not sure if anybody anywhere actually does that.)

If you still think it's worth it, do you prefer that I expose the wlanapi, or
duplicate the code?

[pauljensen, 2014/09/18 18:40:20]

I believe it is worth adding a test when adding a new API and implementation. 
If there is value in adding the functionality, there is value in ensuring that
functionality works now and continues to work.  It's unfortunate that most/all
of our bots don't have Wifi; which begs the question, how is the Cast extension
tested?  i.e. how will you know this new API works?

Exposing the wlanapi just to tests seems fine.

[hubbe, 2014/09/18 23:42:51]

Done.

+}
+
+
 struct NonUniqueNameTestData {
   bool is_unique;
   const char* hostname;