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

Unified Diff: content/renderer/media/media_stream_dispatcher.h

Issue 2763743002: Android: not to pause screen capture when Chrome is put to background (Closed)
Patch Set: address comments on PS#1 Created 3 years, 9 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/renderer/media/media_stream_dispatcher.h
diff --git a/content/renderer/media/media_stream_dispatcher.h b/content/renderer/media/media_stream_dispatcher.h
index 18803c1277de6a49f457159d4f794c91f45b42cc..a243e1c3ebd934e22fe3bc727086b12c01b75131 100644
--- a/content/renderer/media/media_stream_dispatcher.h
+++ b/content/renderer/media/media_stream_dispatcher.h
@@ -75,6 +75,9 @@ class CONTENT_EXPORT MediaStreamDispatcher
// This method is called when the stream is started successfully.
void OnStreamStarted(const std::string& label);
+ // Get all the media devices of video capture, e.g. webcam.
miu 2017/03/22 23:09:42 nit: It would be helpful to add to this comment, s
braveyao 2017/03/23 03:44:19 Done.
+ void GetVideoCaptureDevices(StreamDeviceInfoArray* video_array);
miu 2017/03/22 23:09:43 This naming doesn't suggest it is filtering based
braveyao 2017/03/23 03:44:20 Done.
+
// Check if the label is a valid stream.
virtual bool IsStream(const std::string& label);
// Get the video session_id given a label. The label identifies a stream.
@@ -90,6 +93,7 @@ class CONTENT_EXPORT MediaStreamDispatcher
FRIEND_TEST_ALL_PREFIXES(MediaStreamDispatcherTest, BasicVideoDevice);
FRIEND_TEST_ALL_PREFIXES(MediaStreamDispatcherTest, TestFailure);
FRIEND_TEST_ALL_PREFIXES(MediaStreamDispatcherTest, CancelGenerateStream);
+ FRIEND_TEST_ALL_PREFIXES(MediaStreamDispatcherTest, GetVideoCaptureDevices);
struct Request;
« no previous file with comments | « no previous file | content/renderer/media/media_stream_dispatcher.cc » ('j') | content/renderer/media/media_stream_dispatcher.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698