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

Side by Side Diff: remoting/client/jni/jni_frame_consumer.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: Add PixelFormat to FrameConsumer interface Created 7 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 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 REMOTING_CLIENT_JNI_JNI_FRAME_CONSUMER_H_ 5 #ifndef REMOTING_CLIENT_JNI_JNI_FRAME_CONSUMER_H_
6 #define REMOTING_CLIENT_JNI_JNI_FRAME_CONSUMER_H_ 6 #define REMOTING_CLIENT_JNI_JNI_FRAME_CONSUMER_H_
7 7
8 #include "remoting/client/frame_consumer.h" 8 #include "remoting/client/frame_consumer.h"
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 18 matching lines...) Expand all
29 void set_frame_producer(FrameProducer* producer); 29 void set_frame_producer(FrameProducer* producer);
30 30
31 // FrameConsumer implementation. 31 // FrameConsumer implementation.
32 virtual void ApplyBuffer(const SkISize& view_size, 32 virtual void ApplyBuffer(const SkISize& view_size,
33 const SkIRect& clip_area, 33 const SkIRect& clip_area,
34 webrtc::DesktopFrame* buffer, 34 webrtc::DesktopFrame* buffer,
35 const SkRegion& region) OVERRIDE; 35 const SkRegion& region) OVERRIDE;
36 virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) OVERRIDE; 36 virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) OVERRIDE;
37 virtual void SetSourceSize(const SkISize& source_size, 37 virtual void SetSourceSize(const SkISize& source_size,
38 const SkIPoint& dpi) OVERRIDE; 38 const SkIPoint& dpi) OVERRIDE;
39 virtual PixelFormat GetPixelFormat() OVERRIDE;
39 40
40 private: 41 private:
41 // Variables are to be used from the display thread. 42 // Variables are to be used from the display thread.
42 43
43 // Used to obtain task runner references and make calls to Java methods. 44 // Used to obtain task runner references and make calls to Java methods.
44 ChromotingJniRuntime* jni_runtime_; 45 ChromotingJniRuntime* jni_runtime_;
45 46
46 // Whether we're currently in the constructor, and should deallocate the 47 // Whether we're currently in the constructor, and should deallocate the
47 // buffer instead of passing it back to the producer. 48 // buffer instead of passing it back to the producer.
48 bool in_dtor_; 49 bool in_dtor_;
49 50
50 FrameProducer* frame_producer_; 51 FrameProducer* frame_producer_;
51 SkISize view_size_; 52 SkISize view_size_;
52 SkIRect clip_area_; 53 SkIRect clip_area_;
53 54
54 // If |provide_buffer_|, allocates a new buffer of |view_size_|, informs 55 // If |provide_buffer_|, allocates a new buffer of |view_size_|, informs
55 // Java about it, and tells the producer to draw onto it. Otherwise, no-op. 56 // Java about it, and tells the producer to draw onto it. Otherwise, no-op.
56 void AllocateBuffer(); 57 void AllocateBuffer();
57 58
58 DISALLOW_COPY_AND_ASSIGN(JniFrameConsumer); 59 DISALLOW_COPY_AND_ASSIGN(JniFrameConsumer);
59 }; 60 };
60 61
61 } // namespace remoting 62 } // namespace remoting
62 63
63 #endif 64 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698