Bug 198657 – Limit simulated mouse events on Google Maps to entering Street View

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Antoine Quint]

Limit simulated mouse events on Google Maps to entering Street View

[Antoine Quint]

Created attachment 371584 [details]
Patch

[Antoine Quint]

<rdar://problem/51345064>

[EWS Watchlist]

Attachment 371584 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:9:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Antoine Quint]

Created attachment 371585 [details]
Patch

[Brent Fulgham]

Comment on attachment 371585 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=371585&action=review

> Source/WebCore/page/Quirks.cpp:336
> +#endif

In file included from /Volumes/Data/EWS/WebKit/WebKitBuild/Release/DerivedSources/WebCore/unified-sources/UnifiedSource321.cpp:4:
./page/Quirks.cpp:324:70: error: unused parameter 'target' [-Werror,-Wunused-parameter]
bool Quirks::shouldDispatchSimulatedMouseEventsOnTarget(EventTarget* target) const
                                                                     ^
1 error generated.

You need an UNUSED_PARAM(target) here.

[Brent Fulgham]

Comment on attachment 371585 [details]
Patch

r=me with the fix I suggested.

[Antoine Quint]

(In reply to Brent Fulgham from comment #5)
> Comment on attachment 371585 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=371585&action=review
> 
> > Source/WebCore/page/Quirks.cpp:336
> > +#endif
> 
> In file included from
> /Volumes/Data/EWS/WebKit/WebKitBuild/Release/DerivedSources/WebCore/unified-
> sources/UnifiedSource321.cpp:4:
> ./page/Quirks.cpp:324:70: error: unused parameter 'target'
> [-Werror,-Wunused-parameter]
> bool Quirks::shouldDispatchSimulatedMouseEventsOnTarget(EventTarget* target)
> const
>                                                                      ^
> 1 error generated.
> 
> You need an UNUSED_PARAM(target) here.

Yeah, I figured :) But instead I'm going to make the two Quirk methods related to simulated mouse events only defined for PLATFORM(IOS_FAMILY) since it really doesn't make sense to ever call those on another platform.

[Antoine Quint]

Committed r246205: <https://trac.webkit.org/changeset/246205>

[Michael Catanzaro]

Committed r246216: <https://trac.webkit.org/changeset/246216>

[Michael Catanzaro]

Ah apologies Brent, I improperly pasted a Reviewed By line with your name in my build fix, but that is not correct because you didn't review my patch. Either I forgot to remove that line, or maybe (I suspect) webkit-patch added it in. (I don't think it used to do that. Or perhaps it did it because I forgot to write "unreviewed" in the commit message.)

Anyway, for the record, Brent didn't review r246216.

[Masataka Yakura]

https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/page/Quirks.cpp?rev=246205#L331

> if (equalLettersIgnoringASCIICase(host, "www.google.com") && url.path().startsWithIgnoringASCIICase("/maps/"))

Unlike other Google properties, people can access Google Maps using localized URLs (e.g. https://www.google.co.jp/maps , https://www.google.de/maps , https://www.google.com.hk/maps , and bunch more) and Google doesn't redirect them to google.com so if this quirk is not specific to the US version, we'll have trouble in non-US countries.

[Antoine Quint]

(In reply to Masataka Yakura from comment #11)
> https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/page/Quirks.
> cpp?rev=246205#L331
> 
> > if (equalLettersIgnoringASCIICase(host, "www.google.com") && url.path().startsWithIgnoringASCIICase("/maps/"))
> 
> Unlike other Google properties, people can access Google Maps using
> localized URLs (e.g. https://www.google.co.jp/maps ,
> https://www.google.de/maps , https://www.google.com.hk/maps , and bunch
> more) and Google doesn't redirect them to google.com so if this quirk is not
> specific to the US version, we'll have trouble in non-US countries.

Thanks Masataka, I filed https://bugs.webkit.org/show_bug.cgi?id=199904 to track that issue.