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

Side by Side Diff: remoting/host/fake_host_extension.cc

Issue 455073004: Switch DesktopEnvironment to return a DesktopCapturer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove ShapedDesktopCapturer::Create() 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
« no previous file with comments | « remoting/host/fake_desktop_environment.cc ('k') | remoting/host/fake_screen_capturer.h » ('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 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 #include "remoting/host/fake_host_extension.h" 5 #include "remoting/host/fake_host_extension.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "remoting/codec/video_encoder.h" 10 #include "remoting/codec/video_encoder.h"
11 #include "remoting/host/host_extension_session.h" 11 #include "remoting/host/host_extension_session.h"
12 #include "remoting/proto/control.pb.h" 12 #include "remoting/proto/control.pb.h"
13 #include "third_party/webrtc/modules/desktop_capture/screen_capturer.h" 13 #include "third_party/webrtc/modules/desktop_capture/desktop_capturer.h"
14 14
15 namespace remoting { 15 namespace remoting {
16 16
17 class FakeExtension::Session : public HostExtensionSession { 17 class FakeExtension::Session : public HostExtensionSession {
18 public: 18 public:
19 Session(FakeExtension* extension, const std::string& message_type); 19 Session(FakeExtension* extension, const std::string& message_type);
20 virtual ~Session() {} 20 virtual ~Session() {}
21 21
22 virtual scoped_ptr<webrtc::ScreenCapturer> OnCreateVideoCapturer( 22 virtual scoped_ptr<webrtc::DesktopCapturer> OnCreateVideoCapturer(
23 scoped_ptr<webrtc::ScreenCapturer> encoder) OVERRIDE; 23 scoped_ptr<webrtc::DesktopCapturer> encoder) OVERRIDE;
24 virtual scoped_ptr<VideoEncoder> OnCreateVideoEncoder( 24 virtual scoped_ptr<VideoEncoder> OnCreateVideoEncoder(
25 scoped_ptr<VideoEncoder> encoder) OVERRIDE; 25 scoped_ptr<VideoEncoder> encoder) OVERRIDE;
26 virtual bool ModifiesVideoPipeline() const OVERRIDE; 26 virtual bool ModifiesVideoPipeline() const OVERRIDE;
27 virtual bool OnExtensionMessage( 27 virtual bool OnExtensionMessage(
28 ClientSessionControl* client_session_control, 28 ClientSessionControl* client_session_control,
29 protocol::ClientStub* client_stub, 29 protocol::ClientStub* client_stub,
30 const protocol::ExtensionMessage& message) OVERRIDE; 30 const protocol::ExtensionMessage& message) OVERRIDE;
31 31
32 private: 32 private:
33 FakeExtension* extension_; 33 FakeExtension* extension_;
34 std::string message_type_; 34 std::string message_type_;
35 35
36 DISALLOW_COPY_AND_ASSIGN(Session); 36 DISALLOW_COPY_AND_ASSIGN(Session);
37 }; 37 };
38 38
39 FakeExtension::Session::Session( 39 FakeExtension::Session::Session(
40 FakeExtension* extension, const std::string& message_type) 40 FakeExtension* extension, const std::string& message_type)
41 : extension_(extension), 41 : extension_(extension),
42 message_type_(message_type) { 42 message_type_(message_type) {
43 } 43 }
44 44
45 scoped_ptr<webrtc::ScreenCapturer> 45 scoped_ptr<webrtc::DesktopCapturer>
46 FakeExtension::Session::OnCreateVideoCapturer( 46 FakeExtension::Session::OnCreateVideoCapturer(
47 scoped_ptr<webrtc::ScreenCapturer> capturer) { 47 scoped_ptr<webrtc::DesktopCapturer> capturer) {
48 extension_->has_wrapped_video_capturer_ = true; 48 extension_->has_wrapped_video_capturer_ = true;
49 if (extension_->steal_video_capturer_) { 49 if (extension_->steal_video_capturer_) {
50 capturer.reset(); 50 capturer.reset();
51 } 51 }
52 return capturer.Pass(); 52 return capturer.Pass();
53 } 53 }
54 54
55 scoped_ptr<VideoEncoder> FakeExtension::Session::OnCreateVideoEncoder( 55 scoped_ptr<VideoEncoder> FakeExtension::Session::OnCreateVideoEncoder(
56 scoped_ptr<VideoEncoder> encoder) { 56 scoped_ptr<VideoEncoder> encoder) {
57 extension_->has_wrapped_video_encoder_ = true; 57 extension_->has_wrapped_video_encoder_ = true;
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 DCHECK(was_instantiated()); 113 DCHECK(was_instantiated());
114 return has_wrapped_video_capturer_; 114 return has_wrapped_video_capturer_;
115 } 115 }
116 116
117 bool FakeExtension::has_handled_message() { 117 bool FakeExtension::has_handled_message() {
118 DCHECK(was_instantiated()); 118 DCHECK(was_instantiated());
119 return has_handled_message_; 119 return has_handled_message_;
120 } 120 }
121 121
122 } // namespace remoting 122 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/fake_desktop_environment.cc ('k') | remoting/host/fake_screen_capturer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698