Closed
Bug 331376
Opened 18 years ago
Closed 10 years ago
nsIDocShellTreeItem and nsIDocShellTreeNode should be merged
Categories
(Core :: DOM: Navigation, defect)
Core
DOM: Navigation
Tracking
()
RESOLVED
FIXED
mozilla29
People
(Reporter: hwaara, Assigned: tbsaunde)
References
Details
Attachments
(1 file)
55.74 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
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.
Comment 1•18 years ago
|
||
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.
Updated•15 years ago
|
Assignee: adamlock → nobody
QA Contact: adamlock → docshell
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8356276 -
Flags: review?(bzbarsky)
Comment 4•10 years ago
|
||
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•10 years ago
|
Attachment #8356276 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4f7f249f6d18
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4f7f249f6d18
Assignee: nobody → trev.saunders
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in
before you can comment on or make changes to this bug.
Description
•