Closed Bug 1167402 Opened 5 years ago Closed 5 years ago
Pres Context::m Link Handler as MOZ _NON _OWNING _REF
1.35 KB, patch
|Details | Diff | Splinter Review|
Could you explain what steps you took to convince yourself that this patch won't introduce new leaks?
In retrospect, this should have been a MOZ_NON_OWNING_REF. Thanks for making me look closer at it :). I'll push an updated patch.
Comment on attachment 8612320 [details] [diff] [review] Make nsPresContext::mLinkHandler a smart pointer Please fix the commit message and the bug summary to reflect that you're now marking it MOZ_NON_OWNING_REF. r=dbaron
Attachment #8612320 - Flags: review?(dbaron) → review+
Oops commit message
Attachment #8612320 - Attachment is obsolete: true
Summary: Make nsPresContext::mLinkHandler a smart pointer → Mark nsPresContext::mLinkHandler as MOZ_NON_OWNING_REF
You need to log in before you can comment on or make changes to this bug.