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

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

Issue 639393002: Remove Observer methods in RTCVideoEncoderFactory. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « no previous file | content/renderer/media/rtc_video_encoder_factory.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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"
(...skipping 14 matching lines...) Expand all
25 class CONTENT_EXPORT RTCVideoEncoderFactory 25 class CONTENT_EXPORT RTCVideoEncoderFactory
26 : NON_EXPORTED_BASE(public cricket::WebRtcVideoEncoderFactory) { 26 : NON_EXPORTED_BASE(public cricket::WebRtcVideoEncoderFactory) {
27 public: 27 public:
28 explicit RTCVideoEncoderFactory( 28 explicit RTCVideoEncoderFactory(
29 const scoped_refptr<media::GpuVideoAcceleratorFactories>& gpu_factories); 29 const scoped_refptr<media::GpuVideoAcceleratorFactories>& gpu_factories);
30 virtual ~RTCVideoEncoderFactory(); 30 virtual ~RTCVideoEncoderFactory();
31 31
32 // cricket::WebRtcVideoEncoderFactory implementation. 32 // cricket::WebRtcVideoEncoderFactory implementation.
33 virtual webrtc::VideoEncoder* CreateVideoEncoder( 33 virtual webrtc::VideoEncoder* CreateVideoEncoder(
34 webrtc::VideoCodecType type) override; 34 webrtc::VideoCodecType type) override;
35 virtual void AddObserver(Observer* observer) override;
36 virtual void RemoveObserver(Observer* observer) override;
37 virtual const std::vector<VideoCodec>& codecs() const override; 35 virtual const std::vector<VideoCodec>& codecs() const override;
38 virtual void DestroyVideoEncoder(webrtc::VideoEncoder* encoder) override; 36 virtual void DestroyVideoEncoder(webrtc::VideoEncoder* encoder) override;
39 37
40 private: 38 private:
41 const scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_; 39 const scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_;
42 40
43 // Codec support list of cricket::WebRtcVideoEncoderFactory::VideoCodec 41 // Codec support list of cricket::WebRtcVideoEncoderFactory::VideoCodec
44 // instances. 42 // instances.
45 std::vector<VideoCodec> codecs_; 43 std::vector<VideoCodec> codecs_;
46 44
47 DISALLOW_COPY_AND_ASSIGN(RTCVideoEncoderFactory); 45 DISALLOW_COPY_AND_ASSIGN(RTCVideoEncoderFactory);
48 }; 46 };
49 47
50 } // namespace content 48 } // namespace content
51 49
52 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_ 50 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_FACTORY_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/media/rtc_video_encoder_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698