Closed Bug 855050 Opened 12 years ago Closed 8 years ago

Intermittent layout/reftests/image/sync-image-switch-1d.html,sync-image-switch-1c.html,sync-image-switch-1b.html | image comparison (==), max difference: 255, number of differing pixels: 1024

Categories

(Core :: Layout: Images, Video, and HTML Frames, defect, P5)

x86
Windows XP
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: RyanVM, Unassigned)

References

Details

(Keywords: intermittent-failure, Whiteboard: [leave open][test disabled on WinXP])

Seth, I'm guessing this is related to bug 716140, but I don't know that for sure. Looks like an issue where the test image is blank and the reference shows the blue square. https://tbpl.mozilla.org/php/getParsedLog.php?id=21099694&tree=Mozilla-Inbound 03:31:05 INFO - REFTEST TEST-START | file:///c:/talos-slave/test/build/tests/reftest/tests/layout/reftests/image/sync-image-switch-1d.html | 5201 / 9232 (56%) 03:31:05 INFO - REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/tests/reftest/tests/layout/reftests/image/sync-image-switch-1d.html | image comparison (==), max difference: 255, number of differing pixels: 1024 03:31:05 INFO - REFTEST IMAGE 1 (TEST): <see log> 03:31:05 INFO - REFTEST IMAGE 2 (REFERENCE): <see log> 03:31:05 INFO - REFTEST INFO | Saved log: START file:///c:/talos-slave/test/build/tests/reftest/tests/layout/reftests/image/sync-image-switch-1d.html 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering WaitForTestEnd 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] WaitForTestEnd: Adding listeners 03:31:05 INFO - REFTEST INFO | Saved log: Initializing canvas snapshot 03:31:05 INFO - REFTEST INFO | Saved log: DoDrawWindow 0,0,800,1000 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_TO_FIRE_INVALIDATE_EVENT 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: dispatching MozReftestInvalidate 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_FOR_REFTEST_WAIT_REMOVAL 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_TO_FINISH 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: Completed 03:31:05 INFO - REFTEST INFO | Saved log: [CONTENT] RecordResult fired 03:31:05 INFO - REFTEST INFO | Saved log: RecordResult fired 03:31:05 INFO - REFTEST INFO | Saved log: RecordResult fired
Summary: Intermittent layout/reftests/image/sync-image-switch-1d.html | image comparison (==), max difference: 255, number of differing pixels: 1024 → Intermittent layout/reftests/image/sync-image-switch-1d.html,sync-image-switch-1b.html | image comparison (==), max difference: 255, number of differing pixels: 1024
Seth, any ideas (given comment 0)?
Flags: needinfo?(seth)
I concur that in all likelihood this is related to the image loading or decoding changes that were made for bug 716140. I'd have to debug it a bit (which I'll get to once I deal with the various crash / image flashing bugs) but I wouldn't be surprised to see it had to do with either broken onload blocking or this test implicitly relying on us decoding small images synchronously on the main thread. Note also that if it's the latter, bug 695763 might well be the right way to fix this if I can manage to get it landed.
Flags: needinfo?(seth)
Summary: Intermittent layout/reftests/image/sync-image-switch-1d.html,sync-image-switch-1b.html | image comparison (==), max difference: 255, number of differing pixels: 1024 → Intermittent layout/reftests/image/sync-image-switch-1d.html,sync-image-switch-1c.html,sync-image-switch-1b.html | image comparison (==), max difference: 255, number of differing pixels: 1024
Seth, any good news to report by chance?
Depends on: 695763
Flags: needinfo?(seth)
No good news I'm afraid. While this is on my radar, there are other things I absolutely have to fix before the next uplift that take priority. I probably won't revisit this until after then.
Flags: needinfo?(seth)
Whiteboard: [leave open][test disabled on WinXP]
Priority: -- → P5
WONTFIXing, since this is WinXP-specific and we're desupporting XP as noted in bug 1315083 comment 0. (Aryx is removing the reftest.list annotation for this failure in bug 1334898.)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.