Mojo C++ Bindings: Make SyncHandleRegistry Leaky

Mojo C++ Bindings: Make SyncHandleRegistry Leaky

This avoids a potential data race on the ThreadLocalPointer
if someone happens to leave a Mojo-using thread alive during
shutdown.

BUG=685157
R=yzshen@chromium.org

Review-Url: https://codereview.chromium.org/2658583003
Cr-Commit-Position: refs/heads/master@{#446080}
Committed: https://chromium.googlesource.com/chromium/src/+/8499f3a55d732d50c2a743fc3a499c43cc086234

[Ken Rockot(use gerrit already), 2017-01-25 17:56:14]

The CQ bit was checked by rockot@chromium.org to run a CQ dry run

[Ken Rockot(use gerrit already), 2017-01-25 17:56:30]

PTAL!

[commit-bot: I haz the power, 2017-01-25 17:56:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yzshen1, 2017-01-25 17:59:37]

On 2017/01/25 17:59:30, yzshen1 wrote:

LGTM!

[Ken Rockot(use gerrit already), 2017-01-25 18:08:26]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-01-25 18:08:27]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2017-01-25 18:08:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ken Rockot(use gerrit already), 2017-01-25 18:36:40]

Description was changed from

==========
Mojo C++ Bindings: Make SyncHandleRegistry Leaky

This avoids a potential data race on the ThreadLocalPointer's
LazyInstance if someone happens to leave a Mojo-using thread
alive during shutdown.

BUG=685157
R=yzshen@chromium.org
==========

to

==========
Mojo C++ Bindings: Make SyncHandleRegistry Leaky

This avoids a potential data race on the ThreadLocalPointer
if someone happens to leave a Mojo-using thread alive during
shutdown.

BUG=685157
R=yzshen@chromium.org
==========

[commit-bot: I haz the power, 2017-01-25 19:22:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485367707722230, "parent_rev":
"d8754536ca1e2f4fa65166585517193810515b7b", "commit_rev":
"8499f3a55d732d50c2a743fc3a499c43cc086234"}

[commit-bot: I haz the power, 2017-01-25 19:23:36]

Description was changed from

==========
Mojo C++ Bindings: Make SyncHandleRegistry Leaky

This avoids a potential data race on the ThreadLocalPointer
if someone happens to leave a Mojo-using thread alive during
shutdown.

BUG=685157
R=yzshen@chromium.org
==========

to

==========
Mojo C++ Bindings: Make SyncHandleRegistry Leaky

This avoids a potential data race on the ThreadLocalPointer
if someone happens to leave a Mojo-using thread alive during
shutdown.

BUG=685157
R=yzshen@chromium.org

Review-Url: https://codereview.chromium.org/2658583003
Cr-Commit-Position: refs/heads/master@{#446080}
Committed:
https://chromium.googlesource.com/chromium/src/+/8499f3a55d732d50c2a743fc3a49...
==========

[commit-bot: I haz the power, 2017-01-25 19:23:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8499f3a55d732d50c2a743fc3a49...