Closed Bug 1041467 Opened 5 years ago Closed 5 years ago

[MTBF] App only covers top half of the screen

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:2.0+, b2g-v2.0 fixed, b2g-v2.1 fixed)

VERIFIED FIXED
2.1 S1 (1aug)
blocking-b2g 2.0+
Tracking Status
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: pyang, Assigned: gduan)

References

Details

(Whiteboard: [p=2])

Attachments

(6 files, 2 obsolete files)

Attached image photo1.JPG
Device: flame
Memory: 273MB

Gaia      8cb1a949f2e9650bb2c5598e78a6f24a58bbaf97
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/c6f9f0396153
BuildID   20140720160208
Version   32.0a2
ro.build.version.incremental=109
ro.build.date=Mon Jun 16 16:51:29 CST 2014

STR: 
Setup MTBF test
Test suite including keyboard, sms, music
Randomly run automation test cases for about 2~6 hours.

EXPECT: Homescreen can be showed correctly

ACTUAL: foreground app was still active as figure.  System tray still worked.
Homescreen scrollable but can't launch other app.
Attached image photo2.JPG
Attached file full_debug_log.zip (obsolete) —
Component: General → Gaia::System::Window Mgmt
Flags: needinfo?(alive)
Phone is on you desk with log enabled.
Some thoughts:
* Keyboard seems being hidden/removed for some reason but it shouldn't be (in process).
* AppWindowManager actvie app is home when this bug occurs.

Not sure what happens but deserves investigating. Lemme know if you have time to investigate. Not a blocker now but if this happens in 319MB we will need to fix.
Flags: needinfo?(alive) → needinfo?(gduan)
Blocks: MTBF-B2G
Attached file output.zip (obsolete) —
update debug log
Attachment #8459505 - Attachment is obsolete: true
(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #3)
> Phone is on you desk with log enabled.
> Some thoughts:
> * Keyboard seems being hidden/removed for some reason but it shouldn't be
> (in process).
> * AppWindowManager actvie app is home when this bug occurs.
> 
> Not sure what happens but deserves investigating. Lemme know if you have
> time to investigate. Not a blocker now but if this happens in 319MB we will
> need to fix.

It should be a blocker since we can see it on even 512MB device, but not often.
Strongly recommend to fix because it make test and real user blocked, and need to reboot eventually.
Attached file last_logcat
Attach post test run logcat
Attached file output.zip
Update logcat
Attachment #8462481 - Attachment is obsolete: true
(In reply to Paul Yang [: pyang] from comment #5)
> (In reply to Alive Kuo [:alive][NEEDINFO!] from comment #3)
> > Phone is on you desk with log enabled.
> > Some thoughts:
> > * Keyboard seems being hidden/removed for some reason but it shouldn't be
> > (in process).
> > * AppWindowManager actvie app is home when this bug occurs.
> > 
> > Not sure what happens but deserves investigating. Lemme know if you have
> > time to investigate. Not a blocker now but if this happens in 319MB we will
> > need to fix.
> 
> It should be a blocker since we can see it on even 512MB device, but not
> often.
> Strongly recommend to fix because it make test and real user blocked, and
> need to reboot eventually.

Paul - If you think this is a blocker, can you nom it for 2.0?
Flags: needinfo?(pyang)
(In reply to Paul Yang [: pyang] from comment #8)
> Hang at 17:11
> 48 minutes in the video
> https://vreplay.mozilla.com/replay/showRecordingExternal.
> html?key=kcf4uB9BAUOQU13
this video at 15:11 looks like, homescreen is already launched, but takes longer time to load all icons.
It may not be mgmt's issue.
I found an easy STR,

1. launch keyboard (I tested in uitest app)
2. press home and |adb shell kill HOME_PROCESS_ID|
Flags: needinfo?(gduan)
[Blocking Requested - why for this release]: block v2.0 mtbf test currently
blocking-b2g: --- → 2.0?
Flags: needinfo?(pyang)
Attached file PR to 2.0
Hi Alive,
this patch should fix the bug as I described in my STR.
I think we've already _updateActiveApp in switchApp, so we don't need to do it in display.
Attachment #8463734 - Flags: review?(alive)
Attached file PR to master
PR to master
same issue on master.
Attachment #8463734 - Attachment is obsolete: true
Attachment #8463734 - Flags: review?(alive)
Attachment #8463771 - Flags: review?(alive)
Comment on attachment 8463771 [details] [review]
PR to master

r+ with nits on github, please pay attention before merging. Thanks for digging this.

The other way to fix this.element null issue might be using app suspending mechanism in homescreenWindow by default then we will always have this.element. That is bug 1039203.
Attachment #8463771 - Flags: review?(alive) → review+
Assignee: nobody → gduan
Whiteboard: [p=2]
Target Milestone: --- → 2.1 S1 (1aug)
blocking-b2g: 2.0? → 2.0+
Attachment #8463734 - Attachment is obsolete: false
Thanks Alive,
I still keep original solution but I will dig into bug 1039203.

master, https://github.com/mozilla-b2g/gaia/commit/cc5af5e8130ad81fda59b49d8213fbd67e1a0ddd
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
This still happen in 20140728 pvt build as I ran through it with 319MB. I will make sure it is okay next build run.
Flags: needinfo?(wachen)
I haven't seen any similiar issue in this round of MTBF (8 devices). I will temporarily marked it as verified fixed. thanks.
Status: RESOLVED → VERIFIED
Flags: needinfo?(wachen)
Blocks: 1047645
Blocks: 1054011
No longer blocks: 1054011
You need to log in before you can comment on or make changes to this bug.