nsIDocShellTreeItem and nsIDocShellTreeNode should be merged

RESOLVED FIXED in mozilla29

Status

()

RESOLVED FIXED
13 years ago
5 years ago

People

(Reporter: hwaara, Assigned: tbsaunde)

Tracking

Trunk
mozilla29
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
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
(Assignee)

Comment 3

5 years ago
Created attachment 8356276 [details] [diff] [review]
bug 331376 - merge nsIDocShellTreeNode into nsIDocShellTreeItem
Attachment #8356276 - Flags: review?(bzbarsky)
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)

Updated

5 years ago
Attachment #8356276 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/4f7f249f6d18
Assignee: nobody → trev.saunders
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.