The default bug view has changed. See this FAQ.

No Animation Of WunderMap Interactive Radar on Nightly

VERIFIED FIXED in mozilla11

Status

()

Core
ImageLib
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: WildcatRay, Assigned: mattwoodrow)

Tracking

({regression})

Trunk
mozilla11
x86_64
Windows 7
regression
Points:
---

Firefox Tracking Flags

(firefox11-)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
On WunderMap Interactive Radar of wunderground.com, the map does not animate (goes into motion). It does work in Aurora builds and Firefox. (Regression confirmed in new profile on Nightly.)

To start animate, click on the blue button just to the right of "Animate" to the right of the map.
(Reporter)

Updated

5 years ago
Summary: No Animate Of WunderMap Interactive Radar → No Animation Of WunderMap Interactive Radar on Nightly

Comment 1

5 years ago
Regression window(m-i)
works:
http://hg.mozilla.org/integration/mozilla-inbound/rev/a126c53a037e
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111108 Firefox/11.0a1 ID:20111108160609
Fails(skip middle frames):
http://hg.mozilla.org/integration/mozilla-inbound/rev/20c6e85f8a59
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111108 Firefox/11.0a1 ID:20111108191531
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=a126c53a037e&tochange=20c6e85f8a59
Triggered by:
20c6e85f8a59	Matt Woodrow — Bug 695610 - Move ImageContainer caching into imgIContainer. r=joe
Blocks: 695610
Component: General → ImageLib
Keywords: regressionwindow-wanted
Product: Firefox → Core
QA Contact: general → imagelib
(Assignee)

Comment 2

5 years ago
Created attachment 577188 [details] [diff] [review]
Remove the cached ImageContainer when the current frame changes.

Since mImageContainer is a cache of FRAME_CURRENT, we need to reset it whenever this changes.

It would be nice to move the cache onto the imgFrame itself instead of the container, but this would be a memory usage regression currently.
Attachment #577188 - Flags: review?
(Assignee)

Updated

5 years ago
Attachment #577188 - Flags: review? → review?(joe)
(Assignee)

Comment 3

5 years ago
Created attachment 577189 [details] [diff] [review]
Remove the cached ImageContainer when the current frame changes. v2

Removed unrelated change included by accident.
Attachment #577188 - Attachment is obsolete: true
Attachment #577188 - Flags: review?(joe)
Attachment #577189 - Flags: review?(joe)
http://www.meteox.com/h.aspx?r=&jaar=-3&soort=satradar doesn't animate either, same problem ?
It was noted in the mozillazine forums that changing the opacity slider, does not matter if you change it up or down, the animation starts up on the wunderground radar test url. 

Didn't know if that fact changed anything in the thinking for the fix as adapted and waiting review.
tracking-firefox11: --- → ?
Comment on attachment 577189 [details] [diff] [review]
Remove the cached ImageContainer when the current frame changes. v2

Review of attachment 577189 [details] [diff] [review]:
-----------------------------------------------------------------

Argh. Can we add tests for this?
Attachment #577189 - Flags: review?(joe) → review+
(Assignee)

Comment 7

5 years ago
Created attachment 577471 [details] [diff] [review]
Add animated image test that uses an ImageLayer

Exactly the same as test_animation.html, except that it uses an inconsequential opacity to force the image to have an ImageLayer created.
Attachment #577471 - Flags: review?(joe)
Comment on attachment 577471 [details] [diff] [review]
Add animated image test that uses an ImageLayer

Review of attachment 577471 [details] [diff] [review]:
-----------------------------------------------------------------

As long as this passes try (I presume 0.9999 * 0x2aff00 rounds to 2aff00 on all our platforms) all's well.
Attachment #577471 - Flags: review?(joe) → review+
(Assignee)

Comment 9

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a44df2a172b
https://hg.mozilla.org/integration/mozilla-inbound/rev/180b47429ff3
Assignee: nobody → matt.woodrow
https://hg.mozilla.org/mozilla-central/rev/1a44df2a172b
https://hg.mozilla.org/mozilla-central/rev/180b47429ff3
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Mozilla/5.0 (Windows NT 5.1; rv:11.0a1) Gecko/20111206 Firefox/11.0a1 ID:20111206031117
VERIFIED FIXED for both urls (wunderground/meteox)
Status: RESOLVED → VERIFIED

Updated

5 years ago
tracking-firefox11: ? → -
You need to log in before you can comment on or make changes to this bug.