Title: [248985] branches/safari-608-branch/Source/WebCore
- Revision
- 248985
- Author
- [email protected]
- Date
- 2019-08-21 18:25:36 -0700 (Wed, 21 Aug 2019)
Log Message
Cherry-pick r248944. rdar://problem/54579628
REGRESSION: naver.com - Multiple taps are required to open email
https://bugs.webkit.org/show_bug.cgi?id=200979
<rdar://problem/54295239>
Patch by Antoine Quint <[email protected]> on 2019-08-21
Reviewed by Dean Jackson.
The "click" events on subjects in the message list are prevented by preventDefault() being called from a "mouseup" event listener.
This shouldn't actually happen, but due to how simulated mouse events are implemented, there is no current distinction between
"touchend" and "mouseup" events. It is safe however to opt mail.naver.com out of simulated mouse events altogether.
* page/Quirks.cpp:
(WebCore::Quirks::shouldDispatchSimulatedMouseEvents const):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248944 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-608-branch/Source/WebCore/ChangeLog (248984 => 248985)
--- branches/safari-608-branch/Source/WebCore/ChangeLog 2019-08-22 00:40:05 UTC (rev 248984)
+++ branches/safari-608-branch/Source/WebCore/ChangeLog 2019-08-22 01:25:36 UTC (rev 248985)
@@ -1,3 +1,38 @@
+2019-08-21 Kocsen Chung <[email protected]>
+
+ Cherry-pick r248944. rdar://problem/54579628
+
+ REGRESSION: naver.com - Multiple taps are required to open email
+ https://bugs.webkit.org/show_bug.cgi?id=200979
+ <rdar://problem/54295239>
+
+ Patch by Antoine Quint <[email protected]> on 2019-08-21
+ Reviewed by Dean Jackson.
+
+ The "click" events on subjects in the message list are prevented by preventDefault() being called from a "mouseup" event listener.
+ This shouldn't actually happen, but due to how simulated mouse events are implemented, there is no current distinction between
+ "touchend" and "mouseup" events. It is safe however to opt mail.naver.com out of simulated mouse events altogether.
+
+ * page/Quirks.cpp:
+ (WebCore::Quirks::shouldDispatchSimulatedMouseEvents const):
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248944 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-08-21 Antoine Quint <[email protected]>
+
+ REGRESSION: naver.com - Multiple taps are required to open email
+ https://bugs.webkit.org/show_bug.cgi?id=200979
+ <rdar://problem/54295239>
+
+ Reviewed by Dean Jackson.
+
+ The "click" events on subjects in the message list are prevented by preventDefault() being called from a "mouseup" event listener.
+ This shouldn't actually happen, but due to how simulated mouse events are implemented, there is no current distinction between
+ "touchend" and "mouseup" events. It is safe however to opt mail.naver.com out of simulated mouse events altogether.
+
+ * page/Quirks.cpp:
+ (WebCore::Quirks::shouldDispatchSimulatedMouseEvents const):
+
2019-08-20 Babak Shafiei <[email protected]>
Cherry-pick r248909. rdar://problem/54543354
Modified: branches/safari-608-branch/Source/WebCore/page/Quirks.cpp (248984 => 248985)
--- branches/safari-608-branch/Source/WebCore/page/Quirks.cpp 2019-08-22 00:40:05 UTC (rev 248984)
+++ branches/safari-608-branch/Source/WebCore/page/Quirks.cpp 2019-08-22 01:25:36 UTC (rev 248985)
@@ -323,6 +323,9 @@
// Disable the quirk for tv.naver.com subdomain to be able to simulate hover on videos.
if (equalLettersIgnoringASCIICase(host, "tv.naver.com"))
return false;
+ // Disable the quirk for mail.naver.com subdomain to be able to tap on mail subjects.
+ if (equalLettersIgnoringASCIICase(host, "mail.naver.com"))
+ return false;
// Disable the quirk on the mobile site.
// FIXME: Maybe this quirk should be disabled for "m." subdomains on all sites? These are generally mobile sites that don't need mouse events.
if (equalLettersIgnoringASCIICase(host, "m.naver.com"))
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
