Open Bug 1218954 Opened 4 years ago Updated Last year

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

Categories

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

defect
Not set

Tracking

()

Tracking Status
firefox44 --- affected

People

(Reporter: dholbert, Assigned: seth)

References

Details

Attachments

(2 files)

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.
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.)
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.)
Product: Core → Core Graveyard
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.