device_orientation::ProviderImpl to stop polling thread asynchronously

content/browser/device_orientation/provider_impl.h

Index: content/browser/device_orientation/provider_impl.h
===================================================================
--- content/browser/device_orientation/provider_impl.h	(revision 148849)
+++ content/browser/device_orientation/provider_impl.h	(working copy)
@@ -18,10 +18,6 @@
 
 class MessageLoop;
 
-namespace base {
-class Thread;
-}
-
 namespace device_orientation {
 
 class ProviderImpl : public Provider {
@@ -37,33 +33,21 @@
   virtual void RemoveObserver(Observer* observer) OVERRIDE;
 
  private:
+  class PollingThread;
+  friend class PollingThread;

[hans, 2012/07/30 12:56:40]

nit: it feels like only one of these should be necessary, but if your compiler
insists, then I won't argue about it :)

+
   virtual ~ProviderImpl();
 
   // Starts or Stops the provider. Called from creator_loop_.
   void Start();
   void Stop();
 
-  // Method for finding a suitable DataFetcher and starting the polling.
-  // Runs on the polling_thread_.
-  void DoInitializePollingThread(
-      const std::vector<DataFetcherFactory>& factories);
   void ScheduleInitializePollingThread();
 
-  // Method for polling a DataFetcher. Runs on the polling_thread_.
-  void DoPoll();
-  void ScheduleDoPoll();
-
   // Method for notifying observers of an orientation update.
   // Runs on the creator_thread_.
   void DoNotify(const Orientation& orientation);
-  void ScheduleDoNotify(const Orientation& orientation);
 
-  static bool SignificantlyDifferent(const Orientation& orientation1,
-                                     const Orientation& orientation2);
-
-  enum { kDesiredSamplingIntervalMs = 100 };
-  base::TimeDelta SamplingInterval() const;
-
   // The Message Loop on which this object was created.
   // Typically the I/O loop, but may be something else during testing.
   MessageLoop* creator_loop_;
@@ -73,14 +57,13 @@
   std::set<Observer*> observers_;
   Orientation last_notification_;
 
-  // When polling_thread_ is running, members below are only to be used
-  // from that thread.
-  scoped_ptr<DataFetcher> data_fetcher_;
-  Orientation last_orientation_;
   base::WeakPtrFactory<ProviderImpl> weak_factory_;
 
-  // Polling is done on this background thread.
-  scoped_ptr<base::Thread> polling_thread_;
+  // Polling is done on this background thread. PollingThread is owned by
+  // the ProviderImpl object. But its deletion doesn't happen synchronously
+  // along with deletion of the ProviderImpl. Thus this should be a raw
+  // pointer instead of scoped_ptr.
+  PollingThread* polling_thread_;
 };
 
 }  // namespace device_orientation