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

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

Issue 2624053002: MediaRecorder: use VideoTrackRecorder::GetPreferredCodecId() when available (Closed)
Patch Set: Windows issue: doesn't like static constexpr members w/ initialization Created 3 years, 11 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_VIDEO_TRACK_RECORDER_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_
6 #define CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_ 6 #define CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 19 matching lines...) Expand all
30 30
31 // VideoTrackRecorder is a MediaStreamVideoSink that encodes the video frames 31 // VideoTrackRecorder is a MediaStreamVideoSink that encodes the video frames
32 // received from a Stream Video Track. This class is constructed and used on a 32 // received from a Stream Video Track. This class is constructed and used on a
33 // single thread, namely the main Render thread. This mirrors the other 33 // single thread, namely the main Render thread. This mirrors the other
34 // MediaStreamVideo* classes that are constructed/configured on Main Render 34 // MediaStreamVideo* classes that are constructed/configured on Main Render
35 // thread but that pass frames on Render IO thread. It has an internal Encoder 35 // thread but that pass frames on Render IO thread. It has an internal Encoder
36 // with its own threading subtleties, see the implementation file. 36 // with its own threading subtleties, see the implementation file.
37 class CONTENT_EXPORT VideoTrackRecorder 37 class CONTENT_EXPORT VideoTrackRecorder
38 : NON_EXPORTED_BASE(public MediaStreamVideoSink) { 38 : NON_EXPORTED_BASE(public MediaStreamVideoSink) {
39 public: 39 public:
40 // Do not change the order of codecs; add new ones right before LAST.
40 enum class CodecId { 41 enum class CodecId {
41 VP8, 42 VP8,
42 VP9, 43 VP9,
43 H264, 44 H264,
45 LAST
44 }; 46 };
45 class Encoder; 47 class Encoder;
46 48
47 using OnEncodedVideoCB = 49 using OnEncodedVideoCB =
48 base::Callback<void(const media::WebmMuxer::VideoParameters& params, 50 base::Callback<void(const media::WebmMuxer::VideoParameters& params,
49 std::unique_ptr<std::string> encoded_data, 51 std::unique_ptr<std::string> encoded_data,
50 base::TimeTicks capture_timestamp, 52 base::TimeTicks capture_timestamp,
51 bool is_key_frame)>; 53 bool is_key_frame)>;
52 54
55 static CodecId GetPreferredCodecId();
56
53 VideoTrackRecorder(CodecId codec, 57 VideoTrackRecorder(CodecId codec,
54 const blink::WebMediaStreamTrack& track, 58 const blink::WebMediaStreamTrack& track,
55 const OnEncodedVideoCB& on_encoded_video_cb, 59 const OnEncodedVideoCB& on_encoded_video_cb,
56 int32_t bits_per_second); 60 int32_t bits_per_second);
57 ~VideoTrackRecorder() override; 61 ~VideoTrackRecorder() override;
58 62
59 void Pause(); 63 void Pause();
60 void Resume(); 64 void Resume();
61 65
62 void OnVideoFrameForTesting(const scoped_refptr<media::VideoFrame>& frame, 66 void OnVideoFrameForTesting(const scoped_refptr<media::VideoFrame>& frame,
(...skipping 23 matching lines...) Expand all
86 // Used to track the paused state during the initialization process. 90 // Used to track the paused state during the initialization process.
87 bool paused_before_init_; 91 bool paused_before_init_;
88 92
89 base::WeakPtrFactory<VideoTrackRecorder> weak_ptr_factory_; 93 base::WeakPtrFactory<VideoTrackRecorder> weak_ptr_factory_;
90 94
91 DISALLOW_COPY_AND_ASSIGN(VideoTrackRecorder); 95 DISALLOW_COPY_AND_ASSIGN(VideoTrackRecorder);
92 }; 96 };
93 97
94 } // namespace content 98 } // namespace content
95 #endif // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_ 99 #endif // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_
OLDNEW
« no previous file with comments | « content/renderer/media/media_recorder_handler.cc ('k') | content/renderer/media/video_track_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698