Mac OS X-specific implementation of Networking Private API.

components/wifi/wifi_service.h

Index: components/wifi/wifi_service.h
diff --git a/components/wifi/wifi_service.h b/components/wifi/wifi_service.h
index b45ebfdeb1386feec4ac5d3a20824b74529a85ff..35c95b5800053742c02d23f4caed210ebfa5f55e 100644
--- a/components/wifi/wifi_service.h
+++ b/components/wifi/wifi_service.h
@@ -6,6 +6,7 @@
 #define CHROME_UTILITY_WIFI_WIFI_SERVICE_H_
 
 #include <list>
+#include <set>
 #include <string>
 #include <vector>
 
@@ -19,9 +20,11 @@
 namespace wifi {
 
 // WiFiService interface used by implementation of chrome.networkingPrivate
-// JavaScript extension API. All methods should be called on worker thread.
+// JavaScript extension API. All methods should be called on worker thread or.
+// posted to worker pool using SequencedTaskRunner |task_runner_|.

[tbarzic, 2014/01/27 20:17:24]

You don't need the second part of the sentence. (especially since |task_runner_|
is not part of the |WifiService| interface)

[mef, 2014/01/27 23:08:27]

Done.

 // It could be created on any (including UI) thread, so nothing expensive should
-// be done in the constructor.
+// be done in the constructor. See |NetworkingPrivateService| for wrapper
+// accessible on UI thread.
 class WIFI_EXPORT WiFiService {
  public:
   typedef std::vector<std::string> NetworkGuidList;
@@ -116,7 +119,7 @@ class WIFI_EXPORT WiFiService {
     kFrequency5000 = 5000
   };
 
-  typedef std::list<Frequency> FrequencyList;
+  typedef std::set<Frequency> FrequencySet;
   // Network Properties, used as result of |GetProperties| and
   // |GetVisibleNetworks|.
   struct WIFI_EXPORT NetworkProperties {
@@ -138,7 +141,7 @@ class WIFI_EXPORT WiFiService {
     uint32 signal_strength;
     bool auto_connect;
     Frequency frequency;
-    FrequencyList frequency_list;
+    FrequencySet frequency_set;
 
     std::string json_extra;  // Extra JSON properties for unit tests