[Fetch API] Fix ScriptState leak in Fetch API unittests.

[Fetch API] Fix ScriptState leak in Fetch API unittests.

This CL fixes memory leaks caused by not calling disposePerContextData on
ScriptState.

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

[yhirano, 2015-08-27 06:05:12]

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

[haraken, 2015-08-27 06:11:14]

The change looks ok, but can we probably use ScriptStateForTesting instead? It
will take care of disposePerContext.

[yhirano, 2015-08-27 06:59:21]

On 2015/08/27 06:11:14, haraken wrote:
> The change looks ok, but can we probably use ScriptStateForTesting instead? It
> will take care of disposePerContext.

V8TestingScope uses DOMWrapperWorld::mainWorld(), but the ScriptState works on a
non-main thread. So I think I cannot use it.

[haraken, 2015-08-27 08:53:10]

On 2015/08/27 06:59:21, yhirano wrote:
> On 2015/08/27 06:11:14, haraken wrote:
> > The change looks ok, but can we probably use ScriptStateForTesting instead?
It
> > will take care of disposePerContext.
> 
> V8TestingScope uses DOMWrapperWorld::mainWorld(), but the ScriptState works on
a
> non-main thread. So I think I cannot use it.

ok, LGTM

[yhirano, 2015-08-27 08:58:30]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2015-08-27 08:58:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1314333002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1314333002/1

[commit-bot: I haz the power, 2015-08-27 09:02:36]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201299