Don't assume correct image format in CopyVpxImageToVideoFrame()

media/filters/vpx_video_decoder.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 MEDIA_FILTERS_VPX_VIDEO_DECODER_H_
 #define MEDIA_FILTERS_VPX_VIDEO_DECODER_H_
 
 #include "base/callback.h"
 #include "base/threading/thread_checker.h"
 #include "media/base/demuxer_stream.h"
@@ skipping 48 matching lines @@
   bool ConfigureDecoder(const VideoDecoderConfig& config);
 
   void CloseDecoder();
 
   // Try to decode |buffer| into |video_frame|. Return true if all decoding
   // succeeded. Note that decoding can succeed and still |video_frame| be
   // nullptr if there has been a partial decoding.
   bool VpxDecode(const scoped_refptr<DecoderBuffer>& buffer,
                  scoped_refptr<VideoFrame>* video_frame);
 
-  void CopyVpxImageToVideoFrame(const struct vpx_image* vpx_image,
+  bool CopyVpxImageToVideoFrame(const struct vpx_image* vpx_image,
                                 scoped_refptr<VideoFrame>* video_frame);
 
   base::ThreadChecker thread_checker_;
 
   DecoderState state_;
 
   OutputCB output_cb_;
 
   VideoDecoderConfig config_;
 
   vpx_codec_ctx* vpx_codec_;
   vpx_codec_ctx* vpx_codec_alpha_;
 
   // |memory_pool_| is a single-threaded memory pool used for VP9 decoding
   // with no alpha. |frame_pool_| is used for all other cases.
   class MemoryPool;
   scoped_refptr<MemoryPool> memory_pool_;
 
   VideoFramePool frame_pool_;
 
   DISALLOW_COPY_AND_ASSIGN(VpxVideoDecoder);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_FILTERS_VPX_VIDEO_DECODER_H_