Closed Bug 480389 Opened 11 years ago Closed 11 years ago

Remove nsIDocument's cached JS object

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: peterv, Assigned: peterv)

Details

Attachments

(1 file)

Attached patch v1Splinter Review
Since we cache wrappers on nsINode we don't need to do it on nsIDocument. I've left the special casing of document as parent in nsNodeSH::PreCreate to quickly get the cached wrapper, we could just not special-case and fall through to WrapNative which does the same thing but in the XPConnect wrapping code.
Attachment #364376 - Flags: superreview?(jst)
Attachment #364376 - Flags: review?(jst)
Assignee: nobody → peterv
Attachment #364376 - Flags: superreview?(jst)
Attachment #364376 - Flags: superreview+
Attachment #364376 - Flags: review?(jst)
Attachment #364376 - Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.