824034 - [e.me] Please move e.me app into separate individual processes

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Everything.me, defect)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

## Environment :
Unagi phone, build 2012-12-21

I can't recall in which bug or where, but I recall e.me apps being stated that they weren't OOP and that they can take down the system?

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Reopened.  I found that each e.me app opened will still be in the same process. 

ie open facebook and google plus from e.me; they are all in the same process.  Would it be possible to move each of those apps into a separate child process?

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Yes.

However, the UX spec is that opening app A and then app B in succession from the e.me UI acts like "navigation"; the context created for A is then navigated to B.  So A is effectively closed.

Comment 4

[Evyatar 'Tron' Amitay (everything.me)]

so does this warrant a change, or does it behave correctly?

Comment 5

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Looks like the e.me was moved to a separate process.  Having said that all e.me apps operate under a single process.  Should we have individual e.me apps be pushed as separate processes on launch?

Comment 6

[Evyatar 'Tron' Amitay (everything.me)]

As far as I know it's by design that all apps open on the same process. Cristian?

Comment 7

[Cristian Rodriguez (:crdlc)]

Umm I can explain what we have currently:

1) Homescreen calls window.open bla bla with some parameters, url, app name, etc....
2) Window manager is listening to mozbrowseropenwindow

https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/window_manager.js#L1764

3) If the caller is the homescreen, we do this:

iframe = document.createElement('iframe');
iframe.setAttribute('mozbrowser', 'true');
iframe.setAttribute('remote', 'true');

I guess that it is a similar process to the rest of apps

Comment 8

[Cristian Rodriguez (:crdlc)]

Here the patch that was done by Poirot (bug 807438)

Comment 9

[Ran Ben Aharon [:ranbena] EverythingMe]

Cristian, Naoki, has Bug 807438 fixed this issue?

Comment 10

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I have some priority testing work today that I have to take care of... (APZ testing) I will try to find out ASAP after the APZ testing.

Comment 11

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Marking this as a duplicate.

Facebook seems to have a different quality : 

Facebook         app_1017  1017  137   79004  28428 ffffffff 400a8604 S /system/b2g/plugin-container
(Preallocated a  root      1241  137   64504  20776 ffffffff 40099604 S /system/b2g/plugin-container

I think there's still an issue where it's running as Preallocated as root, rather than having it's own app_id user.  I'll follow up with a separate bug.

Comment 12

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Actually I was mistaken. 

APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
b2g              root      136   1     220596 98192 ffffffff 400cd604 S /system/b2g/b2g
Homescreen       app_512   512   136   98468  33352 ffffffff 40111604 S /system/b2g/plugin-container
(Preallocated a  root      572   136   64504  15960 ffffffff 4005e604 S /system/b2g/plugin-container


APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
b2g              root      136   1     227696 100444 ffffffff 400cd810 R /system/b2g/b2g
Homescreen       app_512   512   136   110052 34632 ffffffff 41652f40 R /system/b2g/plugin-container
(Preallocated a  root      572   136   64504  15960 ffffffff 4005e604 R /system/b2g/plugin-container

It looks like it's the same process as the Homescreen.

Comment 13

[Ran Ben Aharon [:ranbena] EverythingMe]

Naoki, I think this should be attributed to Gaia:system. We have no control over it :/

Comment 14

[Amir Nissim (:amirn)]

No longer relevant. EverythingMe was re-written for vertical homescreen as a separate app.