Title: [248745] branches/safari-608-branch/Source/WebKitLegacy/mac
- Revision
- 248745
- Author
- [email protected]
- Date
- 2019-08-15 13:50:02 -0700 (Thu, 15 Aug 2019)
Log Message
Cherry-pick r248741. rdar://problem/54360841
Yellow Lookup highlight gets stuck over Mail messages
https://bugs.webkit.org/show_bug.cgi?id=200778
<rdar://problem/53868514>
Reviewed by Wenson Hsieh.
* WebView/WebView.mm:
(-[WebView _showDictionaryLookupPopup:]):
Add a dismissal callback so that when Reveal hides the panel, it also
dismisses the yellow indicator. This matches the behavior in modern WebKit.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248741 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-608-branch/Source/WebKitLegacy/mac/ChangeLog (248744 => 248745)
--- branches/safari-608-branch/Source/WebKitLegacy/mac/ChangeLog 2019-08-15 20:50:00 UTC (rev 248744)
+++ branches/safari-608-branch/Source/WebKitLegacy/mac/ChangeLog 2019-08-15 20:50:02 UTC (rev 248745)
@@ -1,3 +1,34 @@
+2019-08-15 Alan Coon <[email protected]>
+
+ Cherry-pick r248741. rdar://problem/54360841
+
+ Yellow Lookup highlight gets stuck over Mail messages
+ https://bugs.webkit.org/show_bug.cgi?id=200778
+ <rdar://problem/53868514>
+
+ Reviewed by Wenson Hsieh.
+
+ * WebView/WebView.mm:
+ (-[WebView _showDictionaryLookupPopup:]):
+ Add a dismissal callback so that when Reveal hides the panel, it also
+ dismisses the yellow indicator. This matches the behavior in modern WebKit.
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248741 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-08-15 Tim Horton <[email protected]>
+
+ Yellow Lookup highlight gets stuck over Mail messages
+ https://bugs.webkit.org/show_bug.cgi?id=200778
+ <rdar://problem/53868514>
+
+ Reviewed by Wenson Hsieh.
+
+ * WebView/WebView.mm:
+ (-[WebView _showDictionaryLookupPopup:]):
+ Add a dismissal callback so that when Reveal hides the panel, it also
+ dismisses the yellow indicator. This matches the behavior in modern WebKit.
+
2019-07-29 Alan Coon <[email protected]>
Cherry-pick r247837. rdar://problem/53648220
Modified: branches/safari-608-branch/Source/WebKitLegacy/mac/WebView/WebView.mm (248744 => 248745)
--- branches/safari-608-branch/Source/WebKitLegacy/mac/WebView/WebView.mm 2019-08-15 20:50:00 UTC (rev 248744)
+++ branches/safari-608-branch/Source/WebKitLegacy/mac/WebView/WebView.mm 2019-08-15 20:50:02 UTC (rev 248745)
@@ -246,6 +246,7 @@
#import <wtf/SetForScope.h>
#import <wtf/SoftLinking.h>
#import <wtf/StdLibExtras.h>
+#import <wtf/WeakObjCPtr.h>
#import <wtf/WorkQueue.h>
#import <wtf/spi/darwin/dyldSPI.h>
@@ -9525,6 +9526,8 @@
[self _setTextIndicator:textIndicator withLifetime:TextIndicatorWindowLifetime::Permanent];
}, [self](FloatRect rectInRootViewCoordinates) {
return [self _convertRectFromRootView:rectInRootViewCoordinates];
+ }, [weakSelf = WeakObjCPtr<WebView>(self)]() {
+ [weakSelf.get() _clearTextIndicatorWithAnimation:TextIndicatorWindowDismissalAnimation::FadeOut];
});
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
