830824 - process /apps should have a specific name, not a generic name of "plugin-container"

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect)

Description

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

1. launch email
2. launch sms
3. do an adb shell ps

Expected: a process should be shown as email.app and sms.app or something similar
Actual: both email app and sms app show plugin-container

Note: 
1. we probably need a separate bug to have this as a app id that gets pushed to the crash reporting?
2. tabs are OOP for browser, the browser app itself runs in core.
3. Dhylands, I couldn't find the bug, so I reported it as you had asked in triage.  :)

Comment 1

[Dave Hylands [:dhylands]]

So it is expected behaviour that ps shows plugin-container (that's the way that android's ps works).

I wrote a script called b2g-ps which will show the app-name:

2117 >adb shell b2g-ps
APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
b2g              root      644   1     170068 57984 ffffffff 40062330 S /system/b2g/b2g
Homescreen       app_690   690   644   63880  23960 ffffffff 40101330 S /system/b2g/plugin-container
Calendar         app_747   747   644   64836  24916 ffffffff 4004c330 S /system/b2g/plugin-container
Clock            app_818   818   644   61752  22800 ffffffff 40044330 S /system/b2g/plugin-container
(Preallocated a  app_848   848   644   55480  17300 ffffffff 40105330 S /system/b2g/plugin-container

So we have the information in the process (ala /proc/PID/comm) and we really just need to pass it along to the crash reporter.

Comment 2

[Robert Kaiser]

(In reply to Naoki Hirata :nhirata from comment #0)
> 1. we probably need a separate bug to have this as a app id that gets pushed
> to the crash reporting?

We already get the app origin in the URL field of crash reports, so nothing needed to be done there. For crash reports, we're good and have been for a few weeks already.

Comment 3

[Robert Kaiser]

Oh, and FYI, that was bug 821710 that added this for crash reports.

Comment 4

[Jason Duell]

Seems like we've covered here.  Reopen if not.