harmony-scoping: Implement debugger support for script scope.

test/cctest/test-debug.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 7613 matching lines @@
     const v8::Debug::EventDetails& event_details) {
   v8::DebugEvent event = event_details.GetEvent();
   if (event != v8::Break) return;
 
   int break_id = CcTest::i_isolate()->debug()->break_id();
 
   char script[128];
   i::Vector<char> script_vector(script, sizeof(script));
   SNPrintF(script_vector, "%%GetFrameCount(%d)", break_id);
   v8::Local<v8::Value> result = CompileRun(script);
+  CHECK_EQ(1, result->Int32Value());

[aandrey, 2014/11/14 15:58:47]

nit: you can use ExpectInt32() here and below

[Dmitry Lomov (no reviews), 2014/11/14 16:37:26]

Done.

 
+  SNPrintF(script_vector, "var frame = new FrameMirror(%d, 0);", break_id);
+  CompileRun(script);
+  result = CompileRun("frame.evaluate('x').value_");
   CHECK_EQ(1, result->Int32Value());
+  result = CompileRun("frame.evaluate('y').value_");
+  CHECK_EQ(2, result->Int32Value());
+
+  CompileRun("var allScopes = frame.allScopes()");
+  result = CompileRun("allScopes.length");
+  CHECK_EQ(2, result->Int32Value());
+
+  result = CompileRun("allScopes[0].scopeType() === ScopeType.Script");
+  CHECK(result->IsTrue());
+
+  result = CompileRun("allScopes[0].scopeObject().value_.x");
+  CHECK_EQ(1, result->Int32Value());
+
+  result = CompileRun("allScopes[0].scopeObject().value_.y");
+  CHECK_EQ(2, result->Int32Value());
+
+  CompileRun("allScopes[0].setVariableValue('x', 5);");
+  CompileRun("allScopes[0].setVariableValue('y', 6);");
+  result = CompileRun("frame.evaluate('x + y').value_");
+  CHECK_EQ(11, result->Int32Value());
 }
 
 
 TEST(DebugBreakInLexicalScopes) {
   i::FLAG_harmony_scoping = true;
   i::FLAG_allow_natives_syntax = true;
 
   DebugLocalContext env;
   v8::Isolate* isolate = env->GetIsolate();
   v8::HandleScope scope(isolate);
   v8::Debug::SetDebugEventListener(DebugHarmonyScopingListener);
 
   CompileRun(
       "'use strict';            \n"
       "let x = 1;               \n");
-  CompileRun(
+  v8::Local<v8::Value> result = CompileRun(
       "'use strict';            \n"
-      "let y = 1;               \n"
-      "debugger                 \n");
+      "let y = 2;               \n"
+      "debugger;                \n"
+      "x * y");
+  CHECK_EQ(30, result->Int32Value());
+
+  result = CompileRun(
+      "x = 1; y = 2; \n"
+      "debugger;"
+      "x * y");
+  CHECK_EQ(30, result->Int32Value());
 }