[V8] ScriptRunner::runCompiledScript() should return Handle<Value>() when OOM occurs
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Feb 2013 23:01:20 +0000 (23:01 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Feb 2013 23:01:20 +0000 (23:01 +0000)
commite7bb17eda555be95dd54d23214fb3f0629bdcc4a
tree371a823873d254738dedc0e8b60e00aba0cdc921
parentfca28785aff0fa999e9db77cdb35a2517c44b84d
[V8] ScriptRunner::runCompiledScript() should return Handle<Value>() when OOM occurs
https://bugs.webkit.org/show_bug.cgi?id=110254

Patch by Kentaro Hara <haraken@google.com> on 2013-02-19
Reviewed by Adam Barth.

Chromium bug: https://code.google.com/p/chromium/issues/detail?id=176951

Currently ScriptRunner::runCompiledScript() calls
ASSERT(script.IsEmpty()) when OOM occurs. The Chromium bug is
hitting the ASSERT(). The ASSERT() does not make sense at all.
Instead, we should return an empty handle when OOM occurs.

No tests. The bug happens only when OOM occurs.

* bindings/v8/ScriptRunner.cpp:
(WebCore::ScriptRunner::runCompiledScript):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143387 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/bindings/v8/ScriptRunner.cpp