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

Side by Side Diff: remoting/client/plugin/pepper_view.h

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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // This class is an implementation of the ChromotingView for Pepper. It is 5 // This class is an implementation of the ChromotingView for Pepper. It is
6 // callable only on the Pepper thread. 6 // callable only on the Pepper thread.
7 7
8 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_ 8 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_
9 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_ 9 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_
10 10
(...skipping 20 matching lines...) Expand all
31 31
32 class ChromotingInstance; 32 class ChromotingInstance;
33 class ClientContext; 33 class ClientContext;
34 class FrameProducer; 34 class FrameProducer;
35 35
36 class PepperView : public FrameConsumer { 36 class PepperView : public FrameConsumer {
37 public: 37 public:
38 // Constructs a PepperView for the |instance|. The |instance| and |context| 38 // Constructs a PepperView for the |instance|. The |instance| and |context|
39 // must outlive this class. 39 // must outlive this class.
40 PepperView(ChromotingInstance* instance, ClientContext* context); 40 PepperView(ChromotingInstance* instance, ClientContext* context);
41 virtual ~PepperView(); 41 ~PepperView() override;
42 42
43 // Allocates buffers and passes them to the FrameProducer to render into until 43 // Allocates buffers and passes them to the FrameProducer to render into until
44 // the maximum number of buffers are in-flight. 44 // the maximum number of buffers are in-flight.
45 void Initialize(FrameProducer* producer); 45 void Initialize(FrameProducer* producer);
46 46
47 // FrameConsumer implementation. 47 // FrameConsumer implementation.
48 virtual void ApplyBuffer(const webrtc::DesktopSize& view_size, 48 void ApplyBuffer(const webrtc::DesktopSize& view_size,
49 const webrtc::DesktopRect& clip_area, 49 const webrtc::DesktopRect& clip_area,
50 webrtc::DesktopFrame* buffer, 50 webrtc::DesktopFrame* buffer,
51 const webrtc::DesktopRegion& region, 51 const webrtc::DesktopRegion& region,
52 const webrtc::DesktopRegion& shape) override; 52 const webrtc::DesktopRegion& shape) override;
53 virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) override; 53 void ReturnBuffer(webrtc::DesktopFrame* buffer) override;
54 virtual void SetSourceSize(const webrtc::DesktopSize& source_size, 54 void SetSourceSize(const webrtc::DesktopSize& source_size,
55 const webrtc::DesktopVector& dpi) override; 55 const webrtc::DesktopVector& dpi) override;
56 virtual PixelFormat GetPixelFormat() override; 56 PixelFormat GetPixelFormat() override;
57 57
58 // Updates the PepperView's size & clipping area, taking into account the 58 // Updates the PepperView's size & clipping area, taking into account the
59 // DIP-to-device scale factor. 59 // DIP-to-device scale factor.
60 void SetView(const pp::View& view); 60 void SetView(const pp::View& view);
61 61
62 // Returns the dimensions of the most recently displayed frame, in pixels. 62 // Returns the dimensions of the most recently displayed frame, in pixels.
63 const webrtc::DesktopSize& get_source_size() const { 63 const webrtc::DesktopSize& get_source_size() const {
64 return source_size_; 64 return source_size_;
65 } 65 }
66 66
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 bool frame_received_; 139 bool frame_received_;
140 140
141 pp::CompletionCallbackFactory<PepperView> callback_factory_; 141 pp::CompletionCallbackFactory<PepperView> callback_factory_;
142 142
143 DISALLOW_COPY_AND_ASSIGN(PepperView); 143 DISALLOW_COPY_AND_ASSIGN(PepperView);
144 }; 144 };
145 145
146 } // namespace remoting 146 } // namespace remoting
147 147
148 #endif // REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_ 148 #endif // REMOTING_CLIENT_PLUGIN_PEPPER_VIEW_H_
OLDNEW
« no previous file with comments | « remoting/client/plugin/pepper_port_allocator.cc ('k') | remoting/client/software_video_renderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698