Moves initialization of exception handler registration

Moves initialization of exception handler registration

Registering exception handlers is per-process, not per thread. The old
code was doing the registration per mojo-app, which is per
thread. This is problematic for a couple of reasons, but primarily
because an app may be unloaded, which results in the system having an
exception handler pointing to a now invalid location.

The fix is to move registration to the place where we launch
applications, which happens only once per process.

BUG=526245
TEST=none
R=yzshen1@chromium.org

Committed: https://crrev.com/03c235bfc0ff5db46f82463fa471bc8ad5236053
Cr-Commit-Position: refs/heads/master@{#346240}

[yzshen1, 2015-08-28 19:43:21]

yzshen@chromium.org changed reviewers:
+ yzshen@chromium.org

[yzshen1, 2015-08-28 19:43:22]

LGTM with one nit

https://codereview.chromium.org/1317333003/diff/1/mojo/runner/desktop/main.cc
File mojo/runner/desktop/main.cc (right):

https://codereview.chromium.org/1317333003/diff/1/mojo/runner/desktop/main.cc...
mojo/runner/desktop/main.cc:21: 
Please consider including build/build_config.h.

[sky, 2015-08-28 19:53:01]

https://codereview.chromium.org/1317333003/diff/1/mojo/runner/desktop/main.cc
File mojo/runner/desktop/main.cc (right):

https://codereview.chromium.org/1317333003/diff/1/mojo/runner/desktop/main.cc...
mojo/runner/desktop/main.cc:21: 
On 2015/08/28 19:43:22, yzshen1 wrote:
> Please consider including build/build_config.h.

Done.

[sky, 2015-08-28 19:53:03]

The CQ bit was checked by sky@chromium.org

[sky, 2015-08-28 19:53:04]

The patchset sent to the CQ was uploaded after l-g-t-m from yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1317333003/#ps20001
(title: "add include")

[commit-bot: I haz the power, 2015-08-28 19:53:16]

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

[sky, 2015-08-28 19:57:11]

The CQ bit was unchecked by sky@chromium.org

[sky, 2015-08-28 20:03:55]

The CQ bit was checked by sky@chromium.org

[sky, 2015-08-28 20:03:55]

The patchset sent to the CQ was uploaded after l-g-t-m from yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1317333003/#ps40001
(title: "update android")

[commit-bot: I haz the power, 2015-08-28 20:04:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1317333003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1317333003/40001

[sky, 2015-08-28 20:55:17]

The CQ bit was checked by sky@chromium.org

[sky, 2015-08-28 20:55:17]

The patchset sent to the CQ was uploaded after l-g-t-m from yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1317333003/#ps60001
(title: "add include")

[commit-bot: I haz the power, 2015-08-28 20:55:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1317333003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1317333003/60001

[commit-bot: I haz the power, 2015-08-28 21:46:54]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-08-28 21:47:39]

Patchset 4 (id:??) landed as
https://crrev.com/03c235bfc0ff5db46f82463fa471bc8ad5236053
Cr-Commit-Position: refs/heads/master@{#346240}