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

Unified Diff: remoting/client/plugin/pepper_view.h

Issue 23677011: Byte-swap the video frame pixels before passing them to Java. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Narrow the synchronization block Created 7 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/client/plugin/pepper_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/pepper_view.h
diff --git a/remoting/client/plugin/pepper_view.h b/remoting/client/plugin/pepper_view.h
index 4027b620945fbcfd2196987b295298d28d3d671e..2b4e69bea08baa3a0fd180de0dd1b987865fa4f2 100644
--- a/remoting/client/plugin/pepper_view.h
+++ b/remoting/client/plugin/pepper_view.h
@@ -35,13 +35,15 @@ class FrameProducer;
class PepperView : public FrameConsumer {
public:
- // Constructs a PepperView for the |instance|. The |instance|, |context|
- // and |producer| must outlive this class.
- PepperView(ChromotingInstance* instance,
- ClientContext* context,
- FrameProducer* producer);
+ // Constructs a PepperView for the |instance|. The |instance| and |context|
+ // must outlive this class.
+ PepperView(ChromotingInstance* instance, ClientContext* context);
virtual ~PepperView();
+ // Allocates buffers and passes them to the FrameProducer to render into until
+ // the maximum number of buffers are in-flight.
+ void Initialize(FrameProducer* producer);
+
// FrameConsumer implementation.
virtual void ApplyBuffer(const webrtc::DesktopSize& view_size,
const webrtc::DesktopRect& clip_area,
@@ -50,6 +52,7 @@ class PepperView : public FrameConsumer {
virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) OVERRIDE;
virtual void SetSourceSize(const webrtc::DesktopSize& source_size,
const webrtc::DesktopVector& dpi) OVERRIDE;
+ virtual PixelFormat GetPixelFormat() OVERRIDE;
// Updates the PepperView's size & clipping area, taking into account the
// DIP-to-device scale factor.
@@ -74,10 +77,6 @@ class PepperView : public FrameConsumer {
// Frees a frame buffer previously allocated by AllocateBuffer.
void FreeBuffer(webrtc::DesktopFrame* buffer);
- // Allocates buffers and passes them to the FrameProducer to render into until
- // the maximum number of buffers are in-flight.
- void InitiateDrawing();
-
// Renders the parts of |buffer| identified by |region| to the view. If the
// clip area of the view has changed since the buffer was generated then
// FrameProducer is supplied the missed parts of |region|. The FrameProducer
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/client/plugin/pepper_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698