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

remoting/client/frame_consumer.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_CLIENT_FRAME_CONSUMER_H_
 #define REMOTING_CLIENT_FRAME_CONSUMER_H_
 
 #include "base/basictypes.h"
 #include "third_party/skia/include/core/SkRect.h"
 #include "third_party/skia/include/core/SkRegion.h"
 #include "third_party/skia/include/core/SkSize.h"
 
 namespace webrtc {
 class DesktopFrame;
 }  // namespace webrtc
 
 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.
   // The function assumes that the passed buffer was scaled to fit a window
   // having |view_size| dimensions.
   //
   // N.B. Both |clip_area| and |region| are in output coordinates relative to
   // the frame.
   virtual void ApplyBuffer(const SkISize& view_size,
                            const SkIRect& clip_area,
                            webrtc::DesktopFrame* buffer,
                            const SkRegion& region) = 0;
 
   // Accepts a buffer that couldn't be used for drawing for any reason (shutdown
   // is in progress, the view area has changed, etc.). The accepted buffer can
   // be freed or reused for another drawing operation.
   virtual void ReturnBuffer(webrtc::DesktopFrame* buffer) = 0;
 
   // Set the dimension of the entire host screen.
   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() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(FrameConsumer);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_FRAME_CONSUMER_H_