Chromium Code Reviews
DescriptionRevert of H264POC: Allow gaps in frame_num. (patchset #4 id:60001 of https://codereview.chromium.org/1967893002/ )
Reason for revert:
Causing perf test failure on mac.
crbug.com/614023
Original issue's description:
> H264POC: Allow gaps in frame_num.
>
> This change only removes the restriction, it does not try to detect
> all the cases where doing so is likely to result in problems. In most
> cases (specifically, those where the number of missing frames is much
> less than max_frame_num), the result will be correct.
>
> In incorrect cases the reorder queue could get confused and result in
> strange playback. The existing TODOs for the reorder queue explain how
> to reduce that, but also inserting dummy frames for gaps would help as
> well.
>
> BUG=550854
>
> Committed: https://crrev.com/b6f4380493084007dc4237d28153c0dd3e5d520b
> Cr-Commit-Position: refs/heads/master@{#395164}
TBR=posciak@chromium.org,sandersd@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=550854
Committed: https://crrev.com/86f1678a243f90e8767c73eed5870ee9986da36e
Cr-Commit-Position: refs/heads/master@{#395338}
Patch Set 1 #
Messages
Total messages: 11 (2 generated)
|
|||||||||||||||||||||||||||||||||||||