Make more JS files beter match the coding standard.

test/mjsunit/debug-breakpoints.js

Index: test/mjsunit/debug-breakpoints.js
diff --git a/test/mjsunit/debug-breakpoints.js b/test/mjsunit/debug-breakpoints.js
index 13020343b0ec61f351dfaf984a599de5c60af979..bae0b7e41409ff09dde34b3180828a20ad35f887 100644
--- a/test/mjsunit/debug-breakpoints.js
+++ b/test/mjsunit/debug-breakpoints.js
@@ -27,30 +27,30 @@
 
 // Flags: --expose-debug-as debug
 // Get the Debug object exposed from the debug context global object.
-Debug = debug.Debug
+Debug = debug.Debug;
 
-function f() {a=1;b=2};
+function f() {a=1;b=2;}

[Rico, 2011/12/08 10:24:37]

move body to (two) individual lines and space around =
if this is due to breakpoint positions just leave as is

[Lasse Reichstein, 2011/12/08 12:33:18]

Actual function body is being used in tests below. Better leave it be.

 function g() {
   a=1;
   b=2;
 }
 

[Rico, 2011/12/08 10:24:37]

there seems to be missing a lot of spaces below, if this is to ease the tests
depending on position I am fine with leaving this as is

[Lasse Reichstein, 2011/12/08 12:33:18]

Again the missing space is because it refers to the actual body above.
I'm not about to rewrite the the checks of the test in this CL.

 bp = Debug.setBreakPoint(f, 0, 0);
-assertEquals("() {[B0]a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;b=2;}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp);
-assertEquals("() {a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;}", Debug.showBreakPoints(f));
 bp1 = Debug.setBreakPoint(f, 0, 8);
-assertEquals("() {a=1;[B0]b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;[B0]b=2;}", Debug.showBreakPoints(f));
 bp2 = Debug.setBreakPoint(f, 0, 4);
-assertEquals("() {[B0]a=1;[B1]b=2}", Debug.showBreakPoints(f));
-bp3 = Debug.setBreakPoint(f, 0, 11);
-assertEquals("() {[B0]a=1;[B1]b=2[B2]}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;[B1]b=2;}", Debug.showBreakPoints(f));
+bp3 = Debug.setBreakPoint(f, 0, 12);
+assertEquals("() {[B0]a=1;[B1]b=2;[B2]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp1);
-assertEquals("() {[B0]a=1;b=2[B1]}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;b=2;[B1]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp2);
-assertEquals("() {a=1;b=2[B0]}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;[B0]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp3);
-assertEquals("() {a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;}", Debug.showBreakPoints(f));
 
 // The following test checks that the Debug.showBreakPoints(g) produces output
 // like follows when changein breakpoints.
@@ -132,21 +132,21 @@ function setBreakpointByPosition(f, position)
 }
 
 bp = setBreakpointByPosition(f, 0);
-assertEquals("() {[B0]a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;b=2;}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp);
-assertEquals("() {a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;}", Debug.showBreakPoints(f));
 bp1 = setBreakpointByPosition(f, 8);
-assertEquals("() {a=1;[B0]b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;[B0]b=2;}", Debug.showBreakPoints(f));
 bp2 = setBreakpointByPosition(f, 4);
-assertEquals("() {[B0]a=1;[B1]b=2}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;[B1]b=2;}", Debug.showBreakPoints(f));
 bp3 = setBreakpointByPosition(f, 11);
-assertEquals("() {[B0]a=1;[B1]b=2[B2]}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;[B1]b=2;[B2]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp1);
-assertEquals("() {[B0]a=1;b=2[B1]}", Debug.showBreakPoints(f));
+assertEquals("() {[B0]a=1;b=2;[B1]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp2);
-assertEquals("() {a=1;b=2[B0]}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;[B0]}", Debug.showBreakPoints(f));
 Debug.clearBreakPoint(bp3);
-assertEquals("() {a=1;b=2}", Debug.showBreakPoints(f));
+assertEquals("() {a=1;b=2;}", Debug.showBreakPoints(f));
 
 bp = setBreakpointByPosition(g, 0);
 //function g() {