[chrome/browser/drive] Avoid use of MessageLoopProxy by TTRH

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

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

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

MessageLoopProxy -> SingleThreadTaskRunner

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

[Pranay, 2015-04-29 05:30:44]

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

[Pranay, 2015-04-29 05:30:45]

PTAL

Thanks

[Pranay, 2015-04-30 10:16:09]

pranay.kumar@samsung.com changed reviewers:
- satorux@chromium.org

[Pranay, 2015-04-30 10:17:23]

pranay.kumar@samsung.com changed reviewers:
+ fukino@chromium.org, satorux@chromium.org

[Pranay, 2015-04-30 10:17:23]

PTAL

[Pranay, 2015-05-06 04:21:34]

pranay.kumar@samsung.com changed reviewers:
- fukino@chromium.org, satorux@chromium.org

[Pranay, 2015-05-06 04:24:44]

pranay.kumar@samsung.com changed reviewers:
+ hashimoto@chromium.org, satorux@chromium.org

[Pranay, 2015-05-06 04:24:45]

satorux@chromium.org: Please review changes in chrome/browser/drive

hashimoto@chromium.org: Please review changes in chrome/browser/drive

[hashimoto, 2015-05-07 05:47:41]

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/drive_...
File chrome/browser/drive/drive_uploader_unittest.cc (right):

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/drive_...
chrome/browser/drive/drive_uploader_unittest.cc:96:
base::MessageLoop::current()->PostTask(FROM_HERE,
While you are here, could you replace MessageLoop::current() with
ThreadTaskRunnerHandle::Get() when it makes sense?
The same goes for the other file.

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/fake_d...
File chrome/browser/drive/fake_drive_service.cc (right):

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/fake_d...
chrome/browser/drive/fake_drive_service.cc:13: #include
"base/message_loop/message_loop.h"
Can't this include be deleted? (after replacing MessageLoop::current)

[Pranay, 2015-05-07 06:49:51]

Done :)

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/drive_...
File chrome/browser/drive/drive_uploader_unittest.cc (right):

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/drive_...
chrome/browser/drive/drive_uploader_unittest.cc:96:
base::MessageLoop::current()->PostTask(FROM_HERE,
On 2015/05/07 05:47:40, hashimoto wrote:
> While you are here, could you replace MessageLoop::current() with
> ThreadTaskRunnerHandle::Get() when it makes sense?
> The same goes for the other file.

Done.

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/fake_d...
File chrome/browser/drive/fake_drive_service.cc (right):

https://codereview.chromium.org/1115573002/diff/1/chrome/browser/drive/fake_d...
chrome/browser/drive/fake_drive_service.cc:13: #include
"base/message_loop/message_loop.h"
On 2015/05/07 05:47:40, hashimoto wrote:
> Can't this include be deleted? (after replacing MessageLoop::current)

Done.

[Pranay, 2015-05-07 06:50:10]

PTAL

Thanks

[hashimoto, 2015-05-07 09:49:41]

Thanks, just one point.

https://codereview.chromium.org/1115573002/diff/20001/chrome/browser/drive/dr...
File chrome/browser/drive/drive_uploader_unittest.cc (right):

https://codereview.chromium.org/1115573002/diff/20001/chrome/browser/drive/dr...
chrome/browser/drive/drive_uploader_unittest.cc:415: protected:
Sorry for being unclear, message_loop.h include can be removed from
fake_drive_service.cc, but not in drive_uploader_unittest.cc.
IIUC this "base::MessageLoop message_loop_;" is needed to run tests properly.

[Pranay, 2015-05-07 10:01:09]

https://codereview.chromium.org/1115573002/diff/20001/chrome/browser/drive/dr...
File chrome/browser/drive/drive_uploader_unittest.cc (right):

https://codereview.chromium.org/1115573002/diff/20001/chrome/browser/drive/dr...
chrome/browser/drive/drive_uploader_unittest.cc:415: protected:
On 2015/05/07 09:49:41, hashimoto wrote:
> Sorry for being unclear, message_loop.h include can be removed from
> fake_drive_service.cc, but not in drive_uploader_unittest.cc.
> IIUC this "base::MessageLoop message_loop_;" is needed to run tests properly.

Done.

[hashimoto, 2015-05-07 10:02:36]

lgtm assuming all tests running properly.

[Pranay, 2015-05-07 10:02:52]

Hi Hashimoto,
Thanks for the review, updated the patch.
Sorry i misunderstood your point :)

PTAL

[Pranay, 2015-05-07 10:03:19]

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

[commit-bot: I haz the power, 2015-05-07 10:03:34]

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

[commit-bot: I haz the power, 2015-05-07 11:36:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-07 11:37:55]

Patchset 3 (id:??) landed as
https://crrev.com/a2a4741fe5eb144e2971e6a4be57ceee039724bc
Cr-Commit-Position: refs/heads/master@{#328742}