| Index: media/video/h264_poc.cc
|
| diff --git a/media/video/h264_poc.cc b/media/video/h264_poc.cc
|
| index 2934f25d3b79e8595717b95b807234fd0750f5f8..cfa91057e4490a63399da198204e15bcad6fb731 100644
|
| --- a/media/video/h264_poc.cc
|
| +++ b/media/video/h264_poc.cc
|
| @@ -67,14 +67,13 @@ bool H264POC::ComputePicOrderCnt(
|
| int32_t max_pic_order_cnt_lsb =
|
| 1 << (sps->log2_max_pic_order_cnt_lsb_minus4 + 4);
|
|
|
| - // Check for invalid (including duplicate) |frame_num| values. All cases are
|
| - // treated as gaps, which is to say that nothing is done. (Gaps don't affect
|
| - // POC computation.)
|
| - if (!slice_hdr.idr_pic_flag &&
|
| - slice_hdr.frame_num != (prev_frame_num_ + 1) % max_frame_num) {
|
| - if (!sps->gaps_in_frame_num_value_allowed_flag)
|
| - DLOG(WARNING) << "Invalid gap in frame_num";
|
| - }
|
| + // Note: Duplicate frame numbers are ignored. They occur in many videos
|
| + // despite appearing to be invalid according to the spec.
|
| + // TODO(sandersd): Check if these videos are using slices or have redundant
|
| + // streams.
|
| +
|
| + // Note: Gaps in frame numbers are also ignored. They do not affect POC
|
| + // computation.
|
|
|
| // Based on T-REC-H.264 8.2.1, "Decoding process for picture order
|
| // count", available from http://www.itu.int/rec/T-REC-H.264.
|
|
|