Revision: 4028
Author: [email protected]
Date: Thu Mar  4 23:05:26 2010
Log: Fix weird issue with test case where certain global function
names were being reused.
Review URL: http://codereview.chromium.org/669148
http://code.google.com/p/v8/source/detail?r=4028

Modified:
 /branches/bleeding_edge/test/mjsunit/debug-scopes.js

=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-scopes.js Thu Oct 15 13:06:08 2009 +++ /branches/bleeding_edge/test/mjsunit/debug-scopes.js Thu Mar 4 23:05:26 2010
@@ -84,16 +84,16 @@
     var scope = exec_state.frame().scope(i);
     assertTrue(scope.isScope());
     assertEquals(scopes[i], scope.scopeType());
-
+
     // Check the global object when hitting the global scope.
     if (scopes[i] == debug.ScopeType.Global) {
       assertEquals(this, scope.scopeObject().value());
     }
   }
-
+
   // Get the debug command processor.
   var dcp = exec_state.debugCommandProcessor("unspecified_running_state");
-
+
   // Send a scopes request and check the result.
   var json;
   request_json = '{"seq":0,"type":"request","command":"scopes"}'
@@ -133,7 +133,7 @@
     }
     count++;
   }
-
+
   // 'arguments' and might be exposed in the local and closure scope. Just
   // ignore this.
   var scope_size = scope.scopeObject().properties().length;
@@ -156,7 +156,7 @@

   // Get the debug command processor.
   var dcp = exec_state.debugCommandProcessor("unspecified_running_state");
-
+
   // Send a scope request for information on a single scope and check the
   // result.
request_json = '{"seq":0,"type":"request","command":"scope","arguments":{"number":'
@@ -622,7 +622,7 @@
     with ({j:13}){
       return function() {
         var x = 14;
-        with ({a:15}) {
+        with ({a:15}) {
           with ({b:16}) {
             debugger;
             some_global = a;
@@ -707,7 +707,7 @@


 BeginTest("Catch block 3");
-function catch_block_1() {
+function catch_block_3() {
   // Do eval to dynamically declare a local variable so that the context's
   // extension slot is initialized with JSContextExtensionObject.
   eval("var y = 78;");
@@ -726,12 +726,12 @@
   CheckScopeContent({e:'Exception'}, 0, exec_state);
   CheckScopeContent({y:78}, 1, exec_state);
 }
-catch_block_1()
+catch_block_3()
 EndTest();


 BeginTest("Catch block 4");
-function catch_block_2() {
+function catch_block_4() {
   // Do eval to dynamically declare a local variable so that the context's
   // extension slot is initialized with JSContextExtensionObject.
   eval("var y = 98;");
@@ -753,7 +753,7 @@
   CheckScopeContent({e:'Exception'}, 1, exec_state);
   CheckScopeContent({y:98}, 2, exec_state);
 }
-catch_block_2()
+catch_block_4()
 EndTest();


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to