Refactor WifiDataProviderChromeOs to implement WifiDataProviderImplBase directly.

content/browser/geolocation/wifi_data_provider_chromeos.h

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_GEOLOCATION_WIFI_DATA_PROVIDER_CHROMEOS_H_
 #define CONTENT_BROWSER_GEOLOCATION_WIFI_DATA_PROVIDER_CHROMEOS_H_
 #pragma once
 
 #include "content/browser/geolocation/wifi_data_provider_common.h"
 
 namespace chromeos {
 class NetworkLibrary;
 }
 
-class WifiDataProviderChromeOs : public WifiDataProviderCommon {
+class WifiDataProviderChromeOs : public WifiDataProviderImplBase {
  public:
   WifiDataProviderChromeOs();
 
+  // WifiDataProviderImplBase implementation

[bulach, 2011/03/15 18:37:51]

nit: maybe remove "implementation"?

+  virtual bool StartDataProvider();
+  virtual void StopDataProvider();
+  virtual bool GetData(WifiData *data);
+
   // Allows injection of |lib| for testing.
-  static WlanApiInterface* NewWlanApi(chromeos::NetworkLibrary* lib);
+  static WifiDataProviderCommon::WlanApiInterface* NewWlanApi(
+      chromeos::NetworkLibrary* lib);
 
  private:
   virtual ~WifiDataProviderChromeOs();
 
+  // The polling task
+  void DoWifiScanTask();
+
   // WifiDataProviderCommon
-  virtual WlanApiInterface* NewWlanApi();
+  virtual WifiDataProviderCommon::WlanApiInterface* NewWlanApi();
   virtual PollingPolicyInterface* NewPollingPolicy();
 
+  // Will schedule a scan; i.e. enqueue DoWifiScanTask deferred task.
+  void ScheduleNextScan(int interval);
+
+  // Underlying OS wifi API.
+  scoped_ptr<WifiDataProviderCommon::WlanApiInterface> wlan_api_;
+
+  // Controls the polling update interval.
+  scoped_ptr<PollingPolicyInterface> polling_policy_;
+
+  WifiData wifi_data_;
+
+  // Whether we have strated the data provider.

[bulach, 2011/03/15 18:37:51]

nit:/strated/started/

+  bool started_;
+
+  // Whether we've successfully completed a scan for WiFi data.
+  bool is_first_scan_complete_;
+
+  // Holder for the tasks which run on the thread; takes care of cleanup.
+  ScopedRunnableMethodFactory<WifiDataProviderChromeOs> task_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(WifiDataProviderChromeOs);
 };
 
 #endif  // CONTENT_BROWSER_GEOLOCATION_WIFI_DATA_PROVIDER_CHROMEOS_H_