MediaStream audio: Refactor 3 separate "glue" implementations into one.

content/renderer/media/media_stream_source.h

 // Copyright 2014 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_RENDERER_MEDIA_MEDIA_STREAM_SOURCE_H_
 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_SOURCE_H_
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/macros.h"
+#include "base/threading/thread_checker.h"
 #include "content/common/content_export.h"
 #include "content/common/media/media_stream_options.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
 
 namespace blink {
 class WebMediaStreamTrack;
 }  // namespace blink
 
 namespace content {
 
@@ skipping 14 matching lines @@
 
   MediaStreamSource();
   ~MediaStreamSource() override;
 
   // Returns device information about a source that has been created by a
   // JavaScript call to GetUserMedia, e.g., a camera or microphone.
   const StreamDeviceInfo& device_info() const {
     return device_info_;
   }
 
-  // Stops the source (by calling DoStopSource()). This sets the
-  // WebMediaStreamSource::readyState to ended, triggers the |stop_callback_|
-  // if set. All pointers to this object are invalid after calling this.
+  // Stops the source (by calling DoStopSource()). This runs the
+  // |stop_callback_| (if set), and then sets the
+  // WebMediaStreamSource::readyState to ended.
   void StopSource();
 
-  void ResetSourceStoppedCallback() {
-    DCHECK(!stop_callback_.is_null());
-    stop_callback_.Reset();
-  }
+  // Sets device information about a source that has been created by a
+  // JavaScript call to GetUserMedia. F.E a camera or microphone.
+  void SetDeviceInfo(const StreamDeviceInfo& device_info);
+
+  // Sets a callback that will be triggered when StopSource is called.
+  void SetStopCallback(const SourceStoppedCallback& stop_callback);
+
+  // Clears the previously-set SourceStoppedCallback so that it will not be run
+  // in the future.
+  void ResetSourceStoppedCallback();
 
  protected:
   // Called when StopSource is called. It allows derived classes to implement
-  // its  own Stop method.
+  // its own Stop method.
   virtual void DoStopSource() = 0;
 
-  // Sets device information about a source that has been created by a
-  // JavaScript call to GetUserMedia. F.E a camera or microphone.
-  void SetDeviceInfo(const StreamDeviceInfo& device_info) {
-    device_info_ = device_info;
-  }
+ private:
+  // Called by both StopSource() and the destructor to ensure the
+  // |stop_callback_| has been run and the blink::WebMediaStreamSource's ready
+  // state has been set to "ended."
+  void RunStopCallbackAndEndStream();
 
-  // Sets a callback that will be triggered when StopSource is called.
-  void SetStopCallback(const SourceStoppedCallback& stop_callback) {
-    DCHECK(stop_callback_.is_null());
-    stop_callback_ = stop_callback;
-  }
-
- private:
   StreamDeviceInfo device_info_;
   SourceStoppedCallback stop_callback_;
 
+  // In debug builds, check that all methods are being called on the main
+  // thread.
+  base::ThreadChecker thread_checker_;
+
   DISALLOW_COPY_AND_ASSIGN(MediaStreamSource);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_SOURCE_H_