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

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

Issue 1019033003: RtcVideo{Encoder,Decoder}{Factory,} minor cleanup: consts, range based loops, comments. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_RTC_VIDEO_ENCODER_FACTORY_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_
6 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_ 6 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "base/memory/ref_counted.h" 11 #include "base/memory/ref_counted.h"
12 #include "content/common/content_export.h" 12 #include "content/common/content_export.h"
13 #include "third_party/libjingle/source/talk/media/webrtc/webrtcvideoencoderfacto ry.h" 13 #include "third_party/libjingle/source/talk/media/webrtc/webrtcvideoencoderfacto ry.h"
14 14
15 namespace media { 15 namespace media {
16
Pawel Osciak 2015/03/19 01:26:33 Ditto.
mcasas 2015/03/19 02:54:26 Acknowledged.
17 class GpuVideoAcceleratorFactories; 16 class GpuVideoAcceleratorFactories;
18
19 } // namespace media 17 } // namespace media
20 18
21 namespace content { 19 namespace content {
22 20
23 // This class creates RTCVideoEncoder instances (each wrapping a 21 // This class creates RTCVideoEncoder instances (each wrapping a
24 // media::VideoEncodeAccelerator) on behalf of the WebRTC stack. 22 // media::VideoEncodeAccelerator) on behalf of the WebRTC stack.
25 class CONTENT_EXPORT RTCVideoEncoderFactory 23 class CONTENT_EXPORT RTCVideoEncoderFactory
26 : NON_EXPORTED_BASE(public cricket::WebRtcVideoEncoderFactory) { 24 : NON_EXPORTED_BASE(public cricket::WebRtcVideoEncoderFactory) {
27 public: 25 public:
28 explicit RTCVideoEncoderFactory( 26 explicit RTCVideoEncoderFactory(
29 const scoped_refptr<media::GpuVideoAcceleratorFactories>& gpu_factories); 27 const scoped_refptr<media::GpuVideoAcceleratorFactories>& gpu_factories);
30 ~RTCVideoEncoderFactory() override; 28 ~RTCVideoEncoderFactory() override;
31 29
32 // cricket::WebRtcVideoEncoderFactory implementation. 30 // cricket::WebRtcVideoEncoderFactory implementation.
33 webrtc::VideoEncoder* CreateVideoEncoder( 31 webrtc::VideoEncoder* CreateVideoEncoder(
34 webrtc::VideoCodecType type) override; 32 webrtc::VideoCodecType type) override;
35 const std::vector<VideoCodec>& codecs() const override; 33 const std::vector<VideoCodec>& codecs() const override;
36 void DestroyVideoEncoder(webrtc::VideoEncoder* encoder) override; 34 void DestroyVideoEncoder(webrtc::VideoEncoder* encoder) override;
37 35
38 private: 36 private:
39 const scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_; 37 const scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_;
40 38
41 // Codec support list of cricket::WebRtcVideoEncoderFactory::VideoCodec 39 // List of supported cricket::WebRtcVideoEncoderFactory::VideoCodec.
42 // instances.
43 std::vector<VideoCodec> codecs_; 40 std::vector<VideoCodec> codecs_;
44 41
45 DISALLOW_COPY_AND_ASSIGN(RTCVideoEncoderFactory); 42 DISALLOW_COPY_AND_ASSIGN(RTCVideoEncoderFactory);
46 }; 43 };
47 44
48 } // namespace content 45 } // namespace content
49 46
50 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_ 47 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698