Closed
Bug 265253
Opened 20 years ago
Closed 20 years ago
nsWebBrowser::RemoveWebBrowserListener incorrectly passes nsIWeakRef
Categories
(Core Graveyard :: Embedding: GRE Core, defect)
Core Graveyard
Embedding: GRE Core
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jhpedemonte, Assigned: dougt)
Details
Attachments
(1 file)
|
1.23 KB,
patch
|
darin.moz
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
RemoveWebBrowserListener() takes an |nsIWeakReference*| as the first parameter. The code at the bottom of that function queries the referent, but then passes the |nsIWeakReference*| to UnBindListener(); this function expects the referent.
|
Reporter | |
Comment 1•20 years ago
|
||
|
|
Reporter | |
Updated•20 years ago
|
Attachment #162716 -
Flags: review?(darin)
|
||
Updated•20 years ago
|
Summary: RemoveWebBrowserListener incorrectly passes nsIWeakRef → nsWebBrowser::RemoveWebBrowserListener incorrectly passes nsIWeakRef
|
||
Comment 2•20 years ago
|
||
Comment on attachment 162716 [details] [diff] [review] patch r+sr=darin
Attachment #162716 -
Flags: superreview+
Attachment #162716 -
Flags: review?(darin)
Attachment #162716 -
Flags: review+
|
|
Reporter | |
Comment 3•20 years ago
|
||
Checked into trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•