ScriptPromiseResolver leaked resource when constructed on stopped context.

ScriptPromiseResolver leaked resource when constructed on stopped context.

ScriptPromiseResolver stops working when the associated ExecutionContext is
stopped. ScriptPromiseResolver can be constructed on a stopped
ExecutionContext. In such a case the constructed resolver is stopped as well.
Moreover, it should release the created (internal) resolver not to leak
resource.

BUG=385384
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183123

[yhirano, 2014-10-02 02:24:22]

yhirano@chromium.org changed reviewers:
+ haraken@chromium.org, sigbjornf@opera.com

[yhirano, 2014-10-02 02:24:22]

I removed sof's workaround as it isn't needed any more.

[yhirano, 2014-10-02 02:26:19]

On 2014/10/02 02:24:22, yhirano wrote:
> I removed sof's workaround as it isn't needed any more.

More background: The leak was fixed at https://codereview.chromium.org/619583004
but it is a ScriptPromiseResolver's bug and I fix it here.

[haraken, 2014-10-02 02:35:17]

LGTM, this fix makes a lot of more sense!

[yhirano, 2014-10-02 03:47:07]

yhirano@chromium.org changed reviewers:
+ toyoshim@chromium.org

[yhirano, 2014-10-02 03:47:08]

This change breaks a WebMIDI test.
toyoshim@, can you review the WebMIDI change?

[Takashi Toyoshima, 2014-10-02 03:59:09]

LGTM.
It does not break the test, but fixes behavior correctly:)

[yhirano, 2014-10-02 06:45:41]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2014-10-02 06:46:21]

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

[commit-bot: I haz the power, 2014-10-02 07:59:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-02 07:59:23]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/29833)

[yhirano, 2014-10-02 08:01:13]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2014-10-02 08:02:07]

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

[commit-bot: I haz the power, 2014-10-02 08:35:51]

Committed patchset #3 (id:40001) as 183123