1295440 - Make HTMLMediaElement::SeekToNextFrame() reject promise with meaningful information

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Tzuhao Kuo [:kaku]]

Per discussion at bug1276272 comment30, the HTMLMediaElement::seekToNextFrame() should reject promise with some meaningful information, instead of just reject with Undefined.

Comment 1

[Tzuhao Kuo [:kaku]]

Attachment: Bug 1295440 - Make HTMLMediaElement::SeekToNextFrame() reject promise with meaningful information;

Review commit: https://reviewboard.mozilla.org/r/72134/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/72134/

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8781795 [details]
Bug 1295440 - Make HTMLMediaElement::SeekToNextFrame() reject promise with meaningful information;

https://reviewboard.mozilla.org/r/72134/#review69688

Comment 3

[Tzuhao Kuo [:kaku]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9ff26000a94b
The Mdm and Mm failures on the Android platform is not thing to do with this bug, they also happens on central.

Thanks for the review.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f430c8ef2789
Make HTMLMediaElement::SeekToNextFrame() reject promise with meaningful information; r=jwwang

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/f430c8ef2789