Closed Bug 930848 Opened 11 years ago Closed 11 years ago

[BrowserAPI] Implement mozbrowserapplaunch event

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: alive, Assigned: alive)

References

Details

Currently we know an app is launched by webapps-launch event. (It was mozChromeEvent:webapps-launch in the past). It's better if we know who is the launching the app, with this we could establish relationships between apps. For example, homescreen is the launcher of most apps, so if an app is terminated in foreground we could just reopen the launcher(homescreen app). For example, settings app is the launcher of keyboard settings app so if it's closed in foreground we could just reopen the launcher(settings app). If the launcher is system app we could just open the homescreen. A clear relationship map between all windows make us have a better management.
Taken.
Assignee: nobody → alive
Implementation proposal: 1. Remove 'webapps-launch' observer in shell.js 2. Add 'webapps-launch' observer in BrowserElementChildPreload 3. Try to figure out how to identify docShell or something like that in http://mxr.mozilla.org/mozilla-central/source/dom/apps/src/Webapps.jsm I guess the API call is also in child process so we won't hurt the launch time?
Summary: Implement mozbrowserapplaunch event → [BrowserAPI] Implement mozbrowserapplaunch event
Fabrice, let me know your thought to this idea. Thanks!
Flags: needinfo?(fabrice)
OK I find one problem: If we are going to implement "open the app from application list in settings app" then the parent of the new app would be settings, this doesn't make sense. :/
Yes, not all events make sense as mozbrowser events. In this case, we also have runtimes that don't use mozbrowser at all and that we need to support as well. I tend to think that this one should be a WONTFIX.
Flags: needinfo?(fabrice)
No longer blocks: task-manager
Make beta clear
No longer blocks: window-management
I am closing this one but we still have the use case in comment 0. Open another bug for that.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.