Clear EmbeddedWorkerContextClient's thread-local ptr in willDestroyWorkerContext()

Clear EmbeddedWorkerContextClient's thread-local ptr in willDestroyWorkerContext()

Back then EmbeddedWorkerContextClient used to be destroyed on the worker
thread when corresponding WorkerGlobalScope's destructed, but after
http://crrev.com/262813004 we started to destruct it on main thread
(after the worker side termination), so resetting thread-local ptr in
the dtor doesn't work (as it runs on main thread).

This change clears the thread-local ptr in willDestroyWorkerContext(),
where we're sure we're still on the worker thread.

BUG=n/a

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272487

[kinuko, 2014-05-22 08:46:59]

I think this fix http://crbug.com/375643... PTL

[kinuko, 2014-05-22 11:50:23]

On 2014/05/22 08:46:59, kinuko wrote:
> I think this fix http://crbug.com/375643... PTL

After some investigation with horo@ this doesn't look related to
http://crbug.com/375643.

[horo, 2014-05-22 11:50:55]

lgtm

[michaeln, 2014-05-22 19:23:51]

lgtm

https://codereview.chromium.org/287193011/diff/1/content/renderer/service_wor...
File content/renderer/service_worker/embedded_worker_context_client.cc (right):

https://codereview.chromium.org/287193011/diff/1/content/renderer/service_wor...
content/renderer/service_worker/embedded_worker_context_client.cc:145: //
g_worker_client_tls.Pointer()->Get() could return NULL if this context
this comment seems out of place here, two lines up we assert its null?

[kinuko, 2014-05-23 02:47:47]

https://codereview.chromium.org/287193011/diff/1/content/renderer/service_wor...
File content/renderer/service_worker/embedded_worker_context_client.cc (right):

https://codereview.chromium.org/287193011/diff/1/content/renderer/service_wor...
content/renderer/service_worker/embedded_worker_context_client.cc:145: //
g_worker_client_tls.Pointer()->Get() could return NULL if this context
On 2014/05/22 19:23:52, michaeln wrote:
> this comment seems out of place here, two lines up we assert its null?

Done.

[kinuko, 2014-05-23 02:47:52]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-05-23 02:49:25]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kinuko@chromium.org/287193011/20001

[commit-bot: I haz the power, 2014-05-23 11:08:23]

Change committed as 272487