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

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in chrome/browser/chromeos/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/229290b50b2c5425371015d9526a71d0e33df4aa
Cr-Commit-Position: refs/heads/master@{#329807}

[Pranay, 2015-05-05 08:56:13]

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

[Pranay, 2015-05-05 08:56:14]

PTAL

Thanks

[xiyuan, 2015-05-05 16:15:19]

+satorux owner of c/b/chromeos/drive

lgtm since the change is a mostly mechanical.

[satorux1, 2015-05-05 16:23:22]

satorux@chromium.org changed reviewers:
+ satorux@chromium.org

[satorux1, 2015-05-05 16:23:23]

https://codereview.chromium.org/1124813003/diff/1/chrome/browser/chromeos/dri...
File chrome/browser/chromeos/drive/change_list_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/1/chrome/browser/chromeos/dri...
chrome/browser/chromeos/drive/change_list_loader_unittest.cc:104:
loader_controller_.get()));
Please keep them aligned vertically for consistency (ex. see line AddNewFile()
call from line 111). 


Likewise, please keep the original style elsewhere as well.

[Pranay, 2015-05-07 12:19:17]

Hi Satorux,
Thanks for the review. Also apologies, all unittest files has other dependencies
w.r.t. SequencedTaskRunner.
I'll be uploading updated patch soon along with your feedback about vertical
alignment consistency

-Pranay

[Pranay, 2015-05-08 10:21:17]

https://codereview.chromium.org/1124813003/diff/1/chrome/browser/chromeos/dri...
File chrome/browser/chromeos/drive/change_list_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/1/chrome/browser/chromeos/dri...
chrome/browser/chromeos/drive/change_list_loader_unittest.cc:104:
loader_controller_.get()));
On 2015/05/05 16:23:23, satorux1 wrote:
> Please keep them aligned vertically for consistency (ex. see line AddNewFile()
> call from line 111). 
> 
> 
> Likewise, please keep the original style elsewhere as well.

Done.

[Pranay, 2015-05-08 10:23:23]

Hi,
updated the patch as per build issues & alignment consistencies :) PTAL

Thanks

[Pranay, 2015-05-08 10:52:50]

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

[Pranay, 2015-05-08 10:52:51]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/1124813003/#ps40001
(title: "Fixed nits")

[commit-bot: I haz the power, 2015-05-08 10:52:58]

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

[Pranay, 2015-05-08 10:53:03]

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

[Pranay, 2015-05-12 13:49:39]

PING :)

[satorux1, 2015-05-12 14:06:35]

I will defer to fukino@ and hashimoto@
PING :)

https://codereview.chromium.org/1124813003/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[hashimoto, 2015-05-13 05:47:10]

hashimoto@chromium.org changed reviewers:
+ hashimoto@chromium.org

[hashimoto, 2015-05-13 05:47:11]

https://codereview.chromium.org/1124813003/diff/40001/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/change_list_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/40001/chrome/browser/chromeos...
chrome/browser/chromeos/drive/change_list_loader_unittest.cc:84:
pref_service_.get(),
This indent looks weird.
Did you run "git cl format"?

[Pranay, 2015-05-13 06:51:34]

PTAL

https://codereview.chromium.org/1124813003/diff/40001/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/change_list_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/40001/chrome/browser/chromeos...
chrome/browser/chromeos/drive/change_list_loader_unittest.cc:84:
pref_service_.get(),
Sure..Done here and in other files, also maintained vertical alignment

[hashimoto, 2015-05-13 15:38:14]

https://codereview.chromium.org/1124813003/diff/60001/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/directory_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/60001/chrome/browser/chromeos...
chrome/browser/chromeos/drive/directory_loader_unittest.cc:81:
pref_service_.get(),
This looks still weird.

How did you run "git cl format"?
Normally, it should automatically format all changed files.

[Pranay, 2015-05-14 04:42:00]

Done

https://codereview.chromium.org/1124813003/diff/60001/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/directory_loader_unittest.cc (right):

https://codereview.chromium.org/1124813003/diff/60001/chrome/browser/chromeos...
chrome/browser/chromeos/drive/directory_loader_unittest.cc:81:
pref_service_.get(),
Since git cl format was interfering with the vertical alignment ( it was merging
two arguments in the same line depending upon length, as Satorux commented in -
"https://codereview.chromium.org/1124813003/diff/1/chrome/browser/chromeos/drive/change_list_loader_unittest.cc#newcode104"
)I have tried to combine both git cl format & vertical alignment manually. Still
this one got missed by me, apologies..
Thanks for the review, Fixed :)

[Pranay, 2015-05-14 04:42:32]

PTAL again :) Thanks

[hashimoto, 2015-05-14 04:44:17]

lgtm

[Pranay, 2015-05-14 04:46:37]

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

[Pranay, 2015-05-14 04:46:38]

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

[commit-bot: I haz the power, 2015-05-14 04:47:18]

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

[commit-bot: I haz the power, 2015-05-14 05:21:11]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-05-14 05:22:33]

Patchset 5 (id:??) landed as
https://crrev.com/229290b50b2c5425371015d9526a71d0e33df4aa
Cr-Commit-Position: refs/heads/master@{#329807}