1097318 - Fix talos regression when new tablet UI is enabled

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P1)

Description

[Lucas Rocha (:lucasr)]

For some reason, when the chrome area is taller than 104dp, testCheck2 misbehaves and reports a massive regression. In bug 1097316, I'm keeping the chrome area exactly at 104dp as a temporary workaround. We need to figure out why the bigger chrome area causes regressions in testCheck2.

Comment 1

[Lucas Rocha (:lucasr)]

Attachment: Restore tab strip to its original height (r=mfinkle)

Comment 2

[Lucas Rocha (:lucasr)]

Comment on attachment 8524629 [details] [diff] [review]
Restore tab strip to its original height (r=mfinkle)

From IRC discussion with snorp: Let's see if this also causes an eideticker regression in timecube or nytimes-scroll. If it does cause regressions then the regression is testCheck2 is real. If not, we'll just take the new testCheck2 results as the new baseline.

I'll be watching the eideticker results in inbound.

Comment 3

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

Comment on attachment 8524629 [details] [diff] [review]
Restore tab strip to its original height (r=mfinkle)

Looks OK, but we need to make sure eideticker tests are working before we land.

Comment 4

[Lucas Rocha (:lucasr)]

wlach, could you please run eideticker timecube and nytimes-scroll with this apk?
https://dl.dropboxusercontent.com/u/1187037/new-tablet-ui.apk

Comment 5

[William Lachance (:wlach)]

This won't actually work because we don't have the tablet UI on the galaxy nexus :(

Could we just do a manual visual inspection of the behaviour here? It should be obvious if a website like timecube is really misbehaving.

Comment 6

[James Willcox (:snorp) ([e-mail]) (he/him)]

I think I'm ok just ignoring this regression for now. I tried it locally and everything seemed fine. I think it's likely some kind of problem with the test.

Comment 7

[Lucas Rocha (:lucasr)]

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

Comment 8

[Lucas Rocha (:lucasr)]

Comment on attachment 8524629 [details] [diff] [review]
Restore tab strip to its original height (r=mfinkle)

Approval Request Comment
[Feature/regressing bug #]: New tablet UI (bug 1014156)
[User impact if declined]: Too small padding on top of tab strip. Makes us look amateur.
[Describe test coverage new/current, TBPL]: Local only. Let's bake this in Nightly for a bit and then uplift. 
[Risks and why]: Low, just tweaks a couple of resource values.
[String/UUID change made/needed]: n/a

Comment 9

[Carsten Book [:Tomcat]]

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

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8524629 [details] [diff] [review]
Restore tab strip to its original height (r=mfinkle)

> let's bake in Nightly for a bit and then uplift.
revert my approval

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/bf5d91d029af

Comment 12

[Flaviu Cos]

Verified as fixed in builds:
- 38.0a1 2015-01-14;
- 37.0a2 2015-01-14;
- 36 Beta 1;
Device: Asus Transformer Tab (Android 4.0.3).

Comment 13

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

If we wanted to run this locally and diagnose this issue properly...

<gbrown> capella: I ran testCheck2 locally -- https://wiki.mozilla.org/Mobile/Fennec/Android#talos updated
<gbrown> capella: for now, note that you will need to apply the patch from bug 1122701