804481 - Add UA override for accounts.google.com

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect, P1)

Description

[Jason Smith [:jsmith]]

accounts.google.com is going to be used basically when you visit any site that requires login via your google account. We should probably consider putting a UA override here, as we currently get the desktop site right now.

Comment 1

[Jason Smith [:jsmith]]

Many top sites use accounts.google.com - putting this on the nom to block.

Comment 2

[Andrew Overholt [:overholt]]

Is there a corresponding evangelism bug for this site?

Comment 3

[Andrew Overholt [:overholt]]

Lawrence, what do you think about a UA override here?

Comment 4

[Jason Smith [:jsmith]]

(In reply to Andrew Overholt [:overholt] from comment #2)
> Is there a corresponding evangelism bug for this site?

Filed bug 805164.

Comment 5

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

(In reply to Andrew Overholt [:overholt] from comment #3)
> Lawrence, what do you think about a UA override here?

UA override is applicable for Google accounts for the reason Jason stated above. Joanne Nagel is currently in touch with Google, which has been willing to work with us on compat issues. We should first try evangelism before going to the whitelist.

Comment 6

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

Google is investigating the issues that we have on file in Bugzilla. I do not have a timeline on any fix at this point.

Comment 7

[Benoit Jacob [:bjacob] (mostly away)]

I have a question --- exactly the same as I asked in bug 803741 comment 10. Basically, is the patch as simple as

+pref("general.useragent.override.accounts.google.com", "\(Mobile#(Android; Mobile");

?

Comment 8

[Benoit Jacob [:bjacob] (mostly away)]

Attachment: Add UA override for accounts.google.com

Comment 9

[Alexandre Poirot [:ochameau]]

Comment on attachment 686559 [details] [diff] [review]
Add UA override for accounts.google.com

Tested, works fine.
You will have to submit a pull request in order to merge this patch.

Comment 10

[Benoit Jacob [:bjacob] (mostly away)]

pull request: https://github.com/mozilla-b2g/gaia/pull/6726

erm, this seems to also have the commit from my dailymotion branch. I'm a n00b with git and I don't know if that's going to be a problem.

Comment 11

[Alexandre Poirot [:ochameau]]

No git is smart :)
Landed:
https://github.com/mozilla-b2g/gaia/commit/409c7c0ce1309efcd2d4df38a16e5c62acef8167

Comment 12

[cbarker]

Verified Fixed in Unagi Build 20130103070201