Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop shutdown to a ref-counted object, so that users don't have to worry about message loop shutdown.

BUG=604762
Committed: https://crrev.com/1714061d91fe5ec9e20506a76e05e7681e68ad8d
Cr-Commit-Position: refs/heads/master@{#388333}

[yzshen1, 2016-04-19 00:01:17]

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

[yzshen1, 2016-04-19 00:01:18]

Hi, Ken.

Please take a look. Thanks!

[yzshen1, 2016-04-19 16:05:51]

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

[commit-bot: I haz the power, 2016-04-19 16:05:59]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1900953002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1900953002/1

[yzshen1, 2016-04-19 16:06:18]

The CQ bit was unchecked by yzshen@chromium.org

[yzshen1, 2016-04-19 16:06:34]

Description was changed from

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=
==========

to

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=604762
==========

[Ken Rockot(use gerrit already), 2016-04-19 16:12:52]

lgtm

[yzshen1, 2016-04-19 16:17:29]

The CQ bit was checked by yzshen@chromium.org

[yzshen1, 2016-04-19 16:17:30]

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

[commit-bot: I haz the power, 2016-04-19 16:17:47]

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

[commit-bot: I haz the power, 2016-04-19 18:16:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-19 18:16:43]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[yzshen1, 2016-04-19 18:27:44]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-04-19 18:28:10]

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

[commit-bot: I haz the power, 2016-04-19 20:38:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-19 20:38:06]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[yzshen1, 2016-04-19 20:42:43]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-04-19 20:43:18]

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

[commit-bot: I haz the power, 2016-04-19 21:22:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-19 21:22:14]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[yzshen1, 2016-04-19 21:26:50]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-04-19 21:27:11]

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

[commit-bot: I haz the power, 2016-04-19 22:37:02]

Description was changed from

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=604762
==========

to

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=604762
==========

[commit-bot: I haz the power, 2016-04-19 22:37:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:17:15]

Description was changed from

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=604762
==========

to

==========
Mojo C++ bindings: make SyncHandleRegistry a ref-counted object.

This CL changes SyncHandleRegistry from an object destroyed during message loop
shutdown to a ref-counted object, so that users don't have to worry about
message loop shutdown.

BUG=604762

Committed: https://crrev.com/1714061d91fe5ec9e20506a76e05e7681e68ad8d
Cr-Commit-Position: refs/heads/master@{#388333}
==========

[commit-bot: I haz the power, 2016-04-22 19:17:16]

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