628660 - Ghost tabs appear when pressing return in a text box

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Ioana Chiorean]

Attachment: ghost tab

Device: HTC Desire A8181
BuildID: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b8pre) Gecko/20101021
Firefox/4.0b8pre Fennec /4.0b2pre  

Steps to reproduce:

1. go to http://people.mozilla.com/~tchung/html_tp/formsninputv2.html
2. scroll down to the text area box
3. click in the text area box and keep hitting return fast until the caret is at the bottom of the text area box.

Expected result:
- no ghost thumbnail appears and the caret will still be in focus

Actual result:
- ghost tabs appear and the page is covered by the light pink color

Note: 
- please see attached photo

Comment 1

[Ioana Chiorean]

please state if you need also a video

Comment 2

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

I cannot reproduce it in a build with the patch in bug 628732.

I see what you described as a light pink color (I will provide a fix for it) but no tab.

Comment 3

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

Attachment: Patch

Comment 4

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

Comment on attachment 507424 [details] [diff] [review]
Patch

Shouldn't the zoom code be doing this?

Comment 5

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

(In reply to comment #4)
> Comment on attachment 507424 [details] [diff] [review]
> Patch
> 
> Shouldn't the zoom code be doing this?

I was thinking that too but the zoom code take another path, it uses rootView._contentView.scrollTo instead of rootView.scroll (http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser.js#1038) to not update the displayport :/

Comment 6

[Benjamin Stover (:stechz)]

Zoom code is in fact calling updateCacheViewport when it sets browser.scale.

Comment 7

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

(In reply to comment #6)
> Zoom code is in fact calling updateCacheViewport when it sets browser.scale.

hmm, maybe there is a failure because the scale does not change and so updateCacheViewport is not called?
I will check that, thanks.

Comment 8

[Benjamin Stover (:stechz)]

Good point, scale will not update viewport in that case.

Comment 9

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

Attachment: Patch

Added a manual call to _updateCacheViewport in setVisibleRect if the scale level does not change

Comment 10

[Benjamin Stover (:stechz)]

Comment on attachment 507503 [details] [diff] [review]
Patch

Hacky of course, but nothing we can do right now.

Luckily, I think I will soon have a good api for this kind of thing!

Comment 11

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

http://hg.mozilla.org/mobile-browser/rev/07ebdaffee36

Comment 12

[Anna (Waverley)]

Build ID: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b11pre) Gecko/20110128 Firefox/4.0b11pre Fennec /4.0b5pre 

No ghost tab can be seen, but that light pink color, which looks now like a checkerboard is still visible to the user. (please see the attached photo).

As a result of the above mentioned, I will not mark this bug as being verified fixed.

Comment 13

[Anna (Waverley)]

Attachment: checkerboard

Comment 14

[Benjamin Stover (:stechz)]

Reopening. Vivien, can you reproduce?

Comment 15

[Ioana Chiorean]

Attachment: ghost tabs appear also when loading a page

Build Identifier: Mozilla/5.0 (Android; Linux armv7l; rv:2.0b12pre)
Gecko/20110202 Firefox/4.0b12pre Fennec/4.0b5pre 
Device HTC Desire A8181
OS: Android 2.1

Steps to reproduce:
1. Open several tabs 
2. Open a page from All pages from Awesome list

Comment 16

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

(In reply to comment #15)
> Created attachment 509401 [details]
> ghost tabs appear also when loading a page
> 
> Build Identifier: Mozilla/5.0 (Android; Linux armv7l; rv:2.0b12pre)
> Gecko/20110202 Firefox/4.0b12pre Fennec/4.0b5pre 
> Device HTC Desire A8181
> OS: Android 2.1
> 
> Steps to reproduce:
> 1. Open several tabs 
> 2. Open a page from All pages from Awesome list

I have not seen this in a while. Could we close it?

Comment 17

[BMO Automation]

Closing all opened bug in a graveyard component