878657 - Add UA overrides for Hungary, Mexico, Montenegro, and Serbia

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Lawrence Mandel [:lmandel] (use needinfo)]

Attachment: UA overrides for HU, ME, MX, and RS

This bug adds UA overrides for the sites in the Alexa 100 for Hungary, Mexico, Montenegro, and Serbia that have been demonstrated to be functional on a Firefox OS device. This patch rounds out the UA overrides for the locales listed in the following post:

https://blog.mozilla.org/blog/2013/02/24/mozilla-unlocks-the-power-of-the-web-on-mobile-with-firefox-os/

Note: These overrides are specified slightly differently than the existing overrides ('\\' instead of '\') to avoid creating additional overrides that add warnings. See bug 843437.

Comment 1

[[:fabrice] Fabrice Desré]

Comment on attachment 757196 [details] [diff] [review]
UA overrides for HU, ME, MX, and RS

Review of attachment 757196 [details] [diff] [review]:
-----------------------------------------------------------------

sigh...

::: build/ua-override-prefs.js
@@ +166,5 @@
> +pref("general.useragent.override.eluniversal.com.mx", "\\(Mobile#(Android; Mobile"); // bug 878637
> +pref("general.useragent.override.flickr.com", "\\(Mobile#(Android; Mobile"); // bug 878638
> +pref("general.useragent.override.hootsuite.com", "\\(Mobile#(Android; Mobile"); // bug 878640
> +pref("general.useragent.override.mercadolibre.com.mx", "\\(Mobile#(Android; Mobile"); // bug 878642
> +pref("general.useragent.override.netflix.com", "Mozilla/5.0 (Linux; Android 4.0.4; Galaxy Nexus Build/IMM76B) AppleWebKit/535.19 (KHTML, like Gecko) Chrome/18.0.1025.133 Mobile Safari/535.19"); // bug 878643

netflix.com ? is that usable at all on device?

Comment 2

[Lawrence Mandel [:lmandel] (use needinfo)]

Attachment: Updated patch removing netflix

The Netflix.com mobile allows you to sign up for Netflix. At the end of the sign up you are then prompted to install the Android app. As the site is really only good at this point for sign up, I have removed it from the patch.

Carrying forward the r+ from fabrice.

Comment 3

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 757427 [details] [diff] [review]
Updated patch removing netflix

[Approval Request Comment]
Bug caused by (feature/regressing bug #): N/A
User impact if declined: Desktop instead of mobile content for top sites in the four locales
Testing completed: QA has completed functional testing on device for each site on the override list.
Risk to taking this patch (and alternatives if risky): Low risk as this patch adds a number of preferences. The alternative is desktop content for these sites.
String or UUID changes made by this patch: None

Comment 4

[Ryan VanderMeulen [:RyanVM]]

John, can you please assist? :)

Comment 5

[John Ford [:jhford] CET/CEST Berlin Time]

master: 5b01f824e0d8ff6b09be85de7366aa9394888f50

Comment 6

[Ryan VanderMeulen [:RyanVM]]

John, can you please assist with the uplift? :)

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

v1-train: fded0b9