831973 - Non-system apps from Marketplace and everything me scroll up whitespace until only whitepsace remains.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Guillermo Huerta [Disabled MoCo account]]

Noticed this after updating to the latest build.  The Twitter app from Marketplace and the facebook app from Everything.me scroll up whitespace when you scroll down in them.  This can be scrolled down by scrolling back to the top, but no information below the whitespace is visible.

Comment 1

[Guillermo Huerta [Disabled MoCo account]]

Actually I'm seeing this in the marketplace itself.  When the whitespace scrolls up it changes the touch mapping as well proportionally to the size of screen that remains visible.

Comment 4

[Jason Smith [:jsmith]]

Noming to block - smoketest regression. This prevents logging into the facebook app.

Comment 5

[Jason Smith [:jsmith]]

Thought it was a keyboard bug, but after seeing the dupes I don't think it is.

Something blew up royally in the scrolling dept - Marketplace installed apps aren't even rendering post scrolling right now.

Comment 6

[Jason Smith [:jsmith]]

Chris - Any ideas?

This seems like a recent regression.

Comment 7

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Bug 811950 might be responsible.  Possibly bug 830256 which hasn't been uplifted yet.

Comment 8

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Why are apps installed from the marketplace behaving so much differently than preinstalled apps?  Please don't tell me we're launching them as a window.open from the system app ...

Comment 9

[Jeni]

Attachment: [B2G] Showing white space when scrolling upward. White screen raises when scrolling up and lowers when scrolling downward

Issue also happens when trying to login to sync contacts.  Build 20130117070201

Comment 10

[David Lawrence [:dkl]]

FWIW this did not happen with unagi_beta_20130114070200.zip (14-Jan-2013 08:08).

Comment 11

[Jason Smith [:jsmith]]

Bumping up to critical because this is making a bunch of apps/sites partially unusable.

Comment 12

[Jeni]

This is also happening with the HERE Maps app thats on the phone. 

Repro Steps
1. Click on top right hand icon
2. Click on route
3. Enter address/location in From and To location
4. click on the list button on the right hand side of screen
5.  Start scrolling up and view what happens to the screen/page

Build 20130117070201 Kernel Dec 5th

Comment 13

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Chris Jones [:cjones] [:warhammer] from comment #8)
> Why are apps installed from the marketplace behaving so much differently
> than preinstalled apps?  Please don't tell me we're launching them as a
> window.open from the system app ...

Hopefully not. There use the same regular mechanism. app.launch().

Comment 15

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I am also seeing this on the Facebook integration... is this w/ bug 830256 ?  there's a patch that landed in inbound, but not on b2g 18.

Comment 16

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I'm seeing this in the browser as well when clicking to create a new bug in landfill.bugzilla.org.  cc'ing ben on this bug so he's aware.

Comment 20

[Julien Wajsberg [:julienw]]

Bug 832222 has some screenshots.

Comment 21

[Milan Sreckovic [:milan] (needinfo for best results)]

Anthony, Jeff, this is still a blocker, we think - any progress?

Comment 22

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Attachment: Remove extra scroll compensation code

Comment 23

[Mozilla RelEng Bot]

Try run for 1e5bb1ff62c6 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1e5bb1ff62c6
Results (out of 16 total builds):
    success: 15
    warnings: 1
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-1e5bb1ff62c6

Comment 24

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Comment on attachment 704700 [details] [diff] [review]
Remove extra scroll compensation code

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 811950
User impact if declined: Everything broken except the browser
Testing completed: Tested marketplace.mozilla.org both hosted and in browser. Tested nytimes.com and nzherald.co.nz.
Risk to taking this patch (and alternatives if risky): May make other layer positioning bugs more obvious.
String or UUID changes made by this patch: None

Comment 25

[Jason Smith [:jsmith]]

Aren't we able to land tef+ bugs automatically without approval?

Comment 26

[Mozilla RelEng Bot]

Try run for 8092ef4f25f6 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=8092ef4f25f6
Results (out of 81 total builds):
    exception: 1
    success: 73
    warnings: 7
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-8092ef4f25f6

Comment 28

[John Hopkins (:jhopkins)]

I ran into this scrolling issue with the Gasbuddy app.  Here's a video of my experience: http://people.mozilla.org/~jhopkins/bug831973/

Software: Boot2Gecko 1.0.0-prerelease, Last updated: 1/21/2013 2:08PM

Comment 29

[Andrew Overholt [:overholt]]

Comment on attachment 704700 [details] [diff] [review]
Remove extra scroll compensation code

tef+ bugs don't need approval for landing on b2g18 so I'm clearing that request.

Johnny pointed out that this patch does have this oddity:

-+                                                 &treeTransform);
+                                                 &treeTransform);

Comment 33

[Dietrich Ayala (:dietrich)]

Two additional testcases not mentioned in the summary, and filed in duped bugs:

* Persona native login panel (bug 833554)
* Marketplace itself (bug 833558)

Comment 34

[Daniel Holbert [:dholbert]]

Also:
* Wallpaper picker (bug 833053)

Comment 35

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

(In reply to Andrew Overholt [:overholt] from comment #29)
> -+                                                 &treeTransform);
> +                                                 &treeTransform);

The extra + was inadvertently added when rebasing bug 811950 https://hg.mozilla.org/releases/mozilla-b2g18/rev/d43f324b618b but is not significant.

Comment 36

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e327360a74db

Comment 37

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/e3803e97aeff

Comment 38

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e327360a74db

Comment 39

[Jason Smith [:jsmith]]

Verified on 1/23 build by testing out marketplace and persona flows - not seeing the white scrolling content issue now.

Comment 40

[David Lawrence [:dkl]]

(In reply to Jason Smith [:jsmith] from comment #39)
> Verified on 1/23 build by testing out marketplace and persona flows - not
> seeing the white scrolling content issue now.

Is there a link to a flashable image of 1/23 containing this fix that I may try? Or will it make it to a new beta (dogfooding) OTA update real soon? 

Thanks
dkl

Comment 42

[Lukas Blakk [:lsblakk] use ?needinfo]

Landed on mozilla-b2g18/gaia master prior to the 1/25 branching to mozilla-b2g18_v1_0_0/v1.0.0, updating status-b2g-v1.0.0 to fixed.