Closed Bug 1103439 Opened 6 years ago Closed 6 years ago
Lib's chrome tests need to clear the chrome image cache, not the regular image cache
ImageLib has a number of chrome tests that need to clear the cache to work properly. Unfortunately, the function they're using to do it, |clearImageCache|, does not clear the correct cache - the images used by chrome tests are chrome images, but |clearImageCache| clears only the regular image cache. We need to add another function that clears both caches, and call it in all ImageLib mochitest-chrome tests. I believe this may be the real cause of bug 1100497, so I'm going to try reenabling it after this lands.
(In reply to Seth Fowler [:seth] from comment #0) > I believe this may be the real cause of bug 1100497, so I'm going to try > reenabling it after this lands. Reenabling 'test_animation.html', to be clear.
Here's the patch.
Try job here: https://tbpl.mozilla.org/?tree=Try&rev=9904a4cbbe31
Attachment #8527249 - Flags: review?(tnikkel)
Attachment #8527249 - Flags: review?(tnikkel) → review+
Thanks for the review! Pushed: remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/faa77ca026ea
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.