Closed Bug 830824 Opened 10 years ago Closed 9 years ago

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

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: nhirata, Unassigned)

Details

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.  :)
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.
(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.
Oh, and FYI, that was bug 821710 that added this for crash reports.
No longer blocks: b2g-crash-reporting
Seems like we've covered here.  Reopen if not.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.