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

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

Issue 667123002: Standardize usage of virtual/override/final in remoting/ (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 | « remoting/host/fake_host_extension.h ('k') | remoting/host/fake_host_status_monitor.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/desktop_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 ~Session() override {}
21 21
22 // HostExtensionSession interface. 22 // HostExtensionSession interface.
23 virtual void OnCreateVideoCapturer( 23 void OnCreateVideoCapturer(
24 scoped_ptr<webrtc::DesktopCapturer>* encoder) override; 24 scoped_ptr<webrtc::DesktopCapturer>* encoder) override;
25 virtual void OnCreateVideoEncoder(scoped_ptr<VideoEncoder>* encoder) override; 25 void OnCreateVideoEncoder(scoped_ptr<VideoEncoder>* encoder) override;
26 virtual bool ModifiesVideoPipeline() const override; 26 bool ModifiesVideoPipeline() const override;
27 virtual bool OnExtensionMessage( 27 bool OnExtensionMessage(ClientSessionControl* client_session_control,
28 ClientSessionControl* client_session_control, 28 protocol::ClientStub* client_stub,
29 protocol::ClientStub* client_stub, 29 const protocol::ExtensionMessage& message) override;
30 const protocol::ExtensionMessage& message) override;
31 30
32 private: 31 private:
33 FakeExtension* extension_; 32 FakeExtension* extension_;
34 std::string message_type_; 33 std::string message_type_;
35 34
36 DISALLOW_COPY_AND_ASSIGN(Session); 35 DISALLOW_COPY_AND_ASSIGN(Session);
37 }; 36 };
38 37
39 FakeExtension::Session::Session( 38 FakeExtension::Session::Session(
40 FakeExtension* extension, const std::string& message_type) 39 FakeExtension* extension, const std::string& message_type)
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 scoped_ptr<HostExtensionSession> FakeExtension::CreateExtensionSession( 90 scoped_ptr<HostExtensionSession> FakeExtension::CreateExtensionSession(
92 ClientSessionControl* client_session_control, 91 ClientSessionControl* client_session_control,
93 protocol::ClientStub* client_stub) { 92 protocol::ClientStub* client_stub) {
94 DCHECK(!was_instantiated()); 93 DCHECK(!was_instantiated());
95 was_instantiated_ = true; 94 was_instantiated_ = true;
96 scoped_ptr<HostExtensionSession> session(new Session(this, message_type_)); 95 scoped_ptr<HostExtensionSession> session(new Session(this, message_type_));
97 return session.Pass(); 96 return session.Pass();
98 } 97 }
99 98
100 } // namespace remoting 99 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/fake_host_extension.h ('k') | remoting/host/fake_host_status_monitor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698