Closed Bug 1263474 Opened 9 years ago Closed 9 years ago

convert three tests in image/test/mochitest/chrome.ini to mochitest-plain

Categories

(Core :: Graphics: ImageLib, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox47 --- fixed
firefox48 --- fixed

People

(Reporter: tnikkel, Assigned: tnikkel)

References

Details

Attachments

(1 file)

These were disabled at the time of bug 1261158. But bug 1262269 shows they don't need to be.
Attached patch patch (deleted) — Splinter Review
Attachment #8739801 - Flags: review?(bugmail.mozilla)
Comment on attachment 8739801 [details] [diff] [review] patch Review of attachment 8739801 [details] [diff] [review]: ----------------------------------------------------------------- ::: image/test/mochitest/mochitest.ini @@ +107,5 @@ > [test_bug89419-2.html] > [test_bug1180105.html] > [test_bug1217571.html] > +[test_animation.html] > +[test_animation2.html] They might still need to be disabled on b2g/android, since the chrome.ini had that. AFAICT you haven't done a try push to verify they pass on android.
Attachment #8739801 - Flags: review?(bugmail.mozilla) → review+
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #2) > They might still need to be disabled on b2g/android, since the chrome.ini > had that. AFAICT you haven't done a try push to verify they pass on android. Ah, I thought I had, but of course they are disabled by default at the start of the file, and when I checked the try server log it was "TEST-SKIP" which I must have read as "TEST-PASS" when I originally checked. I'll disable on b2g and android.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Uplifted to aurora because I'm told e10s test fixes should be uplifted to aurora https://hg.mozilla.org/releases/mozilla-aurora/rev/3312754269d285ad416da1d54cf07e6a163ce479
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: