Deprecate ShillNetworkClient and add GeolocationHandler

chrome/browser/geolocation/wifi_data_provider_chromeos.cc

Index: chrome/browser/geolocation/wifi_data_provider_chromeos.cc
diff --git a/chrome/browser/geolocation/wifi_data_provider_chromeos.cc b/chrome/browser/geolocation/wifi_data_provider_chromeos.cc
index 265ddc1a962c7ab8f8dd695de605112b7ebd8af4..3dd160e1c2c34c48ba3c9f4628f486edfd70d7a0 100644
--- a/chrome/browser/geolocation/wifi_data_provider_chromeos.cc
+++ b/chrome/browser/geolocation/wifi_data_provider_chromeos.cc
@@ -76,19 +76,9 @@ NetworkLibraryWlanApi::~NetworkLibraryWlanApi() {
 bool NetworkLibraryWlanApi::GetAccessPointData(
     WifiData::AccessPointDataSet* result) {
   WifiAccessPointVector access_points;
-  if (!network_library_->GetWifiAccessPoints(&access_points))
-    return false;
-  for (WifiAccessPointVector::const_iterator i = access_points.begin();
-       i != access_points.end(); ++i) {
-    AccessPointData ap_data;
-    ap_data.mac_address = ASCIIToUTF16(i->mac_address);
-    ap_data.radio_signal_strength = i->signal_strength;
-    ap_data.channel = i->channel;
-    ap_data.signal_to_noise = i->signal_to_noise;
-    ap_data.ssid = UTF8ToUTF16(i->name);
-    result->insert(ap_data);
-  }
-  return !result->empty() || network_library_->wifi_enabled();
+  // TODO(stevenjb): Re-implement this with ne network handlers.

[Greg Spencer (Chromium), 2013/01/12 00:52:37]

ne->new

Also, what ultimately depends on this info? And if this is still broken, won't
there still be problems (and so should have a bug associated with it)?

[stevenjb, 2013/01/12 01:55:59]

This is part 1/2 of the fix. I added the issue to the comment.

+  // (This code has been broken since switching to Shill).
+  return false;
 }
 
 }  // namespace