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

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

Issue 1315993003: Reland: Change VideoCaptureDeliverFrameCB's signature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address new changes Created 5 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
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 "base/macros.h" 8 #include "base/macros.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/memory/weak_ptr.h" 10 #include "base/memory/weak_ptr.h"
(...skipping 20 matching lines...) Expand all
31 base::Callback<void(const scoped_refptr<media::VideoFrame>& video_frame, 31 base::Callback<void(const scoped_refptr<media::VideoFrame>& video_frame,
32 const base::StringPiece& encoded_data, 32 const base::StringPiece& encoded_data,
33 base::TimeTicks capture_timestamp, 33 base::TimeTicks capture_timestamp,
34 bool is_key_frame)>; 34 bool is_key_frame)>;
35 35
36 VideoTrackRecorder(const blink::WebMediaStreamTrack& track, 36 VideoTrackRecorder(const blink::WebMediaStreamTrack& track,
37 const OnEncodedVideoCB& on_encoded_video_cb); 37 const OnEncodedVideoCB& on_encoded_video_cb);
38 ~VideoTrackRecorder() override; 38 ~VideoTrackRecorder() override;
39 39
40 void OnVideoFrame(const scoped_refptr<media::VideoFrame>& frame, 40 void OnVideoFrame(const scoped_refptr<media::VideoFrame>& frame,
41 const base::TimeTicks& capture_time); 41 base::TimeTicks capture_time);
42 42
43 private: 43 private:
44 friend class VideoTrackRecorderTest; 44 friend class VideoTrackRecorderTest;
45 45
46 // Used to check that we are destroyed on the same thread we were created. 46 // Used to check that we are destroyed on the same thread we were created.
47 base::ThreadChecker main_render_thread_checker_; 47 base::ThreadChecker main_render_thread_checker_;
48 48
49 // We need to hold on to the Blink track to remove ourselves on dtor. 49 // We need to hold on to the Blink track to remove ourselves on dtor.
50 blink::WebMediaStreamTrack track_; 50 blink::WebMediaStreamTrack track_;
51 51
52 // Forward declaration and member of an inner class to encode using VPx. 52 // Forward declaration and member of an inner class to encode using VPx.
53 class VpxEncoder; 53 class VpxEncoder;
54 const scoped_ptr<VpxEncoder> encoder_; 54 const scoped_ptr<VpxEncoder> encoder_;
55 55
56 base::WeakPtrFactory<VideoTrackRecorder> weak_factory_; 56 base::WeakPtrFactory<VideoTrackRecorder> weak_factory_;
57 57
58 DISALLOW_COPY_AND_ASSIGN(VideoTrackRecorder); 58 DISALLOW_COPY_AND_ASSIGN(VideoTrackRecorder);
59 }; 59 };
60 60
61 } // namespace content 61 } // namespace content
62 #endif // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_ 62 #endif // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_
OLDNEW
« no previous file with comments | « content/renderer/media/video_track_adapter.cc ('k') | content/renderer/media/video_track_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698