689928 - White space in the left side of the tab bar

Status: VERIFIED DUPLICATE of bug 691541

(Firefox for Android Graveyard :: General, defect)

Description

[Andreea Pod]

Attachment: screenshot1

Mozilla /5.0 (Android;Linux armv7l;rv:10.0a1) Gecko/20110928 Firefox/10.0a1 Fennec/10.0a1
Device: LG Optimus 2X (Android 2.2)

Steps to reproduce:
1. Open Fennec with a new profile
2. Look at the left side of the screen while the first-run animation is running

Actual result:
you can see a white space from top to bottom (screenshot1)

Expected results:
no white spaces in the sidebars

Notes:
- the page is not rendered correctly in the left side also after you close and open the browser(screenshot2)
- reproducible on Aurora too: Mozilla /5.0 (Android;Linux armv7l;rv:9.0a2) Gecko/20110928 Firefox/9.0a2 Fennec/9.0a2

Comment 1

[Andreea Pod]

Attachment: screenshot2

Comment 2

[Aaron Train [:aaronmt]]

I don't see this on my Galaxy SII, nor a Nexus S. Can you trace a regression?

Comment 3

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

I see this on my Nexus One and Galaxy Tab (7")

My guess is it's either a regression from bug 678480 or bug 686417. Can we try nightlies to find the regression range?

Comment 4

[Matt Brubeck (:mbrubeck)]

I can't reproduce this in desktop Fennec, even after selecting the Froyo skin and setting layout.css.dpi=240 in prefs.js.

Comment 5

[Matt Brubeck (:mbrubeck)]

I also can't reproduce this on T-Mobile G2 (Android 2.3) running the latest trunk Nightly 10.0a1 (20110928).

Comment 6

[Matt Brubeck (:mbrubeck)]

I do see this on Samsung Verizon Galaxy Tab 7" (Android 2.2).

Comment 7

[Andreea Pod]

This started happening since bug 678480 was fixed.

Last build that wasn't with this problem:
http://hg.mozilla.org/mozilla-central/rev/259d1556c221

First build with this problem:
http://hg.mozilla.org/mozilla-central/rev/c71229984353

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=259d1556c221&tochange=c71229984353

Comment 9

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

Who can work on this? It's needed in Aurora ASAP.

Comment 10

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

Attachment: patch

This patch removes the fixed width/height in browser.xul causing less resizes. This fixes the content offset problem and might even help startup a tiny bit.

Test build is here:
http://people.mozilla.com/~mfinkle/fennec/fennec-10.a1-nofixedsize.apk

Comment 11

[Matt Brubeck (:mbrubeck)]

Comment on attachment 565647 [details] [diff] [review]
patch

Nominating for Aurora 9.  This is a low-risk mobile only change that fixes a user-visible regression in the first-run animation on Android 2.2 devices.  The change just removes a hard-coded size from our main XUL window so that it will not need to resize again during startup.  (The hard-coded size has always been useful only on desktop Fennec builds.)

Comment 12

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/194720ba054f

Comment 13

[Aaron Train [:aaronmt]]

Attachment: Nightly (10/11)

This is still visible on (10/11 Nightly); albeit the thin line is now grey.

Comment 14

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

The fix for this has landed and won't be backed out. It did fix the issues for personal builds. We are working through the issues with nightly builds in bug 691541.

Let's leave this bug and move to bug 691541.

Not sure if FIXED or DUPE is better for this one. Choosing DUPE for now.

Comment 15

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

Comment on attachment 565647 [details] [diff] [review]
patch

This hasn't actually fixed the problem on nightlies.