Closed
Bug 1281969
Opened 8 years ago
Closed 5 years ago
Intermittent TEST-UNEXPECTED-CRASH | /html/the-xhtml-syntax/parsing-xhtml-documents/xhtml-mathml-dtd-entity-support.htm | expected TIMEOUT
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
INVALID
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure)
Filed by: philringnalda@gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=30574021&repo=mozilla-inbound http://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-win32-pgo/1466703046/mozilla-inbound_win7_vm_test_pgo-web-platform-tests-3-bm129-tests1-windows-build79.txt.gz
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 25•8 years ago
|
||
Looks like an OOM crash. Not sure if there is anything sensible to do other than disable the test?
Flags: needinfo?(bzbarsky)
Comment hidden (Intermittent Failures Robot) |
Comment 27•8 years ago
|
||
So... this test runs through all the entities in testing/web-platform/tests/common/entities.json and all the doctype/mimetype entries in the test itself. For each combination it does a DOMParser parseFromString call to create a new Document and then examines it. That's, as of today, 2231 entities and 10 doctype/mimetype entries, for a total of 22310 calls to parseFromString, in a loop, without ever coming back to the event loop. I bet with snowwhite we do async deletion of all that stuff, so never get around to deleting it during the test. Is that actually the case? If so, can we fix that? (Note that the test is annotated as timing out in general and crashing on debug win32; turns out we can't parse 23000 documents in the normal test timeout either. We should consider asking for a longer timeout for this test or splitting it up or something).
Flags: needinfo?(bzbarsky) → needinfo?(continuation)
Updated•8 years ago
|
Keywords: leave-open
Comment 28•8 years ago
|
||
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/f478cabf6ecf Disable test that is intermittent or perma-timeout as it tries to parse 22000 documents, a=testonly
Comment 29•8 years ago
|
||
This test also times out locally for me on my OSX developer machine. I'm not sure what is going on in this test, but it is creating documents at very rapid rate. We are running the CC and destroying some of them (though maybe that's a worker CC?), but they are being created much faster. We must be returning to the event loop sometimes. We do call |nsNodeUtils::LastRelease(this);| when the refcount of the document drops to zero, so that should help keep things from building up too much, but we aren't calling it very much. It does seem like insufficient GC/CC is the problem. If I make every 20 calls to DOMParser::ParseFromString() do a GC+CC, then the documents are rapidly destroyed as you'd expect.
Flags: needinfo?(continuation)
Comment 30•8 years ago
|
||
Maybe rewriting the test so it does setTimeout 0 regularly would help? The CC is not set up to deal with the creation and destruction of objects this rapidly, so that may not be enough. In a Mochitest, I'd just recommend you force a GC/CC periodically, but that won't work for web platform tests.
Comment 31•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f478cabf6ecf
Comment hidden (Intermittent Failures Robot) |
Comment 33•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/a00bd4772feb
status-firefox50:
--- → disabled
status-firefox51:
--- → disabled
status-firefox52:
--- → disabled
See Also: → 1281946
Updated•6 years ago
|
Component: web-platform-tests → DOM
Product: Testing → Core
Version: Version 3 → Trunk
Updated•6 years ago
|
Priority: -- → P3
Comment 34•5 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months. :hsinyi, maybe it's time to close this bug?
Flags: needinfo?(htsai)
Comment 35•5 years ago
|
||
The original test xhtml-mathml-dtd-entity-support.htm has been split into smaller pieces in bug 1357844. So I'm going to close this as invalid.
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(htsai)
Resolution: --- → INVALID
Updated•5 years ago
|
Keywords: leave-open
Assignee | ||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•