DescriptionRevert of Relanding 'Always notify the MediaPlayer of any seek' patch (patchset #8 id:180001 of https://codereview.chromium.org/456343002/)
Reason for revert:
Android perf bots failure in media.android.tough_video_cases (timeout)
https://code.google.com/p/chromium/issues/detail?id=448092
Original issue's description:
> Relanding 'Always notify the MediaPlayer of any seek' patch
>
> This change removes short circuit logic for seeking to the current position.
> There are certain situations, like seeking to a truncated duration, where the
> HTMLMediaElement is not in the best position to determine whether a call to
> the MediaPlayer is necessary or not. This change allows the
> MediaPlayer/WebMediaPlayer implementation to determine what the best course
> of action should be. This also makes the seeking behavior more spec
> compliant because even a seek to the current position should have the
> seeking attribute be true until a stable state occurs. This is guaranteed
> when calling into the MediaPlayer and allows all seeks to be treated the
> same at the HTMLMediaElement level.
>
> BUG=266631
> TEST=LayoutTests/media/video-seek-to-current-position.html
>
> Original CL: https://codereview.chromium.org/431903003/
>
> Depends on the chromium patch https://codereview.chromium.org/685993002/
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188150
TBR=philipj@opera.com,dalecurtis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=266631
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188281
Patch Set 1 #
Created: 5 years, 11 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|