Open
Bug 629885
Opened 15 years ago
Updated 3 years ago
Intermittent failure in layout/reftests/svg/as-image/svg-stylesheet-external-1.html | image comparison (==)
Categories
(Core :: SVG, defect, P3)
Core
SVG
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure, Whiteboard: [test marked random])
Attachments
(1 file)
|
14.55 KB,
text/plain
|
Details |
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1296304882.1296305518.5816.gz
Rev3 Fedora 12x64 mozilla-central opt test reftest on 2011/01/29 04:41:22
Reftest failure log attached.
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 9•15 years ago
|
||
Looked into this a bit, since this is my test that's failing.
I can reproduce this failure reliably by moving this reftest to a remote SSH server, which I mount in a local directory via SSHFS. (This is a hack to make files load slowly.)
I'm pretty sure this is what's causing the failure fail:
(1) we finish reading the bits of the SVG file
--> VectorImage tells host document "ready to paint!", in VectorImage::OnStopRequest
--> BUT we haven't necessarily gotten all the bits of the stylesheet yet!
(2) Document tells image to paint itself
(3) Image paints as blank, since the stylesheet still being loaded.
I think we'd really like to delay the VectorImage's "ready to paint!" notification (and hence the <img>/<body> load events) until the <svg> element fires 'SVGLoad' (which I think would happen after the stylesheet is ready (?)).
HOWEVER, this is pretty low-priority and unlikely to cause problems in the wild. Given bug 628747, this bug only affects local SVG images that reference local stylesheets, since HTTP-served SVG images aren't allowed to reference external stylesheets at all now.
I'm going to mark the non-HTTP version of this test as random, to stop the pain that its frequent orange causes, and I or someone else can fix the actual bug when time permits.
Comment 10•15 years ago
|
||
(In reply to comment #9)
> I'm going to mark the non-HTTP version of this test as random
Pushed: http://hg.mozilla.org/mozilla-central/rev/a8f321ab835c
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [orange] → [orange][test marked random]
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Comment hidden (Legacy TBPL/Treeherder Robot) |
| Assignee | ||
Updated•13 years ago
|
Keywords: intermittent-failure
| Assignee | ||
Updated•13 years ago
|
Whiteboard: [orange][test marked random] → [test marked random]
Comment 13•13 years ago
|
||
I suspect (though have not yet confirmed) that this may be fixed by bug 704059.
Depends on: 704059
Comment 14•9 years ago
|
||
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•