[storage] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in complete storage module.

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

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

MessageLoopProxy -> SingleThreadTaskRunner

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

[Pranay, 2015-04-30 07:23:59]

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

[Pranay, 2015-04-30 07:24:00]

PTAL

Thanks

[kinuko, 2015-04-30 07:52:06]

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

[kinuko, 2015-04-30 07:52:06]

Can you run try jobs?

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_s...
File storage/browser/blob/blob_storage_context.cc (right):

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_s...
storage/browser/blob/blob_storage_context.cc:84: ));
Can this be appended at the end of the previous line?

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_u...
File storage/browser/blob/blob_url_request_job.h (right):

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_u...
storage/browser/blob/blob_url_request_job.h:99:
scoped_refptr<base::SingleThreadTaskRunner> file_thread_runner_;
nit: file_task_runner_

[Pranay, 2015-04-30 10:11:52]

Hi Kinuko,

Thanks for the review. I have updated the changes in new patch set.
I don't have the try jobs access as of now, tested locally.

PTAL

[kinuko, 2015-04-30 11:00:09]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-30 11:00:37]

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

[kinuko, 2015-04-30 12:44:49]

lgtm with one more nit

https://codereview.chromium.org/1120553002/diff/20001/storage/browser/blob/bl...
File storage/browser/blob/blob_storage_context.cc (right):

https://codereview.chromium.org/1120553002/diff/20001/storage/browser/blob/bl...
storage/browser/blob/blob_storage_context.cc:83: ));
Um, sorry I meant if we could have '));' on the same line as the previous line.

...(uuid,
    this,
    base::ThreadTaskRunnerHandle::Get().get()));

[kinuko, 2015-04-30 13:02:56]

The CQ bit was unchecked by kinuko@chromium.org

[Pranay, 2015-04-30 14:47:57]

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

[commit-bot: I haz the power, 2015-04-30 14:48:48]

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

[kinuko, 2015-04-30 15:04:53]

The CQ bit was unchecked by kinuko@chromium.org

[kinuko, 2015-04-30 15:05:12]

On 2015/04/30 12:44:49, kinuko wrote:
> lgtm with one more nit
> 
>
https://codereview.chromium.org/1120553002/diff/20001/storage/browser/blob/bl...
> File storage/browser/blob/blob_storage_context.cc (right):
> 
>
https://codereview.chromium.org/1120553002/diff/20001/storage/browser/blob/bl...
> storage/browser/blob/blob_storage_context.cc:83: ));
> Um, sorry I meant if we could have '));' on the same line as the previous
line.
> 
> ...(uuid,
>     this,
>     base::ThreadTaskRunnerHandle::Get().get()));

Could you fix this one before submitting? Thanks

[Pranay, 2015-05-04 03:06:57]

Hi Kinuko,

Sorry for that :) Have updated the patch set.

Thanks

[Pranay, 2015-05-04 03:07:04]

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

[Pranay, 2015-05-04 03:07:05]

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

[commit-bot: I haz the power, 2015-05-04 03:07:22]

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

[Pranay, 2015-05-04 03:20:58]

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_s...
File storage/browser/blob/blob_storage_context.cc (right):

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_s...
storage/browser/blob/blob_storage_context.cc:84: ));
On 2015/04/30 07:52:06, kinuko wrote:
> Can this be appended at the end of the previous line?

Done.

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_u...
File storage/browser/blob/blob_url_request_job.h (right):

https://codereview.chromium.org/1120553002/diff/1/storage/browser/blob/blob_u...
storage/browser/blob/blob_url_request_job.h:99:
scoped_refptr<base::SingleThreadTaskRunner> file_thread_runner_;
On 2015/04/30 07:52:06, kinuko wrote:
> nit: file_task_runner_

Done.

[commit-bot: I haz the power, 2015-05-04 04:12:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-04 04:13:38]

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