Intermittent test_bug733553.html | | Test 0 red.png width not correct - got 40, expected 1 followed by failures in Test 1, 2, 3, 4, 5 and a | Test timed out.

RESOLVED FIXED in Firefox 18

Status

()

Core
ImageLib
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: philor, Assigned: hobophobe)

Tracking

({intermittent-failure})

Trunk
mozilla18
x86
Linux
intermittent-failure
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox17 affected, firefox18 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=14693991&tree=Mozilla-Inbound
Rev3 Fedora 12 mozilla-inbound debug test mochitests-4/5 on 2012-08-24 22:47:42 PDT for push 772c3eed0cfb
slave: talos-r3-fed-014

12820 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 0 red.png width not correct - got 40, expected 1
12821 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 1 animated-gif2.gif width not correct - got 1, expected 40
12822 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 2 red.png width not correct - got 100, expected 1
12823 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 3 lime100x100.svg width not correct - got 40, expected 100
12824 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 4 animated-gif2.gif width not correct - got 1, expected 40
12825 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test 5 red.png width not correct - got 100, expected 1
12826 ERROR TEST-UNEXPECTED-FAIL | /tests/image/test/mochitest/test_bug733553.html | Test timed out.
(Assignee)

Comment 1

6 years ago
Created attachment 655813 [details] [diff] [review]
Make multipart test use a signal

I forgot to get the change from bug 756927 landed, but it wouldn't have helped here, and this kills both.  This makes the test signal-based instead of timing-based.

This adds a separate dummy server |bug733553-informant.sjs| to the test that's used to signal the main server to proceed with the next part.

So the test gets a |load| event for the image, checks it, then loads the dummy server which just sets the shared state with the next test part.  

The main server now sends off a part, and holds until it sees the shared state update.  When it has, it sends the next part.

Try push is at:
https://tbpl.mozilla.org/?tree=Try&rev=273a7f91c3d6
Assignee: nobody → unusualtears
Attachment #655813 - Flags: review?(joe)
(Assignee)

Updated

6 years ago
Blocks: 756927
Comment on attachment 655813 [details] [diff] [review]
Make multipart test use a signal

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

Well this is just wonderful.

If you're not positive that it fixes the intermittent orange, you can trigger a pile of repeats of the test by pressing the "+" link/image on your Try tbpl page.
Attachment #655813 - Flags: review?(joe) → review+
(Assignee)

Comment 3

6 years ago
Created attachment 656295 [details] [diff] [review]
Update for checkin

Thanks Joe!
Attachment #655813 - Attachment is obsolete: true
(Assignee)

Comment 4

6 years ago
Green try at: https://tbpl.mozilla.org/?tree=Try&rev=273a7f91c3d6
Keywords: checkin-needed
(In reply to Adam [:hobophobe] from comment #4)
> Green try at: https://tbpl.mozilla.org/?tree=Try&rev=273a7f91c3d6

https://hg.mozilla.org/integration/mozilla-inbound/rev/5d772c765d4e
Flags: in-testsuite+
Keywords: checkin-needed

Comment 6

6 years ago
https://hg.mozilla.org/mozilla-central/rev/5d772c765d4e
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Needed this to push bug 787899's tests cleanly to aurora, so it got to ride along.

remote:   https://hg.mozilla.org/releases/mozilla-aurora/rev/d234a1e40780
status-firefox17: --- → fixed
status-firefox18: --- → fixed

Comment 8

6 years ago
Backed out from Aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/7a03659b943e
status-firefox17: fixed → affected
Keywords: intermittent-failure
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.