Update PlatformThread to support a couple of new properties.

Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is currently stalled)

Committed: https://crrev.com/845afa8a0212200469e7a11f4fe45ba4a22a36d2
Cr-Commit-Position: refs/heads/master@{#12472}

[tommi, 2016-04-22 13:11:14]

tommi@webrtc.org changed reviewers:
+ perkj@webrtc.org

[tommi, 2016-04-22 13:11:22]

The CQ bit was checked by tommi@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-22 13:11:32]

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

[perkj_webrtc, 2016-04-22 13:32:20]

lgtm with a request

https://codereview.webrtc.org/1908373002/diff/1/webrtc/base/platform_thread_u...
File webrtc/base/platform_thread_unittest.cc (right):

https://codereview.webrtc.org/1908373002/diff/1/webrtc/base/platform_thread_u...
webrtc/base/platform_thread_unittest.cc:42: }
To show the usage- can you add another test with two threads that check
thread1.GetThreadRef() != thread2.GetThreadRef()

[tommi, 2016-04-22 14:46:27]

Add one more test

[tommi, 2016-04-22 14:47:12]

https://codereview.webrtc.org/1908373002/diff/1/webrtc/base/platform_thread_u...
File webrtc/base/platform_thread_unittest.cc (right):

https://codereview.webrtc.org/1908373002/diff/1/webrtc/base/platform_thread_u...
webrtc/base/platform_thread_unittest.cc:42: }
On 2016/04/22 13:32:20, perkj_webrtc wrote:
> To show the usage- can you add another test with two threads that check
> thread1.GetThreadRef() != thread2.GetThreadRef()

Done.

[tommi, 2016-04-22 14:47:19]

The CQ bit was checked by tommi@webrtc.org

[tommi, 2016-04-22 14:47:19]

The patchset sent to the CQ was uploaded after l-g-t-m from perkj@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1908373002/#ps20001 (title:
"Add one more test")

[commit-bot: I haz the power, 2016-04-22 14:47:26]

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

[tommi, 2016-04-22 16:06:13]

Description was changed from

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.
==========

to

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is
currently stalled)
==========

[tommi, 2016-04-22 16:06:18]

The CQ bit was unchecked by tommi@webrtc.org

[tommi, 2016-04-22 16:06:43]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2016-04-22 16:06:49]

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

[commit-bot: I haz the power, 2016-04-22 16:08:48]

Description was changed from

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is
currently stalled)
==========

to

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is
currently stalled)
==========

[commit-bot: I haz the power, 2016-04-22 16:08:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 16:38:21]

Description was changed from

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is
currently stalled)
==========

to

==========
Update PlatformThread to support a couple of new properties.
- name(): Returns the name of the thread (useful for debugging).
- GetThreadRef(): Returns the ID of the worker thread (also useful for
debugging).
- QueueApc(): This is a Windows-only, protected function that allows derived
classes on Windows to queue APCs to the worker thread.

NOTRY=true
(using notry since the patch has already passed the trybots and the cq is
currently stalled)

Committed: https://crrev.com/845afa8a0212200469e7a11f4fe45ba4a22a36d2
Cr-Commit-Position: refs/heads/master@{#12472}
==========

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

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