Do not use set process names in the Activity Monitor, using Process Manager functions in the proces…

Do not set process names in the Activity Monitor.

This involves Process Manager functions, which in 10.9 cause a WindowServer checkin, which causes "not responding" notifications on any child process that doesn't pump events (which is everything but the renderers).

BUG=304860
TEST=Plugin process on Mavericks shouldn't have an app icon to its left, shouldn't be "not responding"
R=mark@chromium.org, thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=231108

[Avi (use Gerrit), 2013-10-25 19:46:08]

Mark for base, Nico for rest

[Nico, 2013-10-25 19:52:40]

lgtm

lebtm if the CL description said why this is happening (but it has a BUG= line
that explains it, so it's ok)

[Mark Mentovai, 2013-10-25 19:59:11]

https://codereview.chromium.org/45253002/diff/1/base/mac/mac_util.mm
File base/mac/mac_util.mm (left):

https://codereview.chromium.org/45253002/diff/1/base/mac/mac_util.mm#oldcode316
base/mac/mac_util.mm:316: // comment for 10.8.
OK, so we never did that…

https://codereview.chromium.org/45253002/diff/1/chrome/app/chromium_strings.grd
File chrome/app/chromium_strings.grd (left):

https://codereview.chromium.org/45253002/diff/1/chrome/app/chromium_strings.g...
chrome/app/chromium_strings.grd:873: </if>
Keep me.

https://codereview.chromium.org/45253002/diff/1/chrome/app/google_chrome_stri...
File chrome/app/google_chrome_strings.grd (left):

https://codereview.chromium.org/45253002/diff/1/chrome/app/google_chrome_stri...
chrome/app/google_chrome_strings.grd:797: </if>
Me too.

[Avi (use Gerrit), 2013-10-25 20:03:10]

ok

https://codereview.chromium.org/45253002/diff/1/base/mac/mac_util.mm
File base/mac/mac_util.mm (left):

https://codereview.chromium.org/45253002/diff/1/base/mac/mac_util.mm#oldcode316
base/mac/mac_util.mm:316: // comment for 10.8.
On 2013/10/25 19:59:11, Mark Mentovai wrote:
> OK, so we never did that…

Old file is old.

Take a look at the function directly below this, CopyNSImageToCGImage. Its
raison d'être is that it is a workaround for a problem in 10.5 that is gone in
10.6.

https://codereview.chromium.org/45253002/diff/1/chrome/app/chromium_strings.grd
File chrome/app/chromium_strings.grd (left):

https://codereview.chromium.org/45253002/diff/1/chrome/app/chromium_strings.g...
chrome/app/chromium_strings.grd:873: </if>
On 2013/10/25 19:59:11, Mark Mentovai wrote:
> Keep me.

Done.

[Mark Mentovai, 2013-10-25 20:38:12]

LGTM

[Avi (use Gerrit), 2013-10-25 20:47:08]

Committed patchset #2 manually as r231108 (presubmit successful).