Bug 132602 – [GTK] [Stable] Can't browse new Google maps with latest stable

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Xabier Rodríguez Calvar]

Since I updated and recompiled my webkit stack, I can't see the new Google Maps interface properly. It opens, but after I search some place I can see nothing.

[Carlos Alberto Lopez Perez]

Seems this is caused by the string "Version/6.0" of the user-agent that WebKitGTK+ sends.

I guess that reverting r166405 <http://trac.webkit.org/r166405> would fix this issue.


If you have epiphany, you can test it with:

# set the UA with the "Version/6.0" substring and try google maps
gsettings set 'org.gnome.Epiphany' 'user-agent' 'Mozilla/5.0 (Macintosh; Intel Mac OS X) AppleWebKit/538.15 (KHTML, like Gecko) Safari/538.15 Version/6.0 Debian/unstable (3.12.0-1) Epiphany/3.12.0'

# now set the UA without the "Version/6.0" substring and try again
gsettings set 'org.gnome.Epiphany' 'user-agent' 'Mozilla/5.0 (Macintosh; Intel Mac OS X) AppleWebKit/538.15 (KHTML, like Gecko) Safari/538.15 Debian/unstable (3.12.0-1) Epiphany/3.12.0'

[Alberto Garcia]

Reproduced with r168307

[Michael Catanzaro]

(In reply to comment #1)
> Seems this is caused by the string "Version/6.0" of the user-agent that WebKitGTK+ sends.
> 
> I guess that reverting r166405 <http://trac.webkit.org/r166405> would fix this issue.

As an FYI: the user agent prior to r166405 caused us to receive the fallback www.google.com (with the black header bar and the I'm feeling lucky button) for ancient browsers. This annoyed me perhaps more than it should have.

[Carlos Garcia Campos]

*** Bug 134631 has been marked as a duplicate of this bug. ***

[Carlos Garcia Campos]

Created attachment 234402 [details]
Patch

This is only for stable for now. The google maps quirk is not needed in trunk, but maybe the other google domains quirk is required, I don't really know.

[Carlos Garcia Campos]

Created attachment 234403 [details]
Updated patch

Patch updated to cover the case where Google Maps is used with url http://google.com/maps instead of maps.google.com

[Sergio Villar Senin]

Comment on attachment 234403 [details]
Updated patch

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

Overall the patch looks good but I disagree on removing only the "Version" part. See bellow.

> Source/WebCore/platform/gtk/UserAgentGtk.cpp:253
> +    uaString.appendLiteral(" Safari/");

I wouldn't do this. As I mentioned in the change that added the "Version" part, there is no point on having "Safari/" without the "Version" string as that's invalid. So either we add them both or we add nothing.

That'd mean that we should likely rename the quirk to something like ClaimToBeSafari or something.

[Alberto Garcia]

(In reply to comment #7)
> there is no point on having "Safari/" without the "Version" string
> as that's invalid. So either we add them both or we add nothing.

Google maps works fine if we add nothing. The interesting thing is
that in that case the "Try the new Google Maps" option doesn't appear.

[Sergio Villar Senin]

(In reply to comment #8)
> (In reply to comment #7)
> > there is no point on having "Safari/" without the "Version" string
> > as that's invalid. So either we add them both or we add nothing.
> 
> Google maps works fine if we add nothing. The interesting thing is
> that in that case the "Try the new Google Maps" option doesn't appear.

Another option could be to downgrade the Version number we use in stable so that google and other providers could safely assume that we have a given JS API. What about using "Version 7"? Does it fix the issues?

[Alberto Garcia]

(In reply to comment #9)
> What about using "Version 7"? Does it fix the issues?

No, it actually fails already with Version 6.

Version 5 (if such a thing exists) works fine though, sending the old
version of the maps.

[Xabier Rodríguez Calvar]

(In reply to comment #8)
> (In reply to comment #7)
> > there is no point on having "Safari/" without the "Version" string
> > as that's invalid. So either we add them both or we add nothing.
> 
> Google maps works fine if we add nothing. The interesting thing is
> that in that case the "Try the new Google Maps" option doesn't appear.

The problem happens when you switched already to the new maps. In that case, it's non-functional.

[Alberto Garcia]

(In reply to comment #11)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > there is no point on having "Safari/" without the "Version" string
> > > as that's invalid. So either we add them both or we add nothing.
> >
> > Google maps works fine if we add nothing. The interesting thing is
> > that in that case the "Try the new Google Maps" option doesn't appear.
>
> The problem happens when you switched already to the new maps. In
> that case, it's non-functional.

If you have only the "Safari/" part of the user-agent it will show you
the old maps plus the button to try the new ones. The interesting
thing is that in that case the new maps do work fine.

[Alberto Garcia]

Anyway, removing both the "Version/" and the "Safari/" part for the maps lgtm, can we do this then? I think it's the only missing patch for the 2.4.4 release, isn't it?

[Carlos Garcia Campos]

Created attachment 234550 [details]
Updated path

Rename the quirk and remove the whole safari part of the user agent as suggested by Sergio

[Alberto Garcia]

(In reply to comment #14)
> Created an attachment (id=234550) [details]
> Updated path
> 
> Rename the quirk and remove the whole safari part of the user agent as suggested by Sergio

Tested, it works fine.

[Sergio Villar Senin]

Comment on attachment 234550 [details]
Updated path

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

Let's go for it

> Source/WebCore/platform/gtk/UserAgentGtk.cpp:252
> +        uaString.appendLiteral(" Safari/");

Use a single appendLiteral() here

[Carlos Garcia Campos]

Committed <http://trac.webkit.org/changeset/170882>