Closed
Bug 995357
Opened 11 years ago
Closed 11 years ago
[Search] Handle OOMs and other browser events
Categories
(Firefox OS Graveyard :: Gaia::Search, defect)
Firefox OS Graveyard
Gaia::Search
Tracking
(Not tracked)
RESOLVED
FIXED
1.4 S6 (25apr)
People
(Reporter: kgrandon, Assigned: kgrandon)
References
Details
(Whiteboard: [p=2],[systemsfe])
Attachments
(1 file)
Bug 963991 implements SearchWindow but does not implement OOM or other mozbrowser events yet. This bug tracks the addition of those events.
|
Assignee | |
Updated•11 years ago
|
Target Milestone: --- → 1.4 S6 (25apr)
|
|
Assignee | |
Comment 1•11 years ago
|
||
|
|
Assignee | |
Comment 2•11 years ago
|
||
Comment on attachment 8405637 [details] [review]
Github pull request
Hey Alive - please review when you have the time. Thanks!
Attachment #8405637 -
Flags: review?(alive)
|
||
Comment 3•11 years ago
|
||
Comment on attachment 8405637 [details] [review]
Github pull request
Hmm..because you didn't override _handle_mozbrowsererror, I guess that means we are closing search app and returned to homescreen/current app?
Attachment #8405637 -
Flags: review?(alive) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
Yes - and I think that behavior should be fine for now? It seems we only return to homescreen when the app is active and gets killed. I think this is ok, and it doesn't seem that the app should be getting OOM killed when active too often? If I am wrong though we can easily override to do something different.
|
|
Assignee | |
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•