Add convenience methods to nsIDocShell




17 years ago
7 years ago


(Reporter: aaronlev, Unassigned)


(Depends on: 1 bug, {helpwanted})


Firefox Tracking Flags

(Not tracked)




17 years ago
Right now we have 5+ degrees of separation between some interfaces that there is
always a 1-1 relationship between. All the getters and QI's in Mozilla code make
it big and hard to read.

For example, to get from a dom window to a tree item, you have to go:
domwin-> domdoc-> doc-> presshell-> prescontext-> container-> docshell->treeitem

Or, to get from a docshell to a window, you have to:
docshell -> presshell -> doc -> script global object -> window

None of this stuff is documented anywhere either, and new coders are forced to
use lxr to try to get a handle on the mess. Is this a problem or not?

Why don't we add at least some of these methods:
and then a way to get an nsIDocShell from all these other objects.
In this way we could make docshell the information center and reduce everything
to 2 degrees of separation.

Something akin to, but not exactly like:
presshell  prescontext   doc  scriptglobalobject  windowinternal    esm
        \       \         |             /             /            /
                     d o c s h e l l !  
[to]    /       /         |       |            \         \       \     \     
presshell  prescontext   doc  scriptglobal  windowinternal  esm  domdoc  domwin

I realize there's not exactly a 1-1 correspondence between presshells and docs
(hence doc->GetShellAt(index, ...), but we can map this out and get it to work.

Also, there's a lot of interfaces I don't know. What else should be added to the
list? nsIDocumentViewer? nsIContentViewer? ...

Comment 1

17 years ago
Hmm, I would love to be able to hang on to one doc-related interface in a class
or method, and get to just about anything I need from it.

Comment 2

17 years ago
Would people want all these doc-related interfaces on nsIDocShell, or
At the *very least* I suggest making nsIDocShellTreeNode extend
nsIDocShellTreeItem.  Better still, make nsIDocShell extend nsIDocShellTreeNode.
 I am certain the other doc-level interfaces could be organized better, but this
would be a modest step that could reduce QIs and improve readability.
Further, "parent" items obviously have children, so those should be of type
nsIDocShellNode or even (possibly) nsIDocShell.

Comment 5

17 years ago
I forgot nsIViewManager.

Comment 6

17 years ago
jkeiser, how should we split this up.

One person could do the merging of classes and interfaces (nswebshell,
nsdocshell, nsidocshelltreenode, nsidocshelltreeitem, as you wish to do it).

Another person could add all of the new convenience getters.
The docshell owners should be aware of this, and they should also approve this
change before all the work is started.
Agreed.  Not only that but we need to determine precisely which getters to put
on which class.

DocShell methods to get document, window and prescontext are reasonable. 
ScriptGlobal can be got from window PresContext methods to get 

Work I can see right now:
- review bryner's nsIPresContext patch (I'm working on this)
- deal with merging the silly TreeItem and TreeNode interfaces
- design the getters, document class relationships, add some getters
Depends on: 154199

Comment 9

17 years ago
Doesn't GetInterface on nsIDocShell work for many of these?
OS: Windows 2000 → All
Hardware: PC → All

Comment 10

17 years ago
Actually yes, but you have to QI to nsIInterfaceRequestor first.
That takes care of going from a docshell interface to lots of other things.
Perhaps we should just support GetInterface directly on nsIDocShell or
nsIDocShellTreeItem, and add a few more things that we can get.
Then we can make it really easy to get a docshell from just about anywhere, and
then we'd have our 2 degrees of separation. 
GetInterface is rather horrid.

GetCurrentDocument(nsIDocument*) is much more descriptive--it tells you exactly
what you are getting and it gets you the result in the right form without QI'ing
either nsIDocShell or the resulting object from GetInterface (2 QI's).  It is
also more discoverable, which GetInterface() is emphatically *not*.

Perhaps there are situations in which GetInterface is good, but having real
getters as well is a must.


15 years ago
Keywords: helpwanted
Mass un-assigning bugs assigned to Aaron.
Assignee: aaronleventhal → nobody
QA Contact: adamlock → docshell
nsIDocument pretty much is this information center now, but without the restrictions of XPIDL.
You need to log in before you can comment on or make changes to this bug.