Reviewers: Michael Starzinger,

Description:
Fix cluster-fuzz found regression in d8 Workers.

Dumb typo introduced in refs/heads/master@{#29306}. I thought I was turning on report_exceptions in Shell::ExecuteString, but instead I turned on print_result
(which assumes an interactive debugger and a HandleScope for the
utility_context_).

BUG=chromium:504727,chromium:504728
[email protected]
LOG=n

Please review this at https://codereview.chromium.org/1219563002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+1, -1 lines):
  M src/d8.cc


Index: src/d8.cc
diff --git a/src/d8.cc b/src/d8.cc
index e165aa47e020e377c713872c552d255ca7acf2f4..5022d93e11ce5f5fc76ba74f2e89e4d0b9b9fbe7 100644
--- a/src/d8.cc
+++ b/src/d8.cc
@@ -1706,7 +1706,7 @@ void Worker::ExecuteInThread() {
         // First run the script
         Handle<String> file_name = String::NewFromUtf8(isolate, "unnamed");
         Handle<String> source = String::NewFromUtf8(isolate, script_);
-        if (Shell::ExecuteString(isolate, source, file_name, true, true)) {
+ if (Shell::ExecuteString(isolate, source, file_name, false, true)) {
           // Get the message handler
           Handle<Value> onmessage =
               global->Get(String::NewFromUtf8(isolate, "onmessage"));


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to