984127 - [Sora][Browser]there is a black area when you re-enter the browser by "I'm thinking of ……"

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect, P1)

Description

[sync-1]

Mozilla BuildID 20140226004002
 
 Created an attachment (id=663699)
 VEDIO
 
 DEFECT DESCRIPTION:
 when I re-enter the browser by an input screen,the lower half of the browser interface is black  
 REPRODUCING PROCEDURES:
 1.lanuch "WWW.sina.com.cn"
 2.press home key to desktop
 3.go into "I'm thinking of……",search “Browser”
 4.click browser in searching result
 5.look at the interface,the  lower half of the screen is black ---->KO
 note: phenomenon of Beetle lite FF and soul3.5 FF is NOT the same,there is no black interface in beetle lite FF 
 EXPECTED BEHAVIOUR: 
 there is no black interface after clicking the browser in searching result 
 ASSOCIATE SPECIFICATION:
 TEST PLAN REFERENCE:
 TOOLS AND PLATFORMS USED:
 USER IMPACT:
 
 REPRODUCING RATE:
 100%
 For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Attachment: VEDIO

Comment 2

[Jason Smith [:jsmith]]

Can someone confirm this on the Moz side on 1.3?

Comment 3

[Jack Liu]

I think it's blocking issue with high reproduce rate and I found a way to reproduce it easily:

1. on e.me page, search any app that with input field like message/contact/email/browser/calendar/music.
2. enter the app by tap app icon from search result.
3. open a page in the app with input field and tap on it to call up keyboard.
4. press home key back to e.me.
5. tap on search bar on e.me page to call up keyboard.
6. enter the app again by tap app icon from search result.
7. will find black screen at the area of keyboard.

Comment 4

[Peter Bylenga [:PBylenga]]

Attachment: Buri_v13_Browser_Bottom_black_Area.png

On the latest Buri v1.3 I am able to reproduce the issue using the STRs in Comment 0.  However, I am unable to reproduce the issue using the STRs in Comment 3.  Returning to the Message app does not reproduce the black area issue, keyboard is disabled and layout is as expected.

v1.3 Environmental Variables:
Device: Buri v1.3 MOZ
BuildID: 20140317004001
Gaia: 0ab8a9cbcef5f23cec904a3d7f7675e44de29951
Gecko: f824e9d91a2d
Version: 28.0
Firmware Version: v1.2-device.cfg

Comment 5

[Peter Bylenga [:PBylenga]]

The Regression Window is 1-20-2014 to 1-21-2014 which falls outside of the date range for available Tinderbox and Inbound builds.  The gecko/gaia swap test reveals this is a Gaia issue.  Also this issue occurs both with and without APZ enabled.

Last Working B2G-RIL build:
v1.3 Environmental Variables:
Device: Buri v1.3 MOZ
BuildID: 20140120004002
Gaia: 63cc8ffd6f2acdce14eb8285e8822ebe62fcab41
Gecko: 37ec3c7fb511
Version: 28.0a2
Firmware Version: v1.2-device.cfg

First Broken B2G-RIL build:
v1.3 Environmental Variables:
Device: Buri v1.3 MOZ
BuildID: 20140121004137
Gaia: 47049555282a9a01fb60d1e1421b57e2810c96f5
Gecko: 6f7dfe36ab6c
Version: 28.0a2
Firmware Version: v1.2-device.cfg


Last Working Gaia / First Broken Gecko: Issue Does NOT reproduce
Gaia: 63cc8ffd6f2acdce14eb8285e8822ebe62fcab41
Gecko: 6f7dfe36ab6c

Last Working Gecko / First Broken Gaia: Issue DOES reproduce
Gaia: 47049555282a9a01fb60d1e1421b57e2810c96f5
Gecko: 37ec3c7fb511

Pushlog:
https://github.com/mozilla-b2g/gaia/compare/63cc8ffd6f2acdce14eb8285e8822ebe62fcab41...47049555282a9a01fb60d1e1421b57e2810c96f5

Comment 6

[Jason Smith [:jsmith]]

This is broken by bug 958224.

Comment 7

[Preeti Raghunath(:Preeti)]

Please help with fix for the same. Made 1.3+

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/17307

Let's do the same thing as v1.4 app_window_manager does.

Comment 10

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8393301 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/17307

