Closed Bug 791731 Opened 12 years ago Closed 12 years ago

Animations on apple.com are not smooth, spend a lot of CPU time in the JPEG decoder

Categories

(Core :: Graphics: ImageLib, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18
Tracking Status
firefox17 + fixed
firefox18 + fixed

People

(Reporter: daniel.nr01, Assigned: khuey)

References

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/18.0 Firefox/18.0 Build ID: 20120914030538 Steps to reproduce: 1. Visit http://www.apple.com/macbook-pro/features/#seq 2. Watch animation of lid opening Actual results: The animation flickers Expected results: The animation should be smooth The animation looks good in current stable (Firefox 15), and also in Chrome and in IE, but not in Aurora or Nightly. This bug is quite possibly related to Bug 683290 and perhaps Bug 685516. I also came across this bug while "streaming" video from an IP-camera. A jpeg image from the camera was loaded onto the page. I then loaded a new image in the background and on onload I switched the old image src out for the just loaded. Then the next frame started loading in the background. The point is this simple solution works great in 15 (and Chrome etc.) but the video flickers a lot in Nightly.
Yeah, this regressed when those bugs landed.
Status: UNCONFIRMED → NEW
Ever confirmed: true
On my powerful Linux x64 desktop machine, the animation in comment 0 is smooth, but the following STR shows the same flickering effect: * http://www.apple.com/ipad/features/ * Scroll down to the smart cover section, second from the bottom. * "Unwrap" and "wrap" the cover by dragging your mouse. Profiling with perf shows that we're spending all our time in the JPEG decoder, so this is very likely a discarding issue. See also https://docs.google.com/document/pub?id=1GWTMLjqQsQS45FWwqNG9ztQTdGF48hQYpjQHR_d1WsI for a description of what Apple is doing here, which is pretty rad.
Summary: Flickering when switching images in DOM → Animations on apple.com are not smooth, spend a lot of CPU time in the JPEG decoder
Like I asked in bug 791779, I don't get why we're discarding these images immediately. Shouldn't we hold on to them until the discard timer fires?
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Yeah, we shouldn't discard immediately. Part 1 of Bug 784591 gives us the option, and then we'll just need a small patch here to flip the switch for this case.
Depends on: 784591
Attached patch PatchSplinter Review
Don't RequestDiscard from UnbindFromTree.
Attachment #667032 - Flags: review?(justin.lebar+bug)
Attachment #667032 - Flags: review?(justin.lebar+bug) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Comment on attachment 667032 [details] [diff] [review] Patch We should take this patch on aurora too. [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 683290 User impact if declined: Increased flickering and CPU usage when switching between certain websites. Testing completed (on m-c, etc.): Landed on m-c. Risk to taking this patch (and alternatives if risky): Low risk String or UUID changes made by this patch: None.
Attachment #667032 - Flags: approval-mozilla-aurora?
Still flickers here using latest hourly m-c win32 build on win7 x64 https://hg.mozilla.org/mozilla-central/rev/5ac283a12f02
Did if flicker for you before Bug 683290 landed? I don't see any flicker here.
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #11) > Did if flicker for you before Bug 683290 landed? I don't see any flicker > here. Yes it does, I see no difference between builds without the patch and a build with. Could be just my crappy video card I guess.
Comment on attachment 667032 [details] [diff] [review] Patch [Triage Comment] Low risk fix for a FF17 regression. Approving for Aurora.
Attachment #667032 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
No luck transplanting this without conflicts: Justin@ORION /d/sources/mozilla-aurora $ hg transplant -s http://hg.mozilla.org/mozilla-central 1797401e19ce searching for changes applying 1797401e19ce patching file content/base/src/nsImageLoadingContent.cpp Hunk #1 FAILED at 1182 1 out of 1 hunks FAILED -- saving rejects to file content/base/src/nsImageLoadingContent.cpp.rej patch failed to apply abort: Fix up the merge and run hg transplant --continue
(In reply to Alex Keybl [:akeybl] from comment #13) > Comment on attachment 667032 [details] [diff] [review] > Patch > > [Triage Comment] > Low risk fix for a FF17 regression. Approving for Aurora. Kyle, did you forget about landing this on Aurora? I can merge this for you if you'd like.
Oh, merge day is tomorrow! That's why we're doing this. Anyway, the transplant here fails with conflicts on the one hunk in the patch. And the conflict is that the existing code matches the patched code.
I couldn't reproduce this issue, after trying the STR from comment 3. The "unwrap" and "wrap" by dragging the mouse works smooth for me. Nightly 2012-08-10 User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:17.0) Gecko/17.0 Firefox/17.0 Build ID: 20120810030512
QA Contact: manuela.muntean
Any suggestions?
(In reply to Manuela Muntean from comment #20) > Any suggestions? It's possible that Apple's website has changed since then. Also, I don't know why you're testing with that particular Nightly, since it looks like this bug was reproduced with nightlies from one month after.
(In reply to Justin Lebar [:jlebar] from comment #21) > (In reply to Manuela Muntean from comment #20) > > Any suggestions? > > It's possible that Apple's website has changed since then. Also, I don't > know why you're testing with that particular Nightly, since it looks like > this bug was reproduced with nightlies from one month after. I've tried the STR from comment 3 on another Nightly, but I still can't reproduce the issue. Everything works smoothly. User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:18.0) Gecko/18.0 Firefox/18.0 Build ID: 20120905030555
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: