Changeset 246205 in webkit

Timestamp:

Jun 7, 2019, 10:11:01 AM (7 years ago)

Author:

graouts@webkit.org

Message:

Limit simulated mouse events on Google Maps to entering Street View
​https://bugs.webkit.org/show_bug.cgi?id=198657
<rdar://problem/51345064>

Reviewed by Brent Fulgham.

• page/Quirks.cpp:

(WebCore::Quirks::shouldDispatchSimulatedMouseEvents const):
(WebCore::Quirks::shouldDispatchSimulatedMouseEventsOnTarget const):

• page/Quirks.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/Quirks.cpp (modified) (2 diffs)
• page/Quirks.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-06-07  Antoine Quint  <graouts@apple.com>
+
+        Limit simulated mouse events on Google Maps to entering Street View
+        https://bugs.webkit.org/show_bug.cgi?id=198657
+        <rdar://problem/51345064>
+
+        Reviewed by Brent Fulgham.
+
+        * page/Quirks.cpp:
+        (WebCore::Quirks::shouldDispatchSimulatedMouseEvents const):
+        (WebCore::Quirks::shouldDispatchSimulatedMouseEventsOnTarget const):
+        * page/Quirks.h:
+
 2019-06-07  Joonghun Park  <jh718.park@samsung.com>
 

trunk/Source/WebCore/page/Quirks.cpp

@@ -283 +283 @@
 }
 
+#if PLATFORM(IOS_FAMILY)
 bool Quirks::shouldDispatchSimulatedMouseEvents() const
 {
-#if PLATFORM(IOS_FAMILY)
     if (!needsQuirks())
         return false;
@@ -318 +318 @@
     if (equalLettersIgnoringASCIICase(host, "naver.com") || host.endsWithIgnoringASCIICase(".naver.com"))
         return true;
-#endif
-    return false;
-}
+    return false;
+}
+
+bool Quirks::shouldDispatchSimulatedMouseEventsOnTarget(EventTarget* target) const
+{
+    if (!needsQuirks() || !shouldDispatchSimulatedMouseEvents())
+        return false;
+
+    // On Google Maps, we want to limit simulated mouse events to dragging the little man that allows entering into Street View.
+    auto& url = m_document->topDocument().url();
+    auto host = url.host();
+    if (equalLettersIgnoringASCIICase(host, "www.google.com") && url.path().startsWithIgnoringASCIICase("/maps/"))
+        return is<Element>(target) && downcast<Element>(target)->getAttribute("class") == "widget-expand-button-pegman-icon";
+    return true;
+}
+#endif
 
 bool Quirks::shouldDisablePointerEventsQuirk() const

trunk/Source/WebCore/page/Quirks.h

@@ -50 +50 @@
     bool hasBrokenEncryptedMediaAPISupportQuirk() const;
     bool hasWebSQLSupportQuirk() const;
+#if PLATFORM(IOS_FAMILY)
     bool shouldDispatchSimulatedMouseEvents() const;
+    bool shouldDispatchSimulatedMouseEventsOnTarget(EventTarget*) const;
+#endif
     bool shouldDisablePointerEventsQuirk() const;
     bool needsInputModeNoneImplicitly(const HTMLElement&) const;