Closed Bug 942161 Opened 11 years ago Closed 11 years ago

We should keep OOM killed sheets in the StackManager

Categories

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

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 935750

People

(Reporter: etienne, Unassigned)

References

Details

And, instead of a screenshot, use the launching "icon on black background" display. Currently on some devices we never have more than 2-3 apps open at the same time, so the app to app edge gestures becomes almost useless. It also means that those "zoombie" sheets will be display in the CardView but we're fine with it. A power user wanting to differentiate between background sheets and zombie sheets can spot the screenshoted sheets vs. the "icon on black background" ones.
Alive, I can work on this one but feel free to "brain dump" the correct way this would fit in the new window manager.
blocking-b2g: --- → 1.3?
(In reply to Etienne Segonzac (:etienne) from comment #1) > Alive, I can work on this one but feel free to "brain dump" the correct way > this would fit in the new window manager. Interesting... First of all we're not able to know the app is killed by OOM now. We only know it's crashed. If we don't care it's terminated by crash or OOM, this feature shall not be difficult. 1. Don't remove appterminated event handler in AppWindowManager. 2. in AppWindow._handle_mozbrowsererror function, do not call appWindow.kill() if !app.isActive() (I think we don't want to do this for foreground app? I mean just let foreground app dies but not become zombie.) 3. per 2., remove the this.browser.element 4. Before app is opened, check this.browser.element, if it doesn't exist, call _render_browser(); (No such function yet) 5. Drain browser element rendering from _render() function to _render_browser. Use this.browser_config to gen the iframe. See how homescreenWindow restarts itself for example. 6. Do not forget to rebind mozbrowser events. Maybe call appWindow._registerEvents again.
(In reply to Etienne Segonzac (:etienne) from comment #0) > And, instead of a screenshot, use the launching "icon on black background" > display. I think if we remove the mozbrowser iframe from appWindow.element, the background would be splash-icon already. If not just call AppWindow.setFrameBackground again. > > Currently on some devices we never have more than 2-3 apps open at the same > time, so the app to app edge gestures becomes almost useless. > > It also means that those "zoombie" sheets will be display in the CardView > but we're fine with it. > > A power user wanting to differentiate between background sheets and zombie > sheets can spot the screenshoted sheets vs. the "icon on black background" > ones.
Edge gestures is being likely moved out of 1.3, so I'm pulling the nom. Will renom for 1.4 after this is confirmed.
blocking-b2g: 1.3? → ---
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.