WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-07-18 05:52:12 PDT
<
rdar://problem/53250104
>
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
Created
attachment 374408
[details]
Patch
Antoine Quint
Comment 5
2019-07-18 14:56:43 PDT
Committed
r247621
: <
https://trac.webkit.org/changeset/247621
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug