Closed
Bug 414706
Opened 16 years ago
Closed 16 years ago
document is still a XOW
Categories
(Core :: DOM: Core & HTML, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: mrbkap, Assigned: mrbkap)
Details
(Keywords: perf)
Attachments
(1 file)
|
1.07 KB,
patch
|
jst
:
review+
jst
:
superreview+
jst
:
approval1.9+
|
Details | Diff | Splinter Review |
In bug 397791, Johnny did a bunch of work so that we don't have to wrap documents. He missed a spot (I should have pointed it out then): because of the caching we do for the document wrapped native, there's some special case code to wrap the document there. I'll attach a patch in a jiffy.
|
Assignee | |
Comment 1•16 years ago
|
||
Attachment #300177 -
Flags: superreview?(jst)
Attachment #300177 -
Flags: review?(jst)
Attachment #300177 -
Flags: approval1.9?
|
|
Assignee | |
Comment 2•16 years ago
|
||
This should really be a P1 blocker to match bug 397791.
Flags: blocking1.9?
Priority: -- → P1
|
||
Updated•16 years ago
|
Flags: blocking1.9? → blocking1.9+
Target Milestone: --- → mozilla1.9beta3
|
|
||
Comment 3•16 years ago
|
||
Comment on attachment 300177 [details] [diff] [review] Trivial patch Duh, didn't realize this was hiding there...
Attachment #300177 -
Flags: superreview?(jst)
Attachment #300177 -
Flags: superreview+
Attachment #300177 -
Flags: review?(jst)
Attachment #300177 -
Flags: review+
Attachment #300177 -
Flags: approval1.9?
Attachment #300177 -
Flags: approval1.9+
|
|
Assignee | |
Comment 4•16 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 16 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
•