Closed
Bug 282807
Opened 20 years ago
Closed 19 years ago
[FIX]Docshell doesn't need an nsIScriptContext pointer
Categories
(Core :: DOM: Navigation, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla1.8beta2
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(1 file)
6.60 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
Patch coming up... We don't need this pointer because the window holds a ref to it.
Assignee | ||
Comment 1•20 years ago
|
||
Note that the change to ~nsWebShell is ok, since that calls Destroy().
Attachment #174764 -
Flags: superreview?(jst)
Attachment #174764 -
Flags: review?(jst)
Assignee | ||
Updated•20 years ago
|
Priority: -- → P2
Summary: Docshell doesn't need an nsIScriptContext pointer → [FIX]Docshell doesn't need an nsIScriptContext pointer
Target Milestone: --- → mozilla1.8beta2
Comment 2•20 years ago
|
||
Comment on attachment 174764 [details] [diff] [review] Patch r+sr=jst
Attachment #174764 -
Flags: superreview?(jst)
Attachment #174764 -
Flags: superreview+
Attachment #174764 -
Flags: review?(jst)
Attachment #174764 -
Flags: review+
Assignee | ||
Comment 3•19 years ago
|
||
Fixed on trunk for 1.8b2.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•