Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(232)

Unified Diff: content/browser/renderer_host/media/video_capture_host.h

Issue 17508005: Minor cleanup to remove the static thread-safe methods on BrowserMainLoop. Initially I thought this… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: revert PostTask change Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/media/video_capture_host.h
===================================================================
--- content/browser/renderer_host/media/video_capture_host.h (revision 207505)
+++ content/browser/renderer_host/media/video_capture_host.h (working copy)
@@ -46,12 +46,13 @@
#include "ipc/ipc_message.h"
namespace content {
+class MediaStreamManager;
class CONTENT_EXPORT VideoCaptureHost
: public BrowserMessageFilter,
public VideoCaptureControllerEventHandler {
public:
- VideoCaptureHost();
+ explicit VideoCaptureHost(MediaStreamManager* media_stream_manager);
// BrowserMessageFilter implementation.
virtual void OnChannelClosing() OVERRIDE;
@@ -133,9 +134,7 @@
void DeleteVideoCaptureControllerOnIOThread(
const VideoCaptureControllerID& controller_id);
- // Returns the video capture manager. This is a virtual function so that
- // the unit tests can inject their own MediaStreamManager.
- virtual VideoCaptureManager* GetVideoCaptureManager();
+ MediaStreamManager* media_stream_manager_;
struct Entry;
typedef std::map<VideoCaptureControllerID, Entry*> EntryMap;

Powered by Google App Engine
This is Rietveld 408576698