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

Unified Diff: test/mjsunit/strict-mode.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/stack-traces.js ('k') | test/mjsunit/strict-mode-implicit-receiver.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/strict-mode.js
diff --git a/test/mjsunit/strict-mode.js b/test/mjsunit/strict-mode.js
index 9c9bdfd52d42662f0dcb13e22402e8fc8f1cf5a9..a5ecd24cdaad257fc365a300fe4197e911962fda 100644
--- a/test/mjsunit/strict-mode.js
+++ b/test/mjsunit/strict-mode.js
@@ -64,7 +64,7 @@ function CheckFunctionConstructorStrictMode() {
// Incorrect 'use strict' directive.
(function UseStrictEscape() {
"use\\x20strict";
- with ({}) {};
+ with ({}) {}
})();
// Incorrectly place 'use strict' directive.
@@ -74,7 +74,7 @@ assertThrows("function foo (x) 'use strict'; {}", SyntaxError);
(function UseStrictNonDirective() {
void(0);
"use strict";
- with ({}) {};
+ with ({}) {}
})();
// Multiple directives, including "use strict".
@@ -143,7 +143,7 @@ function foo(eval) {\
function Strict() {
"use strict";
}
- with ({}) {};
+ with ({}) {}
})();
// Octal literal
@@ -555,7 +555,7 @@ repeat(10, function() {
var strict_number = 17;
var nonstrict_number = 19;
var strict_name_get = "str" + "ict" + "get";
- var nonstrict_name_get = "non" + "str" + "ict" + "get"
+ var nonstrict_name_get = "non" + "str" + "ict" + "get";
var strict_number_get = 23;
var nonstrict_number_get = 29;
@@ -592,7 +592,7 @@ repeat(10, function() {
// Set up fakes
install(String);
install(Number);
- install(Boolean)
+ install(Boolean);
function callStrict(o) {
return o.strict();
@@ -978,10 +978,10 @@ repeat(10, function() {
var seven_string = "-7-".substring(1,2);
for (var i = 0; i < 10; i ++) {
- assertThrows(function() { o[seven_smi] = "value" }, TypeError);
- assertThrows(function() { o[seven_number] = "value" }, TypeError);
- assertThrows(function() { o[seven_symbol] = "value" }, TypeError);
- assertThrows(function() { o[seven_string] = "value" }, TypeError);
+ assertThrows(function() { o[seven_smi] = "value"; }, TypeError);
+ assertThrows(function() { o[seven_number] = "value"; }, TypeError);
+ assertThrows(function() { o[seven_symbol] = "value"; }, TypeError);
+ assertThrows(function() { o[seven_string] = "value"; }, TypeError);
}
assertEquals(o[7], 17);
@@ -1038,7 +1038,7 @@ function CheckPillDescriptor(func, name) {
}
var descriptor = Object.getOwnPropertyDescriptor(func, name);
- CheckPill(descriptor.get)
+ CheckPill(descriptor.get);
CheckPill(descriptor.set);
assertFalse(descriptor.enumerable);
assertFalse(descriptor.configurable);
@@ -1107,8 +1107,8 @@ function CheckPillDescriptor(func, name) {
CheckPillDescriptor(args, "callee");
args = strict(17, "value", strict);
- assertEquals(17, args[0])
- assertEquals("value", args[1])
+ assertEquals(17, args[0]);
+ assertEquals("value", args[1]);
assertEquals(strict, args[2]);
CheckPillDescriptor(args, "caller");
CheckPillDescriptor(args, "callee");
@@ -1126,8 +1126,8 @@ function CheckPillDescriptor(func, name) {
CheckPillDescriptor(args, "callee");
args = outer()(17, "value", strict);
- assertEquals(17, args[0])
- assertEquals("value", args[1])
+ assertEquals(17, args[0]);
+ assertEquals("value", args[1]);
assertEquals(strict, args[2]);
CheckPillDescriptor(args, "caller");
CheckPillDescriptor(args, "callee");
« no previous file with comments | « test/mjsunit/stack-traces.js ('k') | test/mjsunit/strict-mode-implicit-receiver.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698