Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(379)

Unified Diff: test/mjsunit/debug-evaluate-recursive.js

Issue 8888006: Make more JS files beter match the coding standard. Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Address review comments Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/mjsunit/debug-evaluate-locals-optimized-double.js ('k') | test/mjsunit/debug-evaluate-with.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/debug-evaluate-recursive.js
diff --git a/test/mjsunit/debug-evaluate-recursive.js b/test/mjsunit/debug-evaluate-recursive.js
index f34943e5f40ab1be0bd6d8040140ee09c9d20d8f..4d21e2ce3064783c7ebc29f9b142d739956b3d79 100644
--- a/test/mjsunit/debug-evaluate-recursive.js
+++ b/test/mjsunit/debug-evaluate-recursive.js
@@ -27,13 +27,13 @@
// Flags: --expose-debug-as debug
// Get the Debug object exposed from the debug context global object.
-Debug = debug.Debug
+Debug = debug.Debug;
listenerComplete = false;
exception = false;
// The base part of all evaluate requests.
-var base_request = '"seq":0,"type":"request","command":"evaluate"'
+var base_request = '"seq":0,"type":"request","command":"evaluate"';
function safeEval(code) {
try {
@@ -53,7 +53,7 @@ function testRequest(exec_state, arguments, success, result) {
if (arguments) {
request = '{' + base_request + ',"arguments":' + arguments + '}';
} else {
- request = '{' + base_request + '}'
+ request = '{' + base_request + '}';
}
var response = safeEval(dcp.processDebugJSONRequest(request));
if (success) {
@@ -69,8 +69,7 @@ function testRequest(exec_state, arguments, success, result) {
// Event listener which evaluates with break disabled.
function listener(event, exec_state, event_data, data) {
try {
- if (event == Debug.DebugEvent.Break)
- {
+ if (event == Debug.DebugEvent.Break) {
// Call functions with break using the FrameMirror directly.
assertEquals(1, exec_state.evaluateGlobal('f()', true).value());
assertEquals(2, exec_state.evaluateGlobal('g()', true).value());
@@ -97,9 +96,9 @@ function listener(event, exec_state, event_data, data) {
listenerComplete = true;
}
} catch (e) {
- exception = e
- };
-};
+ exception = e;
+ }
+}
// Event listener which evaluates with break enabled one time and the second
@@ -107,8 +106,7 @@ function listener(event, exec_state, event_data, data) {
var break_count = 0;
function listener_recurse(event, exec_state, event_data, data) {
try {
- if (event == Debug.DebugEvent.Break)
- {
+ if (event == Debug.DebugEvent.Break) {
break_count++;
// Call functions with break using the FrameMirror directly.
@@ -124,9 +122,9 @@ function listener_recurse(event, exec_state, event_data, data) {
}
}
} catch (e) {
- exception = e
- };
-};
+ exception = e;
+ }
+}
// Add the debug event listener.
Debug.setListener(listener);
@@ -134,19 +132,19 @@ Debug.setListener(listener);
// Test functions - one with break point and one with debugger statement.
function f() {
return 1;
-};
+}
function g() {
debugger;
return 2;
-};
+}
Debug.setBreakPoint(f, 2, 0);
// Cause a debug break event.
debugger;
-assertFalse(exception, "exception in listener")
+assertFalse(exception, "exception in listener");
// Make sure that the debug event listener vas invoked.
assertTrue(listenerComplete);
@@ -161,7 +159,7 @@ Debug.setBreakPoint(f, 2, 0);
debugger;
-assertFalse(exception, "exception in listener")
+assertFalse(exception, "exception in listener");
// Make sure that the debug event listener vas invoked.
assertTrue(listenerComplete);
assertEquals(2, break_count);
« no previous file with comments | « test/mjsunit/debug-evaluate-locals-optimized-double.js ('k') | test/mjsunit/debug-evaluate-with.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698