711232 - [viewport] Loading a page in a new tab shows multiple zoom animations until first screenful loads

Status: VERIFIED FIXED

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

Description

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

1. Start Fennec, wait for about:home to load
2. Open a new tab
3. In the new tab, load some page (e.g. yahoo.com)

Observed behaviour: there is a zoom animation from the top-left corner to the full visible viewport multiple times, often interleaved with checkerboarding and general spazzing out of the display before the first screenful of content is displayed.

Expected behaviour: no spaz.

Comment 1

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

Attachment: (1/2) Don't do unnecessary bounce animations

Comment 2

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

Attachment: (2/2) Use a reasonable page size before document.readyState = loaded

Comment 3

[Patrick Walton (:pcwalton)]

Comment on attachment 582086 [details] [diff] [review]
(1/2) Don't do unnecessary bounce animations

Review of attachment 582086 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

Comment 4

[Patrick Walton (:pcwalton)]

Comment on attachment 582087 [details] [diff] [review]
(2/2) Use a reasonable page size before document.readyState = loaded

Review of attachment 582087 [details] [diff] [review]:
-----------------------------------------------------------------

Death to 1x1. r=me

Comment 5

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

Landed on m-i:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9d027bf631d3
https://hg.mozilla.org/integration/mozilla-inbound/rev/de081ef87383

Comment 6

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/9d027bf631d3
https://hg.mozilla.org/mozilla-central/rev/de081ef87383

Comment 7

[Doug Turner (:dougt)]

backed out.

Comment 8

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

Re-landed on m-i after testing to verify it doesn't introduce a regression:

https://hg.mozilla.org/integration/mozilla-inbound/rev/e5daca9b7105
https://hg.mozilla.org/integration/mozilla-inbound/rev/dcd69858c2b3

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e5daca9b7105
https://hg.mozilla.org/mozilla-central/rev/dcd69858c2b3

Comment 11

[Brad Lassey [:blassey] (use needinfo?)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e5daca9b7105
https://hg.mozilla.org/releases/mozilla-aurora/rev/dcd69858c2b3

Comment 12

[Andreea Pod]

Verified fixed on Aurora: Mozilla /5.0 (Android;Linux armv7l;rv:11.0a2) Gecko/20120124 Firefox/11.0a2 Fennec/11.0a
Device: LG Optimus 2X (Android 2.2)