Fix SharedWorker leak when script load has failed.

Fix SharedWorker leak when script load has failed.

When SharedWorker script load has failed and the |WebSharedWorkerImpl|
was explicit told to terminate, the |WebSharedWorkerImpl| instance
was never deleted.

This CL fixes the issue by invoking "delete this" when a worker thread
instance does not exist in |stopWorkerThread()|.

TEST=fast/workers/worker-crash-with-invalid-location.html doesn't leak when --enable-leak-detection
BUG=364390
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179011

[kouhei (in TOK), 2014-07-28 06:28:23]

Would you take a look?

[tkent, 2014-07-28 06:47:44]

lgtm

https://codereview.chromium.org/424653002/diff/20001/Source/web/WebSharedWork...
File Source/web/WebSharedWorkerImpl.cpp (right):

https://codereview.chromium.org/424653002/diff/20001/Source/web/WebSharedWork...
Source/web/WebSharedWorkerImpl.cpp:182: bool noPendingActivity = true;
Negative variable is not good for code readability.  hasPendingActivity?

[kouhei (in TOK), 2014-07-28 06:56:26]

Thanks for review!

>
https://codereview.chromium.org/424653002/diff/20001/Source/web/WebSharedWork...
> File Source/web/WebSharedWorkerImpl.cpp (right):
> 
>
https://codereview.chromium.org/424653002/diff/20001/Source/web/WebSharedWork...
> Source/web/WebSharedWorkerImpl.cpp:182: bool noPendingActivity = true;
> Negative variable is not good for code readability.  hasPendingActivity?

Done

[kouhei (in TOK), 2014-07-28 06:56:29]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-07-28 06:57:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kouhei@chromium.org/424653002/40001

[commit-bot: I haz the power, 2014-07-28 09:05:16]

Change committed as 179011

[kouhei (in TOK), 2014-07-29 02:31:39]

A revert of this CL has been created in
https://codereview.chromium.org/424923002/ by kouhei@chromium.org.

The reason for reverting is: May cause crash.