686942 - Video starts playing after seeking to end

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Matthew Gregan [:kinetik]]

Open the testcase, use tab to switch focus to the video (first element), hit "end", then "home.  Video will start playing.

Per the spec, the video starts paused, so it should not start playing in this situation.  I suspect the code that deals with video ended in an unpaused state is causing the video to start playing when seeking to the end of the media.

Comment 1

[Matthew Gregan [:kinetik]]

Once the video starts playing, if you seek a second time, the video is paused after seeking.

Comment 2

[cajbir (:cajbir)]

Attachment: Fix

Respect the state of the paused attribute after seeking.

Comment 3

[cajbir (:cajbir)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7a0db9bd7c37

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/7a0db9bd7c37

Comment 5

[cajbir (:cajbir)]

Attachment: Adds mochitest

Comment 6

[cajbir (:cajbir)]

Pushed test (attachment 563286 [details] [diff] [review]):
https://hg.mozilla.org/integration/mozilla-inbound/rev/728784a13fb4

Comment 7

[Marco Bonardo [:mak]]

https://hg.mozilla.org/mozilla-central/rev/728784a13fb4

Comment 11

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e006ce87ed39cad29876fc32f1568d7a7485d3dd

Comment 12

[Alex Chronopoulos [:achronop]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b8a90fc2a4efc6163aa7c0d1624b618b0b19f84a

Comment 13

[Alex Chronopoulos [:achronop]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0f0b2c65001bebe3a9543f77dd364ffa1db4c822

Comment 14

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=508a644c26ddab5c3743c774b7c673d5dd4a77a3

Comment 15

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1c4d23eb8b6b729aebe005216b6d2596e089c69c