Deeply nested elements are not rendered (TAGLVL has been exceeded)
Categories
(Core :: Layout, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: brian, Assigned: hsivonen)
References
(Blocks 2 open bugs)
Details
(Keywords: testcase, Whiteboard: [webcompat])
Attachments
(6 files, 6 obsolete files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
hsivonen
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
hsivonen
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
Reporter | ||
Comment 1•20 years ago
|
||
Comment 2•20 years ago
|
||
Comment 3•20 years ago
|
||
Reporter | ||
Comment 4•20 years ago
|
||
Comment 5•20 years ago
|
||
Comment 6•20 years ago
|
||
Comment 7•20 years ago
|
||
Updated•20 years ago
|
Comment 8•20 years ago
|
||
Comment 9•20 years ago
|
||
Comment 10•20 years ago
|
||
Comment 11•20 years ago
|
||
Comment 12•20 years ago
|
||
Comment 13•20 years ago
|
||
Comment 14•20 years ago
|
||
Comment 15•20 years ago
|
||
Comment 16•20 years ago
|
||
Updated•20 years ago
|
Comment 17•20 years ago
|
||
Comment 18•20 years ago
|
||
Comment 19•19 years ago
|
||
Comment 20•19 years ago
|
||
Comment 21•19 years ago
|
||
Comment 22•19 years ago
|
||
Comment 23•19 years ago
|
||
Comment 24•19 years ago
|
||
Comment 25•19 years ago
|
||
Comment 26•18 years ago
|
||
Comment 30•17 years ago
|
||
Reporter | ||
Updated•17 years ago
|
Comment 32•16 years ago
|
||
Updated•15 years ago
|
Comment 36•12 years ago
|
||
Comment 39•10 years ago
|
||
Comment 40•9 years ago
|
||
Updated•9 years ago
|
Comment 43•9 years ago
|
||
Comment 45•8 years ago
|
||
Comment 46•8 years ago
|
||
Comment 47•8 years ago
|
||
Assignee | ||
Comment 48•7 years ago
|
||
Comment hidden (mozreview-request) |
Assignee | ||
Comment 51•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 69•7 years ago
|
||
Assignee | ||
Comment 70•7 years ago
|
||
Assignee | ||
Comment 71•7 years ago
|
||
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Comment 85•7 years ago
|
||
mozreview-review |
Assignee | ||
Comment 86•7 years ago
|
||
mozreview-review-reply |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 88•7 years ago
|
||
mozreview-review-reply |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 90•7 years ago
|
||
Comment 92•7 years ago
|
||
mozreview-review |
Comment 93•7 years ago
|
||
mozreview-review |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 96•7 years ago
|
||
mozreview-review-reply |
Assignee | ||
Comment 97•7 years ago
|
||
mozreview-review-reply |
Assignee | ||
Comment 98•7 years ago
|
||
mozreview-review-reply |
Comment 99•7 years ago
|
||
mozreview-review |
Comment 100•7 years ago
|
||
Comment 101•7 years ago
|
||
Comment 102•7 years ago
|
||
Assignee | ||
Comment 103•7 years ago
|
||
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 119•7 years ago
|
||
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Updated•7 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 125•6 years ago
|
||
Assignee | ||
Comment 126•6 years ago
|
||
Assignee | ||
Comment 127•6 years ago
|
||
Assignee | ||
Comment 128•6 years ago
|
||
Assignee | ||
Comment 129•6 years ago
|
||
Assignee | ||
Comment 130•6 years ago
|
||
Assignee | ||
Comment 131•6 years ago
|
||
Assignee | ||
Comment 132•6 years ago
|
||
Assignee | ||
Comment 133•6 years ago
|
||
Assignee | ||
Comment 134•6 years ago
|
||
Assignee | ||
Comment 135•6 years ago
|
||
Assignee | ||
Comment 136•6 years ago
|
||
Assignee | ||
Comment 137•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 138•6 years ago
|
||
Assignee | ||
Comment 139•6 years ago
|
||
Comment 141•6 years ago
|
||
Comment 142•6 years ago
|
||
Comment 143•6 years ago
|
||
Assignee | ||
Comment 144•6 years ago
|
||
Updated•6 years ago
|
Assignee | ||
Comment 146•6 years ago
|
||
Assignee | ||
Comment 147•6 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #145)
Please file a followup.
Thanks for the r+. The follow-up is already on file: bug 1257522.
Comment 148•6 years ago
|
||
Comment 149•6 years ago
|
||
Backed out for failing win xpcshell at xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_temporary.js
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=220631321&repo=mozilla-inbound&lineNumber=14160
Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/e956eb6131305a6e9b4e1c581f2b35fe64baf61d
Assignee | ||
Comment 150•6 years ago
|
||
(In reply to Andreea Pavel [:apavel] from comment #149)
Backed out for failing win xpcshell at xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_temporary.js
Treeherder on try showed the title of bug 1469904, which says "perma-failing", so I thought it was ignorable orange. :-(
Assignee | ||
Comment 151•6 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #150)
(In reply to Andreea Pavel [:apavel] from comment #149)
Backed out for failing win xpcshell at xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_temporary.js
Treeherder on try showed the title of bug 1469904, which says "perma-failing", so I thought it was ignorable orange. :-(
On surface, test_temporary.js looks completely unrelated to these patches.
Assignee | ||
Comment 152•6 years ago
|
||
The test only ever fails on Windows. aswan, blame shows you've added Windows special-casing to the test. Do you have any idea what the test might be doing that could be affected by a larger-than-previous run-time stack size? Could it be using accidental recursion and relying on the SpiderMonkey stack size-based recursion limit taking effect before the test times out?
Assignee | ||
Comment 153•6 years ago
|
||
Try run with only the Windows run-time stack size change:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2bb0cf89881331ed4bc58b331f31fbc86a8aabb4
Assignee | ||
Comment 154•6 years ago
|
||
Just trying things: Making task creation conditional on Windowsness of the platform instead of making the task return early based on the Windowsness of the platform:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8fd97684651e546ffb0e5e11086c38039c86030e
Assignee | ||
Comment 155•6 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #154)
Just trying things: Making task creation conditional on Windowsness of the platform instead of making the task return early based on the Windowsness of the platform:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8fd97684651e546ffb0e5e11086c38039c86030e
This did not help. :-(
Comment 156•6 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #152)
The test only ever fails on Windows. aswan, blame shows you've added Windows special-casing to the test. Do you have any idea what the test might be doing that could be affected by a larger-than-previous run-time stack size? Could it be using accidental recursion and relying on the SpiderMonkey stack size-based recursion limit taking effect before the test times out?
Nothing like that is happening on purpose. My past run-ins with this test have had to do with Windows handling file locking different than mac/unix, that seems totally unrelated to the changes in this bug. Sampling a few other runs on treeherder, it looks like this test is typically taking ~4 minutes on Windows debug which is uncomfortably close to the 5 minute cap. Could your patches have slowed down debug builds a bit more and pushed this test over 5 minutes?
In any case, since we're flirting with 5 minute run times, this test really needs to be broken up into multiple smaller tests. I'd have no objection to you disabling it on Windows debug for now if its blocking something else.
Assignee | ||
Comment 157•6 years ago
|
||
(In reply to Andrew Swan [:aswan] from comment #156)
I'd have no objection to you disabling it on Windows debug for now if its blocking something else.
Thanks. It fails on 64-bit Windows opt builds, too, so here's a try run disabling it on Windows generally:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=13a3e0fca5c6a434c78b94ac7c4d511e77a7cb9f
Comment 158•6 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #150)
(In reply to Andreea Pavel [:apavel] from comment #149)
Backed out for failing win xpcshell at xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_temporary.js
Treeherder on try showed the title of bug 1469904, which says "perma-failing", so I thought it was ignorable orange. :-(
Hi Henri. Yes, that bug is perma-failing, but it's tier2. the failures from the push you landed were tier1, reason why I backed out.
Comment 159•6 years ago
|
||
Comment 160•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/95d8f77451e2
https://hg.mozilla.org/mozilla-central/rev/0fab42583213
https://hg.mozilla.org/mozilla-central/rev/feb776af8fd3
https://hg.mozilla.org/mozilla-central/rev/519e21226224
https://hg.mozilla.org/mozilla-central/rev/e4550e8b5a79
Assignee | ||
Comment 161•6 years ago
|
||
Forgot to add this link here earlier:
https://hg.mozilla.org/projects/htmlparser/rev/209cca99a3a2
Updated•6 years ago
|
Comment 162•6 years ago
|
||
This looks to be crashing on win10/aarch64. See bug 1530033.
Updated•2 years ago
|
Description
•