Move DocShell.watchedByDevtools from DocShell up to BrowsingContext
Categories
(Core :: DOM: Navigation, enhancement, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox78 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-m2-mvp)
Attachments
(1 file, 1 obsolete file)
|
Bug 1620966 - Move DocShell.watchedByDevtools to BrowsingContext and rename it to watchedByDevTools.
(deleted),
text/x-phabricator-request
|
Details |
This attribute is currently on the DocShell:
https://searchfox.org/mozilla-central/source/docshell/base/nsIDocShell.idl#1113
But this forces us to manually apply this attribute in each individual remote iframe:
https://searchfox.org/mozilla-central/source/devtools/server/actors/targets/browsing-context.js#1564
https://searchfox.org/mozilla-central/source/devtools/server/actors/targets/browsing-context.js#781
It would be easier to move this flag up to the BrowsingContext class, so that it is replicated to the whole BrowsingContext tree, and even for remoted iframes.
In addition, it would help bug 1593937 reuse this BrowsingContext attribute in order to start loading DevTools code in the content processes.
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Comment 2•5 years ago
|
||
Tracking Fission DevTools bugs for Fission Nightly (M6) milestone
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
||
Updated•5 years ago
|
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
Description
•