Index: content/renderer/media/webrtc/video_destination_handler.cc |
diff --git a/content/renderer/media/webrtc/video_destination_handler.cc b/content/renderer/media/webrtc/video_destination_handler.cc |
index 160537165d7912e4c514f6d49272f9dd60d0adde..f69949010ea54c1023352f1f39fdf72e5ae6c9e5 100644 |
--- a/content/renderer/media/webrtc/video_destination_handler.cc |
+++ b/content/renderer/media/webrtc/video_destination_handler.cc |
@@ -41,6 +41,12 @@ class PpFrameWriter::FrameWriterDelegate |
private: |
friend class base::RefCountedThreadSafe<FrameWriterDelegate>; |
+ enum PixelEndian { |
+ UNKNOWN, |
+ AXXX, |
+ XXXA, |
+ }; |
+ |
virtual ~FrameWriterDelegate(); |
void StartDeliverOnIO(const VideoCaptureDeliverFrameCB& frame_callback); |
@@ -52,9 +58,11 @@ class PpFrameWriter::FrameWriterDelegate |
scoped_refptr<base::MessageLoopProxy> io_message_loop_; |
- // |frame_pool_| and |new_frame_callback_| are only used on the IO-thread. |
+ // |frame_pool_|, |new_frame_callback_| and |endian_| are only used on the |
+ // IO-thread. |
media::VideoFramePool frame_pool_; |
VideoCaptureDeliverFrameCB new_frame_callback_; |
+ PixelEndian endian_; |
// Used to DCHECK that we are called on the main render thread. |
base::ThreadChecker thread_checker_; |
@@ -62,7 +70,7 @@ class PpFrameWriter::FrameWriterDelegate |
PpFrameWriter::FrameWriterDelegate::FrameWriterDelegate( |
const scoped_refptr<base::MessageLoopProxy>& io_message_loop_proxy) |
- : io_message_loop_(io_message_loop_proxy) { |
+ : io_message_loop_(io_message_loop_proxy), endian_(UNKNOWN) { |
} |
PpFrameWriter::FrameWriterDelegate::~FrameWriterDelegate() { |
@@ -101,6 +109,8 @@ void PpFrameWriter::FrameWriterDelegate::DeliverFrame( |
<< "The image_data's mapped bitmap is NULL."; |
return; |
} |
+ // We only support PP_IMAGEDATAFORMAT_BGRA_PREMUL at the moment. |
+ DCHECK(image_data->format() == PP_IMAGEDATAFORMAT_BGRA_PREMUL); |
io_message_loop_->PostTaskAndReply( |
FROM_HERE, |
base::Bind(&FrameWriterDelegate::DeliverFrameOnIO, this, |
@@ -153,15 +163,42 @@ void PpFrameWriter::FrameWriterDelegate::DeliverFrameOnIO( |
MediaStreamVideoSource::kUnknownFrameRate, |
media::PIXEL_FORMAT_YV12); |
- libyuv::BGRAToI420(data, |
- stride, |
- new_frame->data(media::VideoFrame::kYPlane), |
- new_frame->stride(media::VideoFrame::kYPlane), |
- new_frame->data(media::VideoFrame::kUPlane), |
- new_frame->stride(media::VideoFrame::kUPlane), |
- new_frame->data(media::VideoFrame::kVPlane), |
- new_frame->stride(media::VideoFrame::kVPlane), |
- frame_size.width(), frame_size.height()); |
+ // TODO(magjed): Remove this and always use libyuv::ARGBToI420 when |
+ // crbug/426020 is fixed. |
+ // Due to confusion about endianness, we try to determine it from the data. |
+ // The alpha channel is always 255. It is unlikely for other color channels to |
+ // be 255, so we will most likely break on the first few pixels in the first |
+ // frame. |
+ uint8* row_ptr = data; |
+ for (int y = 0; y < height && endian_ == UNKNOWN; ++y) { |
+ for (int x = 0; x < width; ++x) { |
+ if (row_ptr[x * 4 + 0] != 255) { |
+ endian_ = XXXA; |
+ break; |
+ } |
+ if (row_ptr[x * 4 + 3] != 255) { |
+ endian_ = AXXX; |
+ break; |
+ } |
+ } |
+ row_ptr += stride; |
+ } |
+ if (endian_ == UNKNOWN) { |
+ LOG(WARNING) << "PpFrameWriter::FrameWriterDelegate::DeliverFrameOnIO - " |
+ << "Could not determine endianness."; |
+ } |
+ const auto xxxxToI420 = |
+ (endian_ == AXXX) ? libyuv::BGRAToI420 : libyuv::ARGBToI420; |
brucedawson
2014/10/22 17:58:41
This reads very strangely. If endian_ is AXXX then
magjed_chromium
2014/10/22 18:18:14
I agree that it looks strange, but AXXX/XXXA is in
|
+ xxxxToI420(data, |
+ stride, |
+ new_frame->data(media::VideoFrame::kYPlane), |
+ new_frame->stride(media::VideoFrame::kYPlane), |
+ new_frame->data(media::VideoFrame::kUPlane), |
+ new_frame->stride(media::VideoFrame::kUPlane), |
+ new_frame->data(media::VideoFrame::kVPlane), |
+ new_frame->stride(media::VideoFrame::kVPlane), |
+ width, |
+ height); |
// The local time when this frame is generated is unknown so give a null |
// value to |estimated_capture_time|. |