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

Unified Diff: test/mjsunit/debug-sourceinfo.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-setexceptionbreak.js ('k') | test/mjsunit/debug-sourceslice.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/debug-sourceinfo.js
diff --git a/test/mjsunit/debug-sourceinfo.js b/test/mjsunit/debug-sourceinfo.js
index ddf80dc51c54aeac8e68ec2b4621931d04a2de31..df203c5692e3aeeef7d82c7b1849ad44a0fd19a5 100644
--- a/test/mjsunit/debug-sourceinfo.js
+++ b/test/mjsunit/debug-sourceinfo.js
@@ -27,17 +27,17 @@
// Flags: --expose-debug-as debug
// For this test to work this file MUST have CR LF line endings.
-function a() { b(); };
+function a() { b(); }
function b() {
c(true);
-};
+}
function c(x) {
if (x) {
return 1;
} else {
return 1;
}
- };
+ }
function d(x) {
x = 1 ;
x = 2 ;
@@ -57,14 +57,14 @@ function d(x) {
}
// Get the Debug object exposed from the debug context global object.
-Debug = debug.Debug
+Debug = debug.Debug;
// This is the number of comment lines above the first test function.
var comment_lines = 29;
// This is the last position in the entire file (note: this equals
// file size of <debug-sourceinfo.js> - 1, since starting at 0).
-var last_position = 14312;
+var last_position = 14311;
// This is the last line of entire file (note: starting at 0).
var last_line = 351;
// This is the last column of last line (note: starting at 0 and +2, due
@@ -75,9 +75,9 @@ var last_column = 2;
// of characters before 'function a(...'.
var comment_line_length = 1726;
var start_a = 10 + comment_line_length;
-var start_b = 37 + comment_line_length;
-var start_c = 71 + comment_line_length;
-var start_d = 163 + comment_line_length;
+var start_b = 36 + comment_line_length;
+var start_c = 69 + comment_line_length;
+var start_d = 160 + comment_line_length;
// The position of the first line of d(), i.e. "x = 1 ;".
var start_code_d = start_d + 7;
@@ -199,12 +199,12 @@ assertEquals(0, script.locationFromLine(0, 0, start_a).line - comment_lines);
assertEquals(10, script.locationFromLine(0, 0, start_a).column);
// Test second line offset function a().
-assertEquals(start_a + 14, script.locationFromLine(1, 0, start_a).position);
+assertEquals(start_a + 13, script.locationFromLine(1, 0, start_a).position);
assertEquals(1, script.locationFromLine(1, 0, start_a).line - comment_lines);
assertEquals(0, script.locationFromLine(1, 0, start_a).column);
// Test second line column 2 offset function a().
-assertEquals(start_a + 14 + 2, script.locationFromLine(1, 2, start_a).position);
+assertEquals(start_a + 13 + 2, script.locationFromLine(1, 2, start_a).position);
assertEquals(1, script.locationFromLine(1, 2, start_a).line - comment_lines);
assertEquals(2, script.locationFromLine(1, 2, start_a).column);
@@ -268,7 +268,7 @@ var location;
location = script.locationFromLine(1, 0, start_b);
assertEquals(' c(true);', location.sourceText());
-result = ['c', ' c', ' c(', ' c(', ' c(t']
+result = ['c', ' c', ' c(', ' c(', ' c(t'];
for (var i = 1; i <= 5; i++) {
location = script.locationFromLine(1, 2, start_b);
location.restrict(i);
« no previous file with comments | « test/mjsunit/debug-setexceptionbreak.js ('k') | test/mjsunit/debug-sourceslice.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698