Cleaner organization of WifiDataProvider code.

content/browser/geolocation/network_location_provider.h

Index: content/browser/geolocation/network_location_provider.h
diff --git a/content/browser/geolocation/network_location_provider.h b/content/browser/geolocation/network_location_provider.h
index daf0c8b4417e5175ef16e4d430f74334f4e5e6eb..709ffad728fa58f56c1ffba9dc0da479d9920401 100644
--- a/content/browser/geolocation/network_location_provider.h
+++ b/content/browser/geolocation/network_location_provider.h
@@ -17,7 +17,7 @@
 #include "base/threading/thread.h"
 #include "content/browser/geolocation/location_provider_base.h"
 #include "content/browser/geolocation/network_location_request.h"
-#include "content/browser/geolocation/wifi_data_provider.h"
+#include "content/browser/geolocation/wifi_data_provider_manager.h"
 #include "content/common/content_export.h"
 #include "content/public/common/geoposition.h"
 
@@ -83,7 +83,7 @@ class NetworkLocationProvider
   void RequestPosition();
 
   // Gets called when new wifi data is available.
-  void OnWifiDataUpdate(WifiDataProvider* provider);
+  void OnWifiDataUpdate(WifiDataProviderManager* provider);

[timvolodine, 2014/08/18 15:21:42]

provider -> manager as argument name would probably be more readable.

[Michael van Ouwerkerk, 2014/08/19 13:02:39]

Done.

 
   // Internal helper used by OnWifiDataUpdate.
   void OnWifiDataUpdated();
@@ -98,9 +98,9 @@ class NetworkLocationProvider
   scoped_refptr<AccessTokenStore> access_token_store_;
 
   // The wifi data provider, acquired via global factories.
-  WifiDataProvider* wifi_data_provider_;
+  WifiDataProviderManager* wifi_data_provider_manager_;
 
-  WifiDataProvider::WifiDataUpdateCallback wifi_data_update_callback_;
+  WifiDataProviderManager::WifiDataUpdateCallback wifi_data_update_callback_;
 
   // The  wifi data and a flag to indicate if the data set is complete.
   WifiData wifi_data_;