RESOLVED FIXED 199904
Ensure Quirks::shouldDispatchSimulatedMouseEvents() works for all Google Maps and Amazon domains
https://bugs.webkit.org/show_bug.cgi?id=199904
Summary Ensure Quirks::shouldDispatchSimulatedMouseEvents() works for all Google Maps...
Antoine Quint
Reported 2019-07-18 05:51:50 PDT
A comment made on https://bugs.webkit.org/show_bug.cgi?id=198657 indicates that the quirk added to dispatch simulated mouse events to the Street View icon on Google Maps only works on the google.com domain but Google Maps could also be served via a google.co.jp or google.fr domain for which the quirk doesn't work! Likewise, Quirks::shouldDispatchSimulatedMouseEvents() uses amazon.com to check whether we are on an Amazon property, but this is bad since Amazon also uses amazon.fr and other local domains.
Attachments
Patch (4.09 KB, patch)
2019-07-18 13:33 PDT, Antoine Quint
dino: review+
Radar WebKit Bug Importer
Comment 1 2019-07-18 05:52:12 PDT
Michael Catanzaro
Comment 2 2019-07-18 10:53:33 PDT
The solution we use for Google user agent quirks is: String baseDomain = topPrivatelyControlledDomain(url.host().toString()); if (baseDomain.startsWith("google.")) // do something We also have quirks for gstatic.com, googleapis.com, and googleusercontent.com, but none of those are likely relevant to your Maps problem.
Antoine Quint
Comment 3 2019-07-18 10:55:25 PDT
Thanks Michael, this is really helpful! I'll use a similar approach to detect a Google and Amazon property.
Antoine Quint
Comment 4 2019-07-18 13:33:46 PDT
Antoine Quint
Comment 5 2019-07-18 14:56:43 PDT
Note You need to log in before you can comment on or make changes to this bug.