Pepper: CallOnMainThread should do nothing if Globals are gone

Pepper: CallOnMainThread should do nothing if Globals are gone

BUG=339912
Committed: https://crrev.com/18a876a21927546da4868a3afaf78aa6fdd77caa
Cr-Commit-Position: refs/heads/master@{#322259}

[dmichael (off chromium), 2015-03-25 20:31:25]

dmichael@chromium.org changed reviewers:
+ bbudge@chromium.org

[bbudge, 2015-03-25 20:54:45]

lgtm

https://codereview.chromium.org/1036913002/diff/1/ppapi/proxy/ppb_core_proxy.cc
File ppapi/proxy/ppb_core_proxy.cc (right):

https://codereview.chromium.org/1036913002/diff/1/ppapi/proxy/ppb_core_proxy....
ppapi/proxy/ppb_core_proxy.cc:68: // at shutdown, it's possible that the globals
or the main loop has been
nit: s/has/have

[dmichael (off chromium), 2015-03-25 21:33:06]

thanks!

https://codereview.chromium.org/1036913002/diff/1/ppapi/proxy/ppb_core_proxy.cc
File ppapi/proxy/ppb_core_proxy.cc (right):

https://codereview.chromium.org/1036913002/diff/1/ppapi/proxy/ppb_core_proxy....
ppapi/proxy/ppb_core_proxy.cc:68: // at shutdown, it's possible that the globals
or the main loop has been
On 2015/03/25 20:54:45, bbudge wrote:
> nit: s/has/have
I was treating both parts of the "or" as singular. I tried to make it read a
little better.

[dmichael (off chromium), 2015-03-25 21:33:08]

The CQ bit was checked by dmichael@chromium.org

[dmichael (off chromium), 2015-03-25 21:33:08]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/1036913002/#ps20001
(title: "improve comment")

[commit-bot: I haz the power, 2015-03-25 21:34:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1036913002/20001

[commit-bot: I haz the power, 2015-03-25 23:28:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-25 23:29:20]

Patchset 2 (id:??) landed as
https://crrev.com/18a876a21927546da4868a3afaf78aa6fdd77caa
Cr-Commit-Position: refs/heads/master@{#322259}