Fix leaks in DumpRenderTree and WebKitTestRunner found by clang static analyzer. Also clean up a function signature and add CF_RETURNS_RETAINED where necessary to document behavior.
<rdar://problem/55387900>
Created attachment 378842 [details] Patch v1
Created attachment 378846 [details] Patch v2
(In reply to David Kilzer (:ddkilzer) from comment #2) > Created attachment 378842 [details] > Patch v1 Apparently CGEventRelease() isn't a thing!
Comment on attachment 378846 [details] Patch v2 View in context: https://bugs.webkit.org/attachment.cgi?id=378846&action=review > Tools/WebKitTestRunner/cocoa/TestRunnerWKWebView.mm:124 > + self.didStartFormControlInteractionCallback = nil; > + self.didEndFormControlInteractionCallback = nil; > + self.didShowForcePressPreviewCallback = nil; > + self.didDismissForcePressPreviewCallback = nil; > + self.willBeginZoomingCallback = nil; > + self.didEndZoomingCallback = nil; > + self.didShowKeyboardCallback = nil; > + self.didHideKeyboardCallback = nil; > + self.didShowMenuCallback = nil; > + self.didHideMenuCallback = nil; > + self.willPresentPopoverCallback = nil; > + self.didDismissPopoverCallback = nil; > + self.didEndScrollingCallback = nil; > + self.rotationDidEndCallback = nil; These should be taken care of by -resetInteractionCallbacks, no?
Comment on attachment 378846 [details] Patch v2 View in context: https://bugs.webkit.org/attachment.cgi?id=378846&action=review >> Tools/WebKitTestRunner/cocoa/TestRunnerWKWebView.mm:124 >> + self.rotationDidEndCallback = nil; > > These should be taken care of by -resetInteractionCallbacks, no? Oops! You are correct. Will remove.
Created attachment 378855 [details] Patch v3
Comment on attachment 378855 [details] Patch v3 Clearing flags on attachment: 378855 Committed r249901: <https://trac.webkit.org/changeset/249901>
All reviewed patches have been landed. Closing bug.