1156723 - Add "system-remote" entry to system app along with a simplified window manager

Status: RESOLVED FIXED

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

Description

[Luke Chang [:lchang] (inactive)]

For handling the remote desktop, we will create a new entry "system-remote", which can be launched by shell-remote, in system app. We will also create a simplified window manager to control the apps' life cycle in system-remote (because the original one in system app is overkill for our use case).

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] luke-chang:1156723_add_system_remote > mozilla-b2g:master

Comment 2

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8606150 [details] [review]
[gaia] luke-chang:1156723_add_system_remote > mozilla-b2g:master

Hi Alive,

Another patch needs your feedback. Thanks.

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8606150 [details] [review]
[gaia] luke-chang:1156723_add_system_remote > mozilla-b2g:master

I'd like to be sync with current system refactoring, so here is my proposal:

1. remote/bootstrap.js, nothing more then start app.js in window.onload
2. remote/app.js, base module, load WallpaperManager & WindowManager
3. remote/window_manager.js, base module

Comment 4

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8606150 [details] [review]
[gaia] luke-chang:1156723_add_system_remote > mozilla-b2g:master

I've changed my implementation into base module. Could you please help for reviewing it? Thanks.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8606150 [details] [review]
[gaia] luke-chang:1156723_add_system_remote > mozilla-b2g:master

r+ with nits

Comment 6

[Luke Chang [:lchang] (inactive)]

Comments are addressed. Thanks a lot.

Comment 7

[GitHub Autolander Bot]

http://docs.taskcluster.net/tools/task-graph-inspector/#QXzFyFhIRFigAiYxrmNQUg

The pull request failed to pass integration tests. It could not be landed, please try again.

Comment 8

[Luke Chang [:lchang] (inactive)]

Test passed: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=16d819adcbf3bd822b7deabc7a601475251d8583

Landed on master: https://github.com/mozilla-b2g/gaia/commit/ba233a8ade62d62edcea2e103ef26b4e36c002ef