Byte-swap the video frame pixels before passing them to Java.

remoting/client/frame_consumer.h

Index: remoting/client/frame_consumer.h
diff --git a/remoting/client/frame_consumer.h b/remoting/client/frame_consumer.h
index b5f937ca5acd07db7098115fcc83eca38d2c2eee..f11541fb91fcd7d9fc8ba0b651697987757204bc 100644
--- a/remoting/client/frame_consumer.h
+++ b/remoting/client/frame_consumer.h
@@ -18,6 +18,13 @@ namespace remoting {
 
 class FrameConsumer {
  public:
+
+  // List of supported pixel formats needed by various platforms.
+  enum PixelFormat {
+    FORMAT_BGRA,  // Used by the Pepper plugin.
+    FORMAT_RGBA,  // Used for Android's Bitmap class.
+  };
+
   // Accepts a buffer to be painted to the screen. The buffer's dimensions and
   // relative position within the frame are specified by |clip_area|. Only
   // pixels falling within |region| and the current clipping area are painted.
@@ -40,6 +47,10 @@ class FrameConsumer {
   virtual void SetSourceSize(const SkISize& source_size,
                              const SkIPoint& dpi) = 0;
 
+  // Returns the preferred pixel encoding for the platform. This may be called
+  // from any thread.

[Sergey Ulanov, 2013/09/27 21:51:13]

Don't need the threading comment. This interface is not thread-safe. No need to
make it different for this method.

[Lambros, 2013/09/28 00:22:18]

Done.

+  virtual PixelFormat GetPixelFormat() = 0;
+
  protected:
   FrameConsumer() {}
   virtual ~FrameConsumer() {}