1218954 - XUL <treecell> doesn't load image synchronously (making it impossible to use reliably in reftests)

Status: NEW

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

Description

[Daniel Holbert [:dholbert]]

As noted in bug 1218041 comment 11, a XUL <treecell> apparently doesn't decode its image synchronously, which makes it hard to use them in reftests.

Seth says in IRC he thought bug 1209765 should have fixed this, but apparently it did not.

Reftest patch coming up, to demonstrate the problem.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: reftest patch to demonstrate the bug

STR:
 1. Apply the attached patch.
 2. Run the following command:
    ./mach reftest layout/reftests/xul/reftest-bug1218954.list

ACTUAL RESULTS: Test failure. (The tests look like their "notref" file, indicating that the image didn't render in time.)

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: reftest patch #2, to add setTimeouts & work around the problem

Here's a second patch (which layers on top of the first) to add a 1-second setTimeout to each test. This "fixes" the tests, making them pass. (The image renders, and they look different from the "notref", and hence pass.)

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.