Closed
Bug 1015812
Opened 10 years ago
Closed 10 years ago
[Monkey Test]After a few hours monkey test ,there is no icons on idle ,the iframe of homescreen is hidden while the div is active
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(b2g-v1.3T fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v1.3T | --- | fixed |
People
(Reporter: yang.zhao, Assigned: alive)
Details
(Whiteboard: [sprd316143])
Attachments
(3 files)
After a few hours monkey test ,there is no icons on the idle.
The homescreen app has been disabled OOP on v1.3t.
From the app-manager ,you could see that in system app ,the iframe of homescreen is hidden while the div is 'active'.
Here is the screenshot of the phone.
Flags: needinfo?(james.zhang)
Comment 2•10 years ago
|
||
After Fabrice disable home screen oop, we met home screen issue again.
Flags: needinfo?(tzhuang)
Flags: needinfo?(james.zhang)
Flags: needinfo?(fabrice)
Comment 3•10 years ago
|
||
Alive, any idea what could happen here?
Flags: needinfo?(fabrice) → needinfo?(alive)
Assignee | ||
Comment 4•10 years ago
|
||
I guess this is because
https://github.com/mozilla-b2g/gaia/blob/v1.3t/apps/system/js/window_manager.js#L1119
So disable oop makes homescreen background when some overlay event occurs but no one recovers.
Therefore this fix should be add app.isHomescreen checkage.
Flags: needinfo?(alive)
Assignee | ||
Comment 6•10 years ago
|
||
I could make a patch but needs someone's help to do the monkey test.
Assignee | ||
Comment 7•10 years ago
|
||
Do not send inproc homescreen to background while overlay event occurs.
Attachment #8429000 -
Flags: review?(timdream)
Attachment #8429000 -
Flags: feedback?(yangman)
(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #7)
> Created attachment 8429000 [details] [review]
> https://github.com/mozilla-b2g/gaia/pull/19660
>
> Do not send inproc homescreen to background while overlay event occurs.
Hi,Alive
I'll try it on our build and verify it for several days,if the patch works then let't merge it into v1.3t.Thank you.
NI myself for reminding me of telling the result.
Flags: needinfo?(yang.zhao)
Comment 9•10 years ago
|
||
Comment on attachment 8429000 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/19660
Add a comment on top of the changed line maybe?
Attachment #8429000 -
Flags: review?(timdream) → review+
Updated•10 years ago
|
Flags: needinfo?(tzhuang)
Comment 10•10 years ago
|
||
My email was apparently flagged for feedback in error. Please reflag to appropriate user; I'm not involved with this bug.
Attachment #8429000 -
Flags: feedback?(yangman) → feedback-
Attachment #8429000 -
Flags: feedback-
Reporter | ||
Comment 11•10 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO!][OOO until 6/4] from comment #7)
> Created attachment 8429000 [details] [review]
> https://github.com/mozilla-b2g/gaia/pull/19660
>
> Do not send inproc homescreen to background while overlay event occurs.
Hi,Alive
After land this patch on our build,it seems that this issue didn't happen .Could you help to merge it to v1.3t,I see the travis is failed in https://github.com/mozilla-b2g/gaia/pull/19660.Please also see comment #9 .Thank you very much.
Flags: needinfo?(yang.zhao)
Assignee | ||
Comment 12•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
status-b2g-v1.3T:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•