659405 - content page bleeds through Desktop Tab view

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Tony Chung [:tchung]]

Attachment: Desktop Tab screenshot

On the nightly that Gingerbread theme landed, the Desktop View seems to pass through the content page in the background, instead of loading Sync'd tabs. 

See screenshot

Repro:
1) install android nightly (must have gingerbread OS device--HTC Flyer, 2.3.3): Mozilla/5.0 (Android; Linux armv71; rv:6.0a1) Gecko/20110524 Firefox/6.0a1 Fennec/6.0a1
2) setup sync and connect
3) launch awesomescreen, click Desktop Tab
4) Verify the previous content page will bleed through the Desktop Tab view  (throbber included!)

Expected;
- sync'd tab view 

Actual:
- content page instead

Comment 1

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

Tony, is this a gingerbread only issue? Can you try it on a Android 2.2 (or less) phone?

Comment 2

[Tony Chung [:tchung]]

I've repro'd this on 2.3.3, 2.3.4, and 3.1 (xoom-Old Theme)

Comment 3

[Tony Chung [:tchung]]

Flagging for blocking fennec 6

Comment 4

[Tony Chung [:tchung]]

Attachment: Honeycomb theme desktop tab screenshot

screenshot for Xoom, running Android 3.1 (old theme)

Comment 5

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

Attachment: patch

the browser.css is out of sync and missing some new rules for the awesomebar binding. This likely happened because the two patches landed too closely to each other.

Hopefully, we have an easier time catching these problems.

Comment 6

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

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

Stealing the r? since I know this code for having wrote it :)

Comment 7

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

pushed:
http://hg.mozilla.org/mozilla-central/rev/32b009addadd

Comment 8

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

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

Needed for Fx6. The theme bustage is in aurora too.

Comment 10

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

http://hg.mozilla.org/releases/mozilla-aurora/rev/873ad7961640

Comment 11

[Tony Chung [:tchung]]

Verified fix on 6/13 trunk and 6/13 aurora