Remove arguments arity check in mjsunit because it vastly slows down the tests.
authorBen L. Titzer <titzer@chromium.org>
Fri, 14 Nov 2014 10:30:12 +0000 (11:30 +0100)
committerBen L. Titzer <titzer@chromium.org>
Fri, 14 Nov 2014 10:30:25 +0000 (10:30 +0000)
R=mstarzinger@chromium.org
BUG=

Review URL: https://codereview.chromium.org/724133004

Cr-Commit-Position: refs/heads/master@{#25353}

test/mjsunit/mjsunit.js

index 0430279..b360425 100644 (file)
@@ -231,16 +231,7 @@ var assertUnoptimized;
     return deepObjectEquals(a, b);
   }
 
-  function checkArity(args, arity, name) {
-    if (args.length < arity) {
-      fail(PrettyPrint(arity), args.length,
-           name + " requires " + arity + " or more arguments");
-    }
-  }
-
   assertSame = function assertSame(expected, found, name_opt) {
-    checkArity(arguments, 2, "assertSame");
-
     // TODO(mstarzinger): We should think about using Harmony's egal operator
     // or the function equivalent Object.is() here.
     if (found === expected) {
@@ -253,8 +244,6 @@ var assertUnoptimized;
 
 
   assertEquals = function assertEquals(expected, found, name_opt) {
-    checkArity(arguments, 2, "assertEquals");
-
     if (!deepEquals(found, expected)) {
       fail(PrettyPrint(expected), found, name_opt);
     }