| Index: media/base/video_frame.cc
|
| diff --git a/media/base/video_frame.cc b/media/base/video_frame.cc
|
| index c0ed3ca2aa5268f59ec8bd9fbe09ed2840dba5b6..3be20dad900a5f160d00735709c589673b419234 100644
|
| --- a/media/base/video_frame.cc
|
| +++ b/media/base/video_frame.cc
|
| @@ -829,6 +829,8 @@ scoped_refptr<VideoFrame> VideoFrame::WrapExternalStorage(
|
| size_t data_offset) {
|
| DCHECK(IsStorageTypeMappable(storage_type));
|
|
|
| + // TODO(miu): This function should support any pixel format.
|
| + // http://crbug.com/555909
|
| if (format != PIXEL_FORMAT_I420) {
|
| DLOG(ERROR) << "Only PIXEL_FORMAT_I420 format supported: "
|
| << VideoPixelFormatToString(format);
|
| @@ -852,6 +854,10 @@ scoped_refptr<VideoFrame> VideoFrame::WrapExternalStorage(
|
| natural_size, timestamp);
|
| }
|
| frame->strides_[kYPlane] = coded_size.width();
|
| + // TODO(miu): This always rounds widths down, whereas VideoFrame::RowBytes()
|
| + // always rounds up. This inconsistency must be resolved. Perhaps a
|
| + // CommonAlignment() check should be made in IsValidConfig()?
|
| + // http://crbug.com/555909
|
| frame->strides_[kUPlane] = coded_size.width() / 2;
|
| frame->strides_[kVPlane] = coded_size.width() / 2;
|
| frame->data_[kYPlane] = data;
|
|
|