1095702 - [Browser] Limited scrolling on "Firefox Marketplace Privacy Policy" screen

Status: VERIFIED FIXED

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

Description

[Fred Muyumba [:FredM]]

Attachment: logcat_20141107_1217.txt

Description:
The User can NOT access text under "Firefox Marketplace Privacy Policy" due to limited scrolling
   
Repro Steps:
1) Update a Flame device to BuildID: 20141107001205
2) Launch "Settings" > Tap on "Device information"
3) Tap on "Your Privacy"
4) Tap on "Marketplace"
5) Scroll down to "what if we change this policy?"
6) Scroll down further
  
Actual:
The User is NOT able to scroll further down
  
Expected: 
The User is able to scroll further
  
Environmental Variables:
Device: Flame 2.1 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141107001205
Gaia: 6295f6acfe91c6ae659712747dd2b9c8f51d0339
Gecko: 8c23b4f2ba29
Version: 34.0
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
  
  
Repro frequency: 100%
See attached: video clip & logcat:http://youtu.be/OWfzK8rZ2Dc

Comment 1

[Fred Muyumba [:FredM]]

This issue is NOT reproducible on Flame 2.2 Master (319mb)(Kitkat Base)(Shallow Flash) and Flame 2.0 (319mb)(Kitkat Base)(Shallow Flash

Result: 
The User is able to scroll further and access text under "Firefox Marketplace Privacy Policy"

Environmental Variables:
Device: Flame 2.2(319mb)(KitKat)(Shallow Flash)
BuildID: 20141105040206
Gaia: 7c9e7cabbde941b976e0e40a3a1d94e21aa9c5e9
Gecko: 62990ec7ad78
Version: 36.0a1 
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

----------------------------------------------------------------------------------------------------------

Environmental Variables:
Device: Flame 2.0 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141105000201
Gaia: fe2167fa5314c7e71c143a590914cbf3771905a8
Gecko: 093de6b632c5
Version: 32.0 
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Comment 2

[Joshua Mitchell (Inactive)]

[Blocking Requested - why for this release]: Regression, poor UX

Comment 3

[Adam Alldredge [:AdamA][Inactive]]

Unable to obtain a regression window because this issue occurs on earliest 2.1 Flame KK build and does not reproduce on the JB base.

Earliest KK Environmental Variables:
Device: Flame 2.1
BuildID: 20140904062538
Gaia: a47ecb6368c015dd72148acde26413fd90ba3136
Gecko: ffb144a500a4
Version: 34.0a2 (2.1) 
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

JB build Environmental Variables:
Device: Flame 2.1
Build ID: 20141001060122
Gaia: b327c640fea887770d011a127e349838b3b44724
Gecko: 7359d0d0222d
Version: 34.0a2 (2.1)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Comment 4

[Joshua Mitchell (Inactive)]

Re-tagging regression window - if we can't get that, let's at least try and find a fixed-window to find what cause this to be fixed in 2.2

Comment 5

[Cody Roesch [:croesch]]

Removing Adam A. as QA contact since he has switched to a different team

Comment 6

[Pi Wei Cheng [:piwei] (inactive)]

mozilla-inbound reverse regression window:

Last Broken Environmental Variables:
Device: Flame
BuildID: 20140904140137
Gaia: de59e0c3614dd0061881fe284e9f2d74fa0d1d5d
Gecko: 1c0889637e60
Version: 35.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

First Working Environmental Variables:
Device: Flame
BuildID: 20140904140640
Gaia: de59e0c3614dd0061881fe284e9f2d74fa0d1d5d
Gecko: eb7169690808
Version: 35.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0

Gaia is the same so it's fixed by a Gecko commit.

Gecko pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=1c0889637e60&tochange=%20eb7169690808

Fixed by Bug 1062437. One contradiction is that bug 1062437 WAS uplifted to 2.1. We're unsure why 2.1 is still affected.

Comment 7

[Joshua Mitchell (Inactive)]

Also not sure about those results (re: the uplift to 2.1) - perhaps it deals with some combination of that patch and another patch that was uplifted to 2.2 only, prior to the window date - but that's just speculation. 

NI to patch author to take a look

Comment 8

[Botond Ballo [:botond]]

The symptoms are consistent with bug 1062437, so it makes sense that this bug is what fixed the issue in 2.1.

It's not obvious why the same bug didn't fix the issue in 2.1. Another bug that could cause similar problems is bug 1068268, but that has been uplifted to 2.1 as well.

We may just have to debug the issue on 2.1 to get to the bottom of it.

Comment 9

[bhavana bajaj [:bajaj]]

:botond, you the right assignee to help investigate this?

Comment 10

[Kartikaya Gupta (email:[e-mail])]

I have a relatively recent 2.1 tree lying around, I can start the investigation here.

Comment 11

[Kartikaya Gupta (email:[e-mail])]

On 2.1 the marketplace privacy policy content is in the root process, so it doesn't get scrolled with APZ. On master it ends up in a child process, so it does get scrolled with APZ. On 2.1 it gets sync-scrolled so scrollgrab doesn't work, and that means the rocketbar doesn't collapse, and that means the bottom of the content is cut off.

Comment 12

[Kartikaya Gupta (email:[e-mail])]

Somebody on the Gaia browser team should look into why this content is loading in the root process on 2.1. That doesn't seem like it should be happening.

Comment 13

[Gregor Wagner [:gwagner]]

Ben, Kevin, can you take a look here?

Comment 14

[Kevin Grandon :kgrandon]

This issue can be solved by uplifting bug 1066607, I think we should do so. Going to close this and nominate that one as a blocker.

Comment 15

[sarsenyev]

Verified the issue is fixed on 2.2 Flame

The "Privacy page" is scrollable without any blocking issues

"Flame 2.2

Device: Flame 2.2 Master (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141113040205
Gaia: be8b0151d2f9a4c41fc63952128e0b723cd1161d
Gecko: ab137ddd3746
Version: 36.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0"
_ _ _ __ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

Leaving verifyme for 2.1 patch uplift.

Comment 16

[Mike Lien[:mlien]]

verify with v2.1, it's fine
Gaia-Rev        18e02f16972f9d54931d57b13f35d2a2c0eda683
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/671e28afeeec
Build-ID        20141124161201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141124.194416
FW-Date         Mon Nov 24 19:44:27 EST 2014
Bootloader      L1TC10011880

refer to comment 14, mark v2.1 status as verified