1338260 - Add UA override for directTV

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mike Taylor [:miketaylr]]

Pretending to be Android fixes both issues reported in Bug 1338128. Let's ship an override then try to get DirecTV to upgrade their version of Sencha Touch.

Comment 1

[Mike Taylor [:miketaylr]]

Attachment: Bug 1338260. Add Fennec-only UA override for directv.com.

They use an older version of Sencha Touch (v2.1.0) which is known
to not be compatibile with Firefox. However, pretending to be Chrome
fixes the issue of not being able to scroll and various paint bugs.

Review commit: https://reviewboard.mozilla.org/r/111328/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/111328/

Comment 2

[Mike Connor [:mconnor]]

Comment on attachment 8835654 [details]
Bug 1338260. Add Fennec-only UA override for directv.com.

https://reviewboard.mozilla.org/r/111328/#review112558

awesome, thanks!

Comment 3

[Mike Taylor [:miketaylr]]

Thanks! I'm gonna land this w/o a try run because there's no real tests around it (and I've built and tested the patch locally to confirm it like, works and doesn't blow up).

Comment 4

[Mike Taylor [:miketaylr]]

Comment on attachment 8835654 [details]
Bug 1338260. Add Fennec-only UA override for directv.com.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/111328/diff/1-2/

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2ce5a893da4f
Add Fennec-only UA override for directv.com. r=mconnor

Comment 6

[Mike Kaply [:mkaply]]

We'll want to get this uplifted all the way to release for now. I'll work with relman to figure out how we can get a build with it.

Comment 7

[Mike Kaply [:mkaply]]

Comment on attachment 8835654 [details]
Bug 1338260. Add Fennec-only UA override for directv.com.

Approval Request Comment
[Feature/Bug causing the regression]: Make directv work
[User impact if declined]: directv.com doesn't work properly (partner issue)
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: See bug 1338128
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Modifies existing table adding exception for 1 sight.
[String changes made/needed]:


As far as mozilla-release goes, we're going to need to do a one off . release for this partner. I'll contact relman separately.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2ce5a893da4f

Comment 9

[Julien Cristau [:jcristau]]

Comment on attachment 8835654 [details]
Bug 1338260. Add Fennec-only UA override for directv.com.

UA override for fennec, for aurora and beta

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/674ddd0ae1e6

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/fe1a313151e8

Comment 12

[Gerry Chang [:gchang]]

Comment on attachment 8835654 [details]
Bug 1338260. Add Fennec-only UA override for directv.com.

UA override for fennec. Release51+.

Comment 13

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-release/rev/c8c53bdd499ba5c6d08960bf003e648f4b2efd77

Comment 14

[Liz Henry (:lizzard) (relman/hg->git project)]

We're building 51.0.4 especially for this but won't be uploading it to Google Play. 
Do we know that this doesn't break things for other devices or sites?
What testing can we do here (especially to make sure it is OK on beta 52)?

Comment 15

[Mike Taylor [:miketaylr]]

Here's a manual test plan:

-1) verify that the UA override has been synced to the CDN:
https://dynamicua.cdn.mozilla.net/0/%7Baa3c5121-dab2-40e2-81ca-7ea25febc110%7D
0) open browser, clear app data (this forces the browser to load the override list from the APK, otherwise it could be up to a 
week before it calls home for new overrides)


1) navigate to directv.com
2) scroll down

PASS if you can scroll to the bottom
FAIL if it jumps back up automatically

3) rotate device to landscape orientation

PASS if you see the whole viewport, in landscape orientation, and can scroll to the bottom
FAIL if you see partially painted viewport, and cannot scroll to the bottom

4) go to about:config?filter=general.useragent.updates.enabled, set to false
5) try to scroll, note that you cannot.
6) rotate device to landscape orientation, note "cut off" paint bug and inability to scroll
7) go to about:config?filter=general.useragent.updates.enabled, set to true
8) reload directv.com
9) re-test steps 1 through 3

As for testing if it doesn't break other sites, I guess you can just make sure google.com loads, or something, before and after toggling the pref. 

The patch only comes into play for the directv.com TLD and subdomains.

Comment 16

[Bogdan Surd, QA [:bsurd, NI]]

Devices:
 - Nexus 9 (Android 7.1.1)
 - Nexus 6P (Android 7.0)
 - Honor 8 (Android 6.0)

Hello,

 I've verified this issue, it seems that other sites are not affected by this setting (Google, Yahoo, Bing, Amazon), they load and work as intended.
 With the value set to true directtv.com works without any issue. If set to false the page loads and behaves as described, no scrolling is possible and the page is "cut off".

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr52/rev/fe1a313151e8