Fix VideoDecoderVpx to use pixel format the consumer wants.

remoting/codec/video_decoder.h

Index: remoting/codec/video_decoder.h
diff --git a/remoting/codec/video_decoder.h b/remoting/codec/video_decoder.h
index 3ed5f5a6ae9652070a9b11b34776a0a195261b45..4bf4d573c9242dac2fb1f31c12bedd148da18d07 100644
--- a/remoting/codec/video_decoder.h
+++ b/remoting/codec/video_decoder.h
@@ -5,7 +5,6 @@
 #ifndef REMOTING_CODEC_VIDEO_DECODER_H_
 #define REMOTING_CODEC_VIDEO_DECODER_H_
 
-
 namespace webrtc {
 class DesktopFrame;
 }  // namespace webrtc
@@ -17,9 +16,14 @@ class VideoPacket;
 // Interface for a decoder that decodes video packets.
 class VideoDecoder {
  public:
+  // List of supported pixel formats needed by various platforms.
+  enum class PixelFormat { BGRA, RGBA };

[Lambros, 2016/06/24 01:13:58]

Seems a bit weird having this enumeration in two places?

[Sergey Ulanov, 2016/06/24 02:01:09]

I agree. The proper place for this enum would be webrtc::DesktopFrame, but I
don't want to add it there because I think we won't be using DesktopFrame on the
client long-term.

+
   VideoDecoder() {}
   virtual ~VideoDecoder() {}
 
+  virtual void SetPixelFormat(PixelFormat pixel_format) = 0;
+
   // Decodes a video frame. Returns false in case of a failure. The caller must
   // pre-allocate a |frame| with the size specified in the |packet|.
   virtual bool DecodePacket(const VideoPacket& packet,