webcrypto: Get task-runner for worker-thread from base::ThreadTaskRunnerHandle

webcrypto: Get task-runner for worker-thread from base::ThreadTaskRunnerHandle

Both main-thread and worker-threads have their own message-loops. So use
base::ThreadTaskRunnerHandle::Get() to get the task-runner in both main and
worker threads.

BUG=none
R=eroman@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/3a4b2d1ad07c4e79ab2923d6a571846d64f9f1f1

[sadrul, 2015-02-03 22:57:18]

Patchset #2 (id:20001) has been deleted

[sadrul, 2015-02-03 23:12:05]

sadrul@chromium.org changed reviewers:
+ rsleevi@chromium.org

[Ryan Sleevi, 2015-02-03 23:13:55]

rsleevi@chromium.org changed reviewers:
+ eroman@chromium.org

[Ryan Sleevi, 2015-02-03 23:13:56]

Eric should review this. When this was written, the Worker wasn't setting the
ThreadTTRH, AIUI.

[eroman, 2015-02-03 23:55:51]

LGTM.

At that code was first written there was written there wasn't a messageloop for
worker threads.

It looks like that has since changed; I believe it comes from the WorkerThread
-> WebWorker refactoring that nasko@ did.

cc-ing for confirmation.

(Also as an aside, the webcrypto WebWorker LayoutTests are marked flaky crash so
regressions here will not be caught by testing. Sigh. That said I tested this
change and it works).

[nasko, 2015-02-04 00:57:47]

On 2015/02/03 23:55:51, eroman wrote:
> LGTM.
> 
> At that code was first written there was written there wasn't a messageloop
for
> worker threads.
> 
> It looks like that has since changed; I believe it comes from the WorkerThread
> -> WebWorker refactoring that nasko@ did.
> 
> cc-ing for confirmation.

I indeed refactored worker threads from using Blink specific threads to using
WebThread, which is implemented on top of base::Thread. This means that all
threads in Blink these days are building on top of base::Thread and this CL
looks fine to me.

> (Also as an aside, the webcrypto WebWorker LayoutTests are marked flaky crash
so
> regressions here will not be caught by testing. Sigh. That said I tested this
> change and it works).

[commit-bot: I haz the power, 2015-02-04 01:13:13]

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

[sadrul, 2015-02-04 01:13:19]

Committed patchset #2 (id:40001) manually as
3a4b2d1ad07c4e79ab2923d6a571846d64f9f1f1 (presubmit successful).