* SUMMARY Possible DDActionContext leaks based on shadey RetainPtr usage. Expected an adopt and there was none. - RetainPtr<DDActionContext> actionContext = [allocDDActionContextInstance() init]; + RetainPtr<DDActionContext> actionContext = adoptNS([allocDDActionContextInstance() init]);
Created attachment 246607 [details] [PATCH] Proposed Fix
Created attachment 246610 [details] [PATCH] Proposed Fix
Comment on attachment 246610 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=246610&action=review > Source/WebKit/mac/WebView/WebActionMenuController.mm:280 > + RetainPtr<DDActionContext> actionContext = adoptNS([allocDDActionContextInstance() init]); Nothing shady about it, it's just wrong. Good catch!
Comment on attachment 246610 [details] [PATCH] Proposed Fix Clearing flags on attachment: 246610 Committed r180119: <http://trac.webkit.org/changeset/180119>
All reviewed patches have been landed. Closing bug.