[Mac] More descriptive process name for flapper.

[Mac] More descriptive process name for flapper.

Put plugin basename without extension in process name.  The
human-readable name used for NPAPI version will be a little more
complicated to land, so punting for now.

BUG=132244
TEST=Activity Monitor app shows "PepperFlashPlayer" under Process Name.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=143659

[Scott Hess - ex-Googler, 2012-06-13 19:34:37]

content::PepperPluginInfo is manually created up chrome_content_client.cc, and
AFAICT only in the browser.  My best guess would be to pass the name with
PpapiMsg_LoadPlugin , which seems intrusive for something which is only used on
OSX.

[stuartmorgan, 2012-06-18 09:38:41]

LGTM as an interim solution

[Scott Hess - ex-Googler, 2012-06-18 16:59:36]

Brett, I think you are the intersection of OWNERS for these files.

[brettw, 2012-06-18 17:03:03]

lgtm

[commit-bot: I haz the power, 2012-06-18 17:04:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/10548031/1

[commit-bot: I haz the power, 2012-06-18 17:04:46]

Presubmit check for 10548031-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/app

[Scott Hess - ex-Googler, 2012-06-18 17:29:12]

Darn you noparent!  Sorry Brett.

jam, need an owners LGTM for the content_main_runner.cc change.

[jam, 2012-06-18 18:22:54]

lgtm
the OWNERS in content/app shouldn't have set noparent. i'll fix that.

[commit-bot: I haz the power, 2012-06-18 18:25:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/10548031/1

[commit-bot: I haz the power, 2012-06-18 19:11:50]

Try job failure for 10548031-1 (retry) on linux_rel for steps "browser_tests,
check_deps".
It's a second try, previously, steps "browser_tests, check_deps" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Scott Hess - ex-Googler, 2012-06-18 23:37:23]

Oops, forgot about this one.  John, any concerns about the DEPS change?  Seems
like alright to me...

[Scott Hess - ex-Googler, 2012-06-19 22:00:09]

John, WDYT of the DEPS change?

[Scott Hess - ex-Googler, 2012-06-20 22:46:42]

On 2012/06/19 22:00:09, shess wrote:
> John, WDYT of the DEPS change?

jam, DEPS ping.

[jam, 2012-06-20 22:48:58]

On 2012/06/20 22:46:42, shess wrote:
> On 2012/06/19 22:00:09, shess wrote:
> > John, WDYT of the DEPS change?
> 
> jam, DEPS ping.

lgtm. if in the future we need more specialized ContentClient interfaces for
plugin/ppapi, then we can split this off

[commit-bot: I haz the power, 2012-06-20 22:52:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/10548031/3003

[commit-bot: I haz the power, 2012-06-20 23:48:31]

Try job failure for 10548031-3003 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-06-21 19:19:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/10548031/3003

[commit-bot: I haz the power, 2012-06-21 20:23:31]

Try job failure for 10548031-3003 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Scott Hess - ex-Googler, 2012-06-22 00:25:59]

Brett,

When running in-process, it's not a plugin process, so things aren't setup.  So
I think this is a reasonable "solution".  But it's gross.  And I'm not
completely convinced I understand what's going on.

Previous code was breaking on linux_rel at PPAPITest.Broker because plugin() was
null at the point I just changed.

Thanks,
scott

[brettw, 2012-06-22 17:00:45]

lgtm

[commit-bot: I haz the power, 2012-06-22 17:03:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/10548031/35003

[commit-bot: I haz the power, 2012-06-22 19:02:44]

Change committed as 143659