commit 4a90d5fbbb6802c59093b866f3478a814bf02b95
author Ben Murdoch <benm@google.com> Tue Mar 22 12:00:34 2016 +0000
committer Dirk Vogt <dirk@fairphone.com> Fri Mar 17 16:05:42 2017 +0100
tree 486867e3a04e7f17454c94a4c97891bb0e237f1f
parent 3821a70061b1b2077ee51c70eda717bddac94f58

Upgrade V8 to version 4.9.385.28

https://chromium.googlesource.com/v8/v8/+/4.9.385.28

FPIIM-449

Change-Id: I4b2e74289d4bf3667f2f3dc8aa2e541f63e26eb4

test/mjsunit/debug-evaluate-closure.js

diff --git a/test/mjsunit/debug-evaluate-closure.js b/test/mjsunit/debug-evaluate-closure.js
index cf507b5..541dec9 100644
--- a/test/mjsunit/debug-evaluate-closure.js
+++ b/test/mjsunit/debug-evaluate-closure.js
@@ -26,7 +26,7 @@
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 // Flags: --expose-debug-as debug --allow-natives-syntax
-// Flags: --turbo-deoptimization
+// Flags: --debug-eval-readonly-locals
 
 Debug = debug.Debug;
 var listened = false;
@@ -35,17 +35,18 @@
   if (event != Debug.DebugEvent.Break) return;
   try {
     assertEquals("goo", exec_state.frame(0).evaluate("goo").value());
-    exec_state.frame(0).evaluate("goo = 'goo foo'");
+    exec_state.frame(0).evaluate("goo = 'goo foo'");  // no effect
     assertEquals("bar return", exec_state.frame(0).evaluate("bar()").value());
     assertEquals("inner bar", exec_state.frame(0).evaluate("inner").value());
     assertEquals("outer bar", exec_state.frame(0).evaluate("outer").value());
+
     assertEquals("baz inner", exec_state.frame(0).evaluate("baz").value());
     assertEquals("baz outer", exec_state.frame(1).evaluate("baz").value());
     exec_state.frame(0).evaluate("w = 'w foo'");
-    exec_state.frame(0).evaluate("inner = 'inner foo'");
-    exec_state.frame(0).evaluate("outer = 'outer foo'");
-    exec_state.frame(0).evaluate("baz = 'baz inner foo'");
-    exec_state.frame(1).evaluate("baz = 'baz outer foo'");
+    exec_state.frame(0).evaluate("inner = 'inner foo'");  // no effect
+    exec_state.frame(0).evaluate("outer = 'outer foo'");  // has effect
+    exec_state.frame(0).evaluate("baz = 'baz inner foo'");  // no effect
+    exec_state.frame(1).evaluate("baz = 'baz outer foo'");  // has effect
     listened = true;
   } catch (e) {
     print(e);
@@ -67,7 +68,7 @@
 
   with (withv) {
     var bar = function bar() {
-      assertEquals("goo foo", goo);
+      assertEquals("goo", goo);
       inner = "inner bar";
       outer = "outer bar";
       v = "v bar";
@@ -79,8 +80,8 @@
     debugger;
   }
 
-  assertEquals("inner foo", inner);
-  assertEquals("baz inner foo", baz);
+  assertEquals("inner bar", inner);
+  assertEquals("baz inner", baz);
   assertEquals("w foo", withw.w);
   assertEquals("v bar", withv.v);
 }