Closed
Bug 817473
Opened 12 years ago
Closed 12 years ago
Do we still need nsNodeSH::PostCreatePrototype?
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(2 files)
4.82 KB,
patch
|
peterv
:
review+
|
Details | Diff | Splinter Review |
7.53 KB,
patch
|
peterv
:
review+
|
Details | Diff | Splinter Review |
Seems like Node.prototype should have those properties as needed already with the WebIDL bindings, right? So I think we can nuke this function...
Assignee | ||
Comment 1•12 years ago
|
||
So first problem: nsIDOMXULDocument does not inherit from nsIDOMDocument. Which means that the proto chain for XUL documents doesn't actually go through Node, and they're relying on this method (which they get because they _do_ use nsDocumentSH) to make things work.
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #687637 -
Flags: review?(peterv)
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #687639 -
Flags: review?(peterv)
Assignee | ||
Updated•12 years ago
|
Whiteboard: [need review]
Assignee | ||
Comment 4•12 years ago
|
||
Try run: https://tbpl.mozilla.org/?tree=Try&rev=4b1123f00196
Comment 5•12 years ago
|
||
Comment on attachment 687637 [details] [diff] [review] part 1. Make nsIDOMXULDocument inherit from nsIDOMDocument, so that XUL documents end up with Document.prototype and Node.prototype on their proto chain. Review of attachment 687637 [details] [diff] [review]: ----------------------------------------------------------------- ::: content/xul/document/src/nsXULDocument.h @@ +132,5 @@ > NS_IMETHOD OnPrototypeLoadDone(bool aResumeWalk); > bool OnDocumentParserError(); > > + // nsINode interface overrides > + virtual nsresult Clone(nsINodeInfo *aNodeInfo, nsINode **aResult) const; We could keep MOZ_OVERRIDE here?
Attachment #687637 -
Flags: review?(peterv) → review+
Comment 6•12 years ago
|
||
Comment on attachment 687639 [details] [diff] [review] part 2. Remove no-longer-needed nsNodeSH::PostCreatePrototype. Review of attachment 687639 [details] [diff] [review]: ----------------------------------------------------------------- Cool, I thought we needed to hook up chrome-only properties but it'll happen automatically because Node is a non-concrete registered new binding interface.
Attachment #687639 -
Flags: review?(peterv) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Put back the MOZ_OVERRIDE. https://hg.mozilla.org/integration/mozilla-inbound/rev/0a26a48e9315 https://hg.mozilla.org/integration/mozilla-inbound/rev/21ea9ab04f3c
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla20
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0a26a48e9315 https://hg.mozilla.org/mozilla-central/rev/21ea9ab04f3c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•