Summary: | [macOS] WebHTMLView has an internal retain cycle with its flagsChangedEventMonitor. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||||
Component: | WebKit API | Assignee: | Andreas Kling <kling> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aestes, commit-queue, kling | ||||||
Priority: | P2 | Keywords: | Performance | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Andreas Kling
2017-01-30 02:41:26 PST
Created attachment 300093 [details]
Patch
My objective-C is not very strong so please look carefully when reviewing.
Comment on attachment 300093 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300093&action=review > Source/WebKit/mac/WebView/WebHTMLView.mm:3512 > + _private->flagsChangedEventMonitor = [NSEvent addLocalMonitorForEventsMatchingMask:NSEventMaskFlagsChanged handler:[self] (NSEvent *flagsChangedEvent) { I think __block WebHTMLView *weakSelf = self; works in pre-arc world. Created attachment 300095 [details]
Patch for landing
Comment on attachment 300095 [details] Patch for landing Clearing flags on attachment: 300095 Committed r211362: <http://trac.webkit.org/changeset/211362> All reviewed patches have been landed. Closing bug. |