Closed
Bug 386620
Opened 17 years ago
Closed 17 years ago
reftest takes snap too early, perhaps before page fully loaded
Categories
(Testing :: Reftest, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 374458
People
(Reporter: ray, Unassigned)
References
()
Details
Load the page from the URL field. One sees several stages of loading. At first the block has red bits in it, but as the page loads, they disappear and when the page is fully loaded the square is solidly green.
If one runs a reftest on this page, the snap has red in the box.
It seems that the reftest snap is being made when one layer of the page is loaded, as it does catch the box, but there seem to be other types of loading that still need to happen. Does reftest need to be listening for another event before it takes the snap?
Comment 1•17 years ago
|
||
I would guess that this is due to bug 253851
Comment 2•17 years ago
|
||
Now that you mention it, I think I recall seeing one of the current reftests that uses background-images failing intermittently.
We could set the hidden pref. that makes onload fire after background-images are loaded, but I'm not sure if this would be appropriate, plus, the reftest doesn't have a way to set preferences currently.
Comment 3•17 years ago
|
||
Running reftest with the "layout.fire_onload_after_image_background_loads" preference set might be the way to go here, no?
Comment 4•17 years ago
|
||
(In reply to comment #3)
> Running reftest with the "layout.fire_onload_after_image_background_loads"
> preference set might be the way to go here, no?
Looks like we'll just be switching that pref by default (see bug 253851). We should make sure to set it for reftest if we end up reverting that change, though.
Comment 5•17 years ago
|
||
Sounds like bug 371866 was marked fixed a little too eagerly, although maybe this should just be duped against bug 374458.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
Updated•16 years ago
|
Component: Testing → Reftest
Product: Core → Testing
Version: Trunk → unspecified
Updated•16 years ago
|
QA Contact: testing → reftest
You need to log in
before you can comment on or make changes to this bug.
Description
•