Reviewed by John Sullivan and Ada Chan.
* UIProcess/API/mac/WKView.mm:
(-[WKView viewDidMoveToWindow]): Since this method can be called multiple times in a row
while [self window] is non-nil, check if there isn’t already an event monitor before adding
a new one.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@105112
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2012-01-16 Dan Bernstein <mitz@apple.com>
+
+ <rdar://problem/10702758> REGRESSION: Every Safari Reader WKView leaks
+
+ Reviewed by John Sullivan and Ada Chan.
+
+ * UIProcess/API/mac/WKView.mm:
+ (-[WKView viewDidMoveToWindow]): Since this method can be called multiple times in a row
+ while [self window] is non-nil, check if there isn’t already an event monitor before adding
+ a new one.
+
2012-01-16 Anders Carlsson <andersca@apple.com>
Crash when calling SecItemAdd with a null result pointer
[self _updateWindowVisibility];
[self _updateWindowAndViewFrames];
- _data->_flagsChangedEventMonitor = [NSEvent addLocalMonitorForEventsMatchingMask:NSFlagsChangedMask handler:^(NSEvent *flagsChangedEvent) {
- [self _postFakeMouseMovedEventForFlagsChangedEvent:flagsChangedEvent];
- return flagsChangedEvent;
- }];
+ if (!_data->_flagsChangedEventMonitor) {
+ _data->_flagsChangedEventMonitor = [NSEvent addLocalMonitorForEventsMatchingMask:NSFlagsChangedMask handler:^(NSEvent *flagsChangedEvent) {
+ [self _postFakeMouseMovedEventForFlagsChangedEvent:flagsChangedEvent];
+ return flagsChangedEvent;
+ }];
+ }
[self _accessibilityRegisterUIProcessTokens];
} else {