783373 - Change mobile-specific Google search URL to standard Google search URL

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Brian Nicholson (:bnicholson)]

Attachment: Use standard Google search URL

Google's recent UA changes have made forcing the mobile-specific search unnecessary. We should be able to use the standard query URL.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 652558 [details] [diff] [review]
Use standard Google search URL

UX might want this uplifted as far as reasonable

Comment 2

[Brian Nicholson (:bnicholson)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/83f6e76262be

Comment 3

[Brian Nicholson (:bnicholson)]

Comment on attachment 652558 [details] [diff] [review]
Use standard Google search URL

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: Google mobile search is forced, meaning Request Desktop site option won't be honored for searches
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): medium-low risk - everything should be fine on our end, but there's always a chance Google could change something to break it again
String or UUID changes made by this patch: none

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 652558 [details] [diff] [review]
Use standard Google search URL

Approving, if there are changes on google's end we'll have to do outreach but changing our code here makes sense for uplift.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/83f6e76262be

Comment 6

[Brian Nicholson (:bnicholson)]

http://hg.mozilla.org/releases/mozilla-beta/rev/1b1c3270c679

Comment 7

[Brian Nicholson (:bnicholson)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/bbdbc7a0c457

Comment 8

[Axel Hecht [:Pike]]

What testing did this change get outside the US? There's at least bug 757979 that deals with the difference in experience in Japan for the previous iteration.

Comment 9

[Brian Nicholson (:bnicholson)]

(In reply to Axel Hecht [:Pike] from comment #8)
> What testing did this change get outside the US? There's at least bug 757979
> that deals with the difference in experience in Japan for the previous
> iteration.

The search engine XML files are localization-dependent; this patch is only for /mobile/locales/en-US/searchplugins/google.xml.

Comment 10

[Brian Nicholson (:bnicholson)]

(In reply to Brian Nicholson (:bnicholson) from comment #9)
> The search engine XML files are localization-dependent; this patch is only
> for /mobile/locales/en-US/searchplugins/google.xml.

After reading bug 757979, it looks like Japan (and other locales) were just using the English google.xml. https://bugzilla.mozilla.org/show_bug.cgi?id=757979#c25 sounds like the current implementation will be changed to use a single XML for all localizations, but that doesn't appear to be implemented yet.

I assumed different locales would just have their own XML files, meaning this change wouldn't affect them, but that doesn't appear to be the case. AFAIK, there wasn't any testing done for other languages for this patch.