[chrome/browser/policy] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in chrome/browser/policy module.

MessageLoopProxy is deprecated.
This basically does a search and replace of:

MessageLoopProxy::current() -> ThreadTaskRunnerHandle::Get().

MessageLoopProxy -> SingleThreadTaskRunner

BUG=391045
Committed: https://crrev.com/6825391d97429c8bcddf69aac7392d75ce6c481c
Cr-Commit-Position: refs/heads/master@{#327692}

[Pranay, 2015-04-29 12:49:00]

pranay.kumar@samsung.com changed reviewers:
+ pneubeck@chromium.org

[Pranay, 2015-04-29 12:49:00]

PTAL

Thanks

[pneubeck (no reviews), 2015-04-30 08:00:08]

lgtm with minor changes

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
File chrome/browser/policy/cloud/test_request_interceptor.cc (right):

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/test_request_interceptor.cc:290:
base::Bind(base::IgnoreResult(&base::MessageLoopProxy::PostTask),
this reads very inconsistent.
Shouldn't this be changed to
 ...IgnoreResult(&base::TaskRunner::PostTask)...
?

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/test_request_interceptor.cc:335:
base::IgnoreResult(&base::MessageLoopProxy::PostTask),
ditto.

now you should be able to remove the include
"base/message_loop/message_loop_proxy.h"
as well.

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
File chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc (right):

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc:6: #include
"base/message_loop/message_loop.h"
also not required anymore? (I can't find any reference to MessageLoop)

[Pranay, 2015-04-30 09:34:44]

Hi Pneubeck,

Thanks for the review. Have updated the changes in new patch set :)

[Pranay, 2015-04-30 09:34:51]

The CQ bit was checked by pranay.kumar@samsung.com

[Pranay, 2015-04-30 09:34:51]

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

[commit-bot: I haz the power, 2015-04-30 09:35:39]

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

[commit-bot: I haz the power, 2015-04-30 10:38:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-30 10:39:04]

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

[Pranay, 2015-05-04 03:26:59]

Done

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
File chrome/browser/policy/cloud/test_request_interceptor.cc (right):

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/test_request_interceptor.cc:290:
base::Bind(base::IgnoreResult(&base::MessageLoopProxy::PostTask),
On 2015/04/30 08:00:08, pneubeck wrote:
> this reads very inconsistent.
> Shouldn't this be changed to
>  ...IgnoreResult(&base::TaskRunner::PostTask)...
> ?

Done.

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/test_request_interceptor.cc:335:
base::IgnoreResult(&base::MessageLoopProxy::PostTask),
On 2015/04/30 08:00:08, pneubeck wrote:
> ditto.
> 
> now you should be able to remove the include
> "base/message_loop/message_loop_proxy.h"
> as well.

Done.

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
File chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc (right):

https://codereview.chromium.org/1118483002/diff/1/chrome/browser/policy/cloud...
chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc:6: #include
"base/message_loop/message_loop.h"
On 2015/04/30 08:00:08, pneubeck wrote:
> also not required anymore? (I can't find any reference to MessageLoop)

Done.