Please make sure calling |KeyboardManager.hideKeyboardImmediately();| without a showing keyboard does not result any bad side effect.

Comment 11

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8393301 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/17307

Low risk patch to hide keyboard on app launching.

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined:
[Testing completed]:
[Risk to taking this patch] (and alternatives if risky):
[String changes made]:

Comment 12

[bhavana bajaj [:bajaj]]

(In reply to Alive Kuo [:alive][NEEDINFO!][:艾莉芙] from comment #11)
> Comment on attachment 8393301 [details] [review]
> https://github.com/mozilla-b2g/gaia/pull/17307
> 
> Low risk patch to hide keyboard on app launching.
> 
> NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to
> better understand the B2G approval process and landings.
> 
> [Approval Request Comment]
> [Bug caused by] (feature/regressing bug #):
> [User impact] if declined:
> [Testing completed]:
> [Risk to taking this patch] (and alternatives if risky):
> [String changes made]:

:alive, can you please help with more details request in the above comment ?

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to bhavana bajaj [:bajaj] from comment #12)
> (In reply to Alive Kuo [:alive][NEEDINFO!][:艾莉芙] from comment #11)
> > Comment on attachment 8393301 [details] [review]
> > https://github.com/mozilla-b2g/gaia/pull/17307
> > 
> > Low risk patch to hide keyboard on app launching.
> > 
> > NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to
> > better understand the B2G approval process and landings.
> > 
> > [Approval Request Comment]
> > [Bug caused by] (feature/regressing bug #):
> > [User impact] if declined:
> > [Testing completed]:
> > [Risk to taking this patch] (and alternatives if risky):
> > [String changes made]:
> 
> :alive, can you please help with more details request in the above comment ?

I don't know how to say but only technical detail.

The issue is that the browser app is resized by the keyboard once, and if the user is launching the ever-resized app on keyboard is shown we will resize again which is wrong.
The solution is there in v1.4 from bug 907013, we will hide the keyboard on each app launching request. And since v1.4/v1.5 doesn't have this issue and no other regressions had been found I think it's safe to introduce the same thing to v1.3 and the modification is little.

Comment 14

[bhavana bajaj [:bajaj]]

Comment on attachment 8393301 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/17307

Thanks for the comments alive. Lets get this landed and a parallel verification on 1.3 by QA/Reporter

Comment 15

[bhavana bajaj [:bajaj]]

(In reply to Jack Liu from comment #3)
> I think it's blocking issue with high reproduce rate and I found a way to
> reproduce it easily:
> 
> 1. on e.me page, search any app that with input field like
> message/contact/email/browser/calendar/music.
> 2. enter the app by tap app icon from search result.
> 3. open a page in the app with input field and tap on it to call up keyboard.
> 4. press home key back to e.me.
> 5. tap on search bar on e.me page to call up keyboard.
> 6. enter the app again by tap app icon from search result.
> 7. will find black screen at the area of keyboard.

HI Jack,

Can you please confirm this is fixed in the latest build once this lands ?

Comment 16

[(inactive after 6/18) Alive Kuo [:alive]]

v1.3 landed.
https://github.com/mozilla-b2g/gaia/commit/f1c58f9f3b5d538afdd71ad3c22106200bc7ece5

Comment 17

[Roland Kunkel [:RolandK]]

Verified as fixed on the latest v1.3 Buri build.

3/25 Environmental Variables:
Device: Buri 1.3 MOZ RIL
BuildID: 20140325004002
Gaia: b789780c53adaac199787f51615375f721edfef4
Gecko: 37917eb0dfeb
Version: 28.0
Firmware Version: V1.2-device.cfg

Comment 18

[Roland Kunkel [:RolandK]]

Leaving verifyme keyword because this issue still needs to be checked on the v1.3T branch.

Comment 19

[pete siphantong [:Petes]]

Verified on the latest Tarako build

1.3T Environmental Variables:
Device: Tarako 1.3T
BuildID: 20140530014002
Gaia: e68858693b71d917c9c5ee7e215f7ceea04635f7
Gecko: 1945abae19ff
Version: 28.1
Firmware Version: SP6821a-Gonk-4.0-4-29

1.3 tarako: no black interface on the lower half of sina.com

Comment 20

[Paladin]

According to comment #18 and comment #19, Clear Verifyme keyword.