1178761 - Incorrect navigator.userAgent requested via Request Desktop Site

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[u421692]

Open http://www.whatsmyua.com/ or http://people.mozilla.org/~atrain/mobile/tests/ua.html

Enable Request desktop site from menu

Actual result:
Incorrect user agent is displayed 
expected UA: Mozilla/5.0 (X11; Linux x86_64; rv:41.0) Gecko/20100101 Firefox/41.0
actual UA: Mozilla/5.0 (Android 4.4.4; Mobile; rv:41.0) Gecko/20100101 Firefox/41.0

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2694ff2ace6a&tochange=c319f262ce3e

Issue regressed from bug 1169772

Comment 1

[Mike Taylor [:miketaylr]]

Oops, my bad.

The regex at https://dxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#3206 no longer matches. Will write a patch in a bit.

Comment 2

[Mike Taylor [:miketaylr]]

Attachment: 1178761-Fix-DesktopUserAgent-regex-to-account-fo.patch

Mark, would you mind reviewing?

I'm using the following regex which does a non-greedy match from the first digit after "Android " until the first semicolon (same one used in Bug 1175301).

ua.replace(/Android \d.+?; [a-zA-Z]+/, "X11; Linux x86_64")

Comment 3

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

Comment on attachment 8627706 [details] [diff] [review]
1178761-Fix-DesktopUserAgent-regex-to-account-fo.patch

Thanks for the quick fix, Mike

Comment 4

[Mike Taylor [:miketaylr]]

Attachment: 1178761-Fix-DesktopUserAgent-regex-to-account-fo.patch

Adding r=mfinkle to commit message.

Thanks. I filed Bug 1178798 to add tests for this so I don't break it in the future--shouldn't be very hard but will take me away from some important stuff for too long right now. :/

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a49ba4bb6e64

Comment 5

[Mike Taylor [:miketaylr]]

OK, try looks good. Just one known intermittent.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/4dae2b5207a8

Comment 8

[Kevin Brosnan [:kbrosnan]]

[Tracking Requested - why for this release]: Regression in commonly used feature. Should fix before pushing aurora.

Comment 9

[[Back July 11] Ryan VanderMeulen]

https://hg.mozilla.org/mozilla-central/rev/4dae2b5207a8

Comment 10

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Adding a tracking flag for Firefox41. Kevin, if the patch looks good (has it been verified?), can we request uplift to Aurora?

Comment 11

[Flaviu Cos]

Verified as fixed in build 42.0a1 2015-07-02;
Device: Asus Transformer Pad (Android 4.2.1).

Comment 12

[Mike Taylor [:miketaylr]]

Comment on attachment 8627725 [details] [diff] [review]
1178761-Fix-DesktopUserAgent-regex-to-account-fo.patch

Approval Request Comment
[Feature/regressing bug #]:
"Request Desktop Site" functionality in Fennec (Bug 766406)

[User impact if declined]: 
"Request Desktop Site" will be broken for users until 42.

[Describe test coverage new/current, TreeHerder]:
mfinkle landed tests in Bug 1157319.

[Risks and why]: 
Low risk. This change just fixes a failing regular expression that is only used by the DesktopMode code.

[String/UUID change made/needed]:
None

Comment 13

[Kate Glazko]

Comment on attachment 8627725 [details] [diff] [review]
1178761-Fix-DesktopUserAgent-regex-to-account-fo.patch

Approving for uplift to Aurora, has been in m-c for about a week with no known issues. Low risk, tests made.

Comment 14

[[Back July 11] Ryan VanderMeulen]

https://hg.mozilla.org/releases/mozilla-aurora/rev/274a3a3aaade