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

Unified Diff: content/browser/renderer_host/media/web_contents_video_capture_device.cc

Issue 11413078: Tab Audio Capture: Browser-side connect/disconnect functionality. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Minor tweaks and added some useful comments. Created 8 years, 1 month 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/web_contents_video_capture_device.cc
diff --git a/content/browser/renderer_host/media/web_contents_video_capture_device.cc b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
index d9b8b7f9708a6eceb76f1a0b225c33c7fd320181..997d5bfecfb1f04ea3c8aba5998cb3f3da21b8a9 100644
--- a/content/browser/renderer_host/media/web_contents_video_capture_device.cc
+++ b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
@@ -141,6 +141,8 @@ void CalculateFittedSize(int source_width, int source_height,
// Keeps track of the RenderView to be sourced, and executes copying of the
// backing store on the UI BrowserThread.
+// TODO(miu): Use WebContentsCaptureUtil::RenderViewTracker instead of doing the
+// observing ourselves here.
class BackingStoreCopier : public WebContentsObserver {
public:
// Result status and done callback used with StartCopy().

Powered by Google App Engine
This is Rietveld 408576698