727809 - Video scrubber is locked

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P2)

Description

[Cristian Nicolae (:xti)]

Firefox 13.0a1 (2012-02-16)
20120216031231
http://hg.mozilla.org/mozilla-central/rev/a853f4017192
Device: Motorola Droid PRO
OS: Android 2.3.3

Steps to reproduce:
1. Open Fennec
2. Go to http://people.mozilla.com/~nhirata/html_tp/big_buck_bunny_480p.webm
3. Tap on screen for video controls
4. Tap on scrubber bar to skip back and forward in the video

Expected results:
The video will continue playing from the point where the last tap was performed.

Actual result:
Video scrubber doesn't work at all.

Note:
Even if video is switched to full screen mode, there will be the same actual result.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

fixed by bug 727817?

Comment 2

[Wesley Johnston (:wesj)]

This should have been fixed by bug 727817.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

based on Bug 729705... isn't this a dup of bug 657791?  He's using the same video.  We need to change the test case.

Comment 4

[Wesley Johnston (:wesj)]

Well, 727817 made the scrubber "work" on video's, even this one (need to add touch events supports for xul scale elements now so that you can drag them. I think there's a bug for that somewhere...).

This particular video just hits a different issue now that the scrubber does work.

Comment 5

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Ah.  Fair enough.  We would still need to change the test case. Setting myself to do that.

Comment 6

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

http://introducinghtml5.com/examples/ch04/ch4-full-fallback.html is a better test case; scrubber works.

litmus case : 48656, 48668 changed

Comment 7

[Cristian Nicolae (:xti)]

The scrubber is not locked anymore, but still I cannot forward the video to a desired point. If I try to do that, it will forward automatically to its end and there is no way to make the video to play from the beginning, except reloading the page or load it in a new tab.
The scrubber should move regarding the finger movement, until it will be released of the screen. Reopening bug

--
Firefox 13.0a1 (2012-03-01)
Device: Samsung Galaxy S2
OS: Android 2.3.4

Comment 8

[:Margaret Leibovic]

(In reply to Cristian Nicolae (:xti) from comment #7)
> The scrubber is not locked anymore, but still I cannot forward the video to
> a desired point. If I try to do that, it will forward automatically to its
> end and there is no way to make the video to play from the beginning, except
> reloading the page or load it in a new tab.
> The scrubber should move regarding the finger movement, until it will be
> released of the screen. Reopening bug

Cristian, do you mean you want to be able to drag your finger and have the scrubber follow it? If so, I think we should file a different bug about that because AIUI that's a harder problem that XUL fennec didn't even solve (wesj mentioned this in comment 4).

If you single tap on different points on the scrubber bar, does the video move to that point? If so, we should mark this bug as FIXED.

Comment 9

[Cristian Nicolae (:xti)]

(In reply to Margaret Leibovic [:margaret] from comment #8)
> 
> Cristian, do you mean you want to be able to drag your finger and have the
> scrubber follow it? If so, I think we should file a different bug about that
> because AIUI that's a harder problem that XUL fennec didn't even solve (wesj
> mentioned this in comment 4).
> 
> If you single tap on different points on the scrubber bar, does the video
> move to that point? If so, we should mark this bug as FIXED.

I will file a bug for the dragging case.

I tried to do a single tap, and as I mentioned in my previous comment, the scrubber goes automatically to the end of the video and it cannot be reset, only by reloading the page or loading it in a new tab.

Comment 10

[Wesley Johnston (:wesj)]

The video listed here suffers from bug 657791. See comment 6.

Comment 11

[Cristian Nicolae (:xti)]

Testing this using the video from comment 6, everything works as expected. Should I close this bug in this case?

Also bug 732052 was filled regarding the dragging issue. As Wesley said, we should use touch events for video scrubber

Comment 12

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Try using this video:
http://introducinghtml5.com/examples/ch04/ch4-full-fallback.html

Comment 13

[Cristian Nicolae (:xti)]

(In reply to Naoki Hirata :nhirata from comment #12)
> Try using this video:
> http://introducinghtml5.com/examples/ch04/ch4-full-fallback.html

It works as expected using this video. I will close the bug as:

Verified fixed on:

Firefox 13.0a1 (2012-03-01)
20120301031135
http://hg.mozilla.org/mozilla-central/rev/1c3b291d0830

--
Device: Samsung Galaxy S2
OS: Android 2.3.4