964981 - [B2G][Contacts] Scrolling in detail view of a contact with an image produces graphical issues showing layer below

Status: RESOLVED FIXED

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

Description

[Brogan Zumwalt [Inactive]]

Description:
In the contact app, when user is in the detailed view of a contact with an image associated with the individual, scrolling down (to reveal more of the associated image) then rapidly dragging finger up page reveals main contact list beneath detail view layer. This graphics issue occurs only with APZ enabled.

Repro Steps:
1) Updated Buri to BuildID: 20140128040224
2) Ensure APZ is enabled in settings (Settings>Device Information>More Information>Developer)
2) Open contact app (with multiple contacts already in list)
3) Tap existing contact that has an picture associated with the individual
4) Drag finger down screen, revealing more of the image header
5) Scroll up rapidly, observing bottom of screen while doing so

Actual:
Scrolling in detail view of contact reveals layer below.

Expected:
Scrolling in detail view of contact produces no graphical issues.

Environmental Variables:
Device: Buri v1.4 Master Mozilla RIL
BuildID: 20140128040224
Gaia: 6586d2b8b43c6997be5cf5895bbdf3dd20490725
Gecko: 4da3e21a0e5f
Version: 29.0a1
Firmware Version: V1.2-device.cfg

Notes:
Repro frequency: 3/3, 100%
See attached: video clip

Comment 1

[Brogan Zumwalt [Inactive]]

Issue also occurs in latest 1.3

Result:
Scrolling in detail view of contact reveals layer below.

Environmental Variables:
Device: Buri v1.3 Mozilla RIL
BuildID: 20140128004000
Gaia: 3c5119507a985c59cfc264ed23821679b138486d
Gecko: 4936f09590a0
Version: 28.0a2
Firmware Version: V1.2-device.cfg

Comment 2

[Jason Smith [:jsmith]]

Does this reproduce with APZC disabled? Does this reproduce on 1.1?

Comment 3

[Brogan Zumwalt [Inactive]]

Issue only occurs with APZC enabled. Issue does not occur with APZC disabled.

Comment 4

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

Botond, could you look into this one?

Comment 5

[Botond Ballo [:botond]]

(In reply to Brogan Zumwalt from comment #3)
> Issue only occurs with APZC enabled. Issue does not occur with APZC disabled.

Hmm. I see this issue with both APZC enabled and APZC disabled. Tested with Buri running 1.3 and Nexus 4 running master, same results.

Comment 6

[Botond Ballo [:botond]]

(In reply to Botond Ballo [:botond] from comment #5)
> (In reply to Brogan Zumwalt from comment #3)
> > Issue only occurs with APZC enabled. Issue does not occur with APZC disabled.
> 
> Hmm. I see this issue with both APZC enabled and APZC disabled. Tested with
> Buri running 1.3 and Nexus 4 running master, same results.

Actually, there is a difference: with APZC off, I have to _fling_ to see the underlying layers. With APZC on, _panning_ sufficiently quickly exposes them as well.

Comment 7

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

Sounds like an issue with the contacts app then? More importantly it means we should be able to get a regression window of when it started happening, which presumably will be not when APZC was turned on.

Comment 8

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

I can also see this with APZC disabled on the latest 1.3 code on Hamachi. Bouncing back to Gaia team.

Comment 9

[Etienne Segonzac (:etienne)]

I can reproduce it, having a look...

Comment 10

[Etienne Segonzac (:etienne)]

Attachment: Gaia PR

Sweet and simple.

Comment 11

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8368677 [details] [review]
Gaia PR

Hardest review ever!

Thanks Etienne for taking care of this

Comment 12

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/4470879d2fec6c6344e72e7390632923eba224d6

Comment 13

[John Ford [:jhford] CET/CEST Berlin Time]

I was not able to uplift this bug to v1.3.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1.3, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1.3
  git cherry-pick -x -m1 4470879d2fec6c6344e72e7390632923eba224d6
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 14

[Etienne Segonzac (:etienne)]

Landed on v1.3

https://github.com/mozilla-b2g/gaia/commit/74e024ad94769a17893ec1f94fb579698a5d6d2f