Eliminate JS consumers of the nsDocShellTreeItem APIs
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Tracking
()
Fission Milestone | Future |
People
(Reporter: relaas, Unassigned)
References
(Depends on 2 open bugs, Blocks 1 open bug)
Details
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 2•5 years ago
|
||
Kannan says replacing nsIDocShellTreeItem calls should block enabling Fission in Nightly (M6).
Comment 3•5 years ago
|
||
Andreas, is this bug still needed? Kannan filed a 150+ individual bugs for various uses of nsIDocShellTreeItem that need to be replaced, though the bugs look like they only cover C++ code:
https://bugzilla.mozilla.org/showdependencytree.cgi?id=1467214&hide_resolved=1
How do we detect the JS callers of nsIDocShellTreeItem?
Comment 4•5 years ago
|
||
Kannan's list includes only the C++callers so we still need this for the JS callers. We'll need to find all users from the source tree.
Leaving farre's NI in case he has more to add here.
Comment 5•5 years ago
|
||
Comment 6•5 years ago
|
||
In JS land, we can access interfaces related to nsIDocShellTreeItem
without .QueryInterface()
, thus potentially these points which accesses these members defined in idl are JS callers.
Comment 7•5 years ago
|
||
kmag generated a list of callers of JS nsIDocShellTreeItem functions:
https://gist.github.com/kmaglione/37f02aec19a182b55f94f4e32d80c984
I'll file new bugs for these callers.
Comment 8•5 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #7)
kmag generated a list of callers of JS nsIDocShellTreeItem functions:
https://gist.github.com/kmaglione/37f02aec19a182b55f94f4e32d80c984
I'll file new bugs for these callers.
kmag fixed many of those callers in rootTreeItem bug 1638153. He recommended I not bother filing bugs for the remaining treeOwner callers because the existing users aren't actually problematic.
As such, I will defer this bug to Fission Future until we find problems.
Updated•2 years ago
|
Description
•