Closed Bug 1248152 Opened 8 years ago Closed 8 years ago

Short video, mp4 format, flickers frequently when it starts to loop

Categories

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

44 Branch
x86_64
Windows 10
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1257013
Tracking Status
firefox44 --- wontfix
firefox45 --- wontfix
firefox46 --- wontfix
firefox47 --- fixed
firefox48 --- fixed

People

(Reporter: detritusbg, Unassigned)

References

()

Details

(Keywords: correctness, html5, regression)

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:44.0) Gecko/20100101 Firefox/44.0
Build ID: 20160210153822

Steps to reproduce:

While working on a new page design, I put a small video (2 seconds long) which must be looping itself all the time. Then I noticed that the video flickers occasionally when it starts to loop. I tried with longer videos and I think that the problem is persistent only in short videos - 1 to 10 second long.
I created a webpage where you can see for yourself: http://subsolardesigns.com/firefox-test/


Actual results:

The first two, in mp4 format will flicker occasionally, but the webm won't flicker. You might have to refresh the page a couple of times and wait for a while to see it. The videos are 2 seconds short so it won't take very long to spot.


Expected results:

There should be no flickering in all 3 videos. When I tested it in Chrome there is no flickering.
Keywords: correctness, html5
OS: Unspecified → Windows 10
Hardware: Unspecified → x86_64
I see a subtle stagnation while loop to start for mp4, higher rate seems to be more obvious. It may be a delay for every reparsing.
Component: Untriaged → Audio/Video: Playback
Keywords: correctness, html5testcase
Product: Firefox → Core
Via local build,
Last Good: b06457a116b2
First Bad: ab9dce123576
Blocks: 1196408
Chris - I'll let you find someone to fix this.
Flags: needinfo?(cpearce)
Chris -- This is flagged as a carryover regression (starting in Fx44).  Can you find an owner?  (Re-needinfoing you to get this to the top of your stack.)
Flags: needinfo?(cpearce)
Flags: needinfo?(cpearce)
This should be fixed by bug 1257013.
Flags: needinfo?(matt.woodrow)
Matt (and Chris) -- Since this is effectively a dup of bug 1257013, should we uplift the fix on bug 1257013 to resolve this regression, or does that not make sense?   Also, Fx47 is listed as unaffected here, but the fix on bug 1257013 didn't land until Fx48.  Is Fx47 affected by this bug or not?  Thanks.
Flags: needinfo?(matt.woodrow)
I'm pretty sure 47 is affected, yes.

I'd personally rather live with this bug and not uplift bug 1257013, since it's a big change and I want it to ride the trains.
Flags: needinfo?(matt.woodrow)
Thanks, Matt.  cpearce agrees.  The fix for this will ride the trains in bug 1257013.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(cpearce)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.