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

Side by Side Diff: content/renderer/media/media_stream_video_source.h

Issue 442643002: Allow MediaStream constraints to specify higher than 30 FPS tab capture. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile whoops. Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_
6 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_ 6 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 93
94 // An implementation must fetch the formats that can currently be used by 94 // An implementation must fetch the formats that can currently be used by
95 // the source and call OnSupportedFormats when done. 95 // the source and call OnSupportedFormats when done.
96 // |max_requested_height| and |max_requested_width| is the max height and 96 // |max_requested_height| and |max_requested_width| is the max height and
97 // width set as a mandatory constraint if set when calling 97 // width set as a mandatory constraint if set when calling
98 // MediaStreamVideoSource::AddTrack. If max height and max width is not set 98 // MediaStreamVideoSource::AddTrack. If max height and max width is not set
99 // |max_requested_height| and |max_requested_width| are 0. 99 // |max_requested_height| and |max_requested_width| are 0.
100 virtual void GetCurrentSupportedFormats( 100 virtual void GetCurrentSupportedFormats(
101 int max_requested_width, 101 int max_requested_width,
102 int max_requested_height, 102 int max_requested_height,
103 double max_requested_frame_rate,
103 const VideoCaptureDeviceFormatsCB& callback) = 0; 104 const VideoCaptureDeviceFormatsCB& callback) = 0;
104 105
105 // An implementation must start capture frames using the resolution in 106 // An implementation must start capture frames using the resolution in
106 // |params|. When the source has started or the source failed to start 107 // |params|. When the source has started or the source failed to start
107 // OnStartDone must be called. An implementation must call 108 // OnStartDone must be called. An implementation must call
108 // invoke |frame_callback| on the IO thread with the captured frames. 109 // invoke |frame_callback| on the IO thread with the captured frames.
109 // TODO(perkj): pass a VideoCaptureFormats instead of VideoCaptureParams for 110 // TODO(perkj): pass a VideoCaptureFormats instead of VideoCaptureParams for
110 // subclasses to customize. 111 // subclasses to customize.
111 virtual void StartSourceImpl( 112 virtual void StartSourceImpl(
112 const media::VideoCaptureParams& params, 113 const media::VideoCaptureParams& params,
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
175 176
176 // NOTE: Weak pointers must be invalidated before all other member variables. 177 // NOTE: Weak pointers must be invalidated before all other member variables.
177 base::WeakPtrFactory<MediaStreamVideoSource> weak_factory_; 178 base::WeakPtrFactory<MediaStreamVideoSource> weak_factory_;
178 179
179 DISALLOW_COPY_AND_ASSIGN(MediaStreamVideoSource); 180 DISALLOW_COPY_AND_ASSIGN(MediaStreamVideoSource);
180 }; 181 };
181 182
182 } // namespace content 183 } // namespace content
183 184
184 #endif // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_ 185 #endif // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_VIDEO_SOURCE_H_
OLDNEW
« no previous file with comments | « content/renderer/media/media_stream_video_capturer_source.cc ('k') | content/renderer/media/media_stream_video_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698