Remove unused base::Thread::thread_handle() getter.

Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716
Committed: https://crrev.com/8504bffa5efb122e7e5ac960b745a82b79efef39
Cr-Commit-Position: refs/heads/master@{#406831}

[gab, 2016-07-20 02:03:06]

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

[commit-bot: I haz the power, 2016-07-20 02:03:43]

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

[gab, 2016-07-20 02:04:43]

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

[commit-bot: I haz the power, 2016-07-20 02:04:56]

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

[gab, 2016-07-20 02:06:06]

Description was changed from

==========
Remove unused base::Thread::thread_handle() getter.

BUG=629716
==========

to

==========
Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716
==========

[gab, 2016-07-20 02:06:43]

gab@chromium.org changed reviewers:
+ danakj@chromium.org, hashimoto@chromium.org

[gab, 2016-07-20 02:06:44]

@dana for base/
@hashimoto for dbus/

Thanks,
Gab

[hashimoto, 2016-07-20 02:40:05]

lgtm

[commit-bot: I haz the power, 2016-07-20 03:38:08]

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

[commit-bot: I haz the power, 2016-07-20 03:38:09]

Dry run: This issue passed the CQ dry run.

[danakj, 2016-07-20 22:02:29]

LGTM

[gab, 2016-07-21 12:16:17]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-07-21 12:16:36]

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

[commit-bot: I haz the power, 2016-07-21 13:43:13]

Description was changed from

==========
Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716
==========

to

==========
Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716
==========

[commit-bot: I haz the power, 2016-07-21 13:43:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-21 13:44:33]

Description was changed from

==========
Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716
==========

to

==========
Remove unused base::Thread::thread_handle() getter.

Fix only user (dbus/test_server.cc) to use Thread on stack which is
automatically joined on destruction (and thus doesn't need the exposed
handle it was using to manually join).

BUG=629716

Committed: https://crrev.com/8504bffa5efb122e7e5ac960b745a82b79efef39
Cr-Commit-Position: refs/heads/master@{#406831}
==========

[commit-bot: I haz the power, 2016-07-21 13:44:34]

Patchset 1 (id:??) landed as
https://crrev.com/8504bffa5efb122e7e5ac960b745a82b79efef39
Cr-Commit-Position: refs/heads/master@{#406831}