Introduce task_runner() accessors for both base::Thread and base::MessageLoop

Introduce task_runner() accessors for both base::Thread and base::MessageLoop

This is so that callers can code against a TaskRunner interface
directly, rather than converting message_loop_proxy() into a TaskRunner.

BUG=391045
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282767

[Ryan Sleevi, 2014-07-11 20:21:33]

To make cleanup easier for the messy parts

[awong, 2014-07-11 20:23:18]

just a nit

https://codereview.chromium.org/389653005/diff/20001/base/threading/thread.h
File base/threading/thread.h (right):

https://codereview.chromium.org/389653005/diff/20001/base/threading/thread.h#...
base/threading/thread.h:154: // methods to execute code on the thread. This only
returns non-NULL after
How about "Returns NULL if thread is not running?

[Ryan Sleevi, 2014-07-11 20:24:05]

On 2014/07/11 20:23:18, awong wrote:
> just a nit
> 
> https://codereview.chromium.org/389653005/diff/20001/base/threading/thread.h
> File base/threading/thread.h (right):
> 
>
https://codereview.chromium.org/389653005/diff/20001/base/threading/thread.h#...
> base/threading/thread.h:154: // methods to execute code on the thread. This
only
> returns non-NULL after
> How about "Returns NULL if thread is not running?

Was duplicating message_loop_proxy()'s documentation. Will update both.

[Ryan Sleevi, 2014-07-11 20:26:30]

Updated comments

[awong, 2014-07-11 20:30:05]

LGTM

[Ryan Sleevi, 2014-07-11 20:30:17]

The CQ bit was checked by rsleevi@chromium.org

[commit-bot: I haz the power, 2014-07-11 20:32:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rsleevi@chromium.org/389653005/40001

[commit-bot: I haz the power, 2014-07-12 01:11:00]

Change committed as 282767