HTML <link> elements in system privilege documents cause history service to load
Categories
(Core :: DOM: Core & HTML, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: bdahl, Assigned: emilio)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details |
Over in bug 1492582, where I'm migrating <window> to <html> the test "browser/base/content/test/performance/browser_startup.js" started failing because of TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_startup.js | resource://gre/modules/PlacesUtils.jsm is not allowed before first paint
. PlacesUtils is loaded because of the chain:
- changing the "dir" of the root element
- descending into children elements and calling Element::UpdateState
- IntrinsicState on <link> then calls Link::LinkState
- LinkState() causes the places DB to be loaded early
As discussed in bug 1538968, for content priv documents we want to track the visited state of <link> elements. However, for system priv documents we can ignore it.
Reporter | ||
Comment 1•5 years ago
|
||
An HTML link element in a document with a root HTML node causes the
HistoryService to be loaded while loading the document. Once browser.xhtml
starts using a root <html> node the test browser_startup.js will fail
because of this.
Comment 2•5 years ago
|
||
We should consider making this change unconditionally, not just for system principal.
Assignee | ||
Comment 3•5 years ago
|
||
I can send a patch later today to rip all the code that handles visitedness from links, since if doing that we should also change stuff to not go through the link update stuff (otherwise we end up here anyway).
Assignee | ||
Comment 4•5 years ago
|
||
Well, earlier tomorrow I assume.
Assignee | ||
Comment 6•5 years ago
|
||
Intent email: https://groups.google.com/d/msg/mozilla.dev.platform/1NP6oJzK6zg/ftAz_TajAAAJ
For now do the obvious check rather than bigger refactorings, since we keep them
matching :link or not depending on whether they have an href.
I'll file an HTML spec issue about not making them traversable, and a MathML
issue about the craziness that it is that almost all MathML elements can be
links.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 8•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Comment 9•5 years ago
|
||
Documentation completed; please take a look at our relevant sprint task for details: https://github.com/mdn/sprints/issues/2102#issuecomment-530762651
Please let me know if the wording looks OK. Thanks!
Description
•