nsIDocShellTreeItem and nsIDocShellTreeNode should be merged

RESOLVED FIXED in mozilla29

Status

()

defect
RESOLVED FIXED
14 years ago
6 years ago

People

(Reporter: hwaara, Assigned: tbsaunde)

Tracking

Trunk
mozilla29
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Right now, when traversing a docshell tree, you have to use the nodes, and every node in turn correspond to an item.  

Boris said we should just put merge these two, and let all items have the node methods.
Does anyone see any reasons to not do this?
OS: MacOS X → All
Hardware: Macintosh → All
I think this has been talked about for years. I'm all for it.
Assignee: adamlock → nobody
QA Contact: adamlock → docshell
Comment on attachment 8356276 [details] [diff] [review]
bug 331376 - merge nsIDocShellTreeNode into nsIDocShellTreeItem

Olli, please throw this back my way if you don't have time right now.
Attachment #8356276 - Flags: review?(bzbarsky) → review?(bugs)
Attachment #8356276 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/4f7f249f6d18
Assignee: nobody → trev.saunders
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.