Cleanup of mjsunit.js code and make assertEquals more strict.

test/mjsunit/arguments-opt.js

 // Copyright 2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 61 matching lines @@
 // Out-of-bounds integer access with and without argument
 // adaptor frames.
 assertTrue(typeof(A(-10000)) == 'undefined');
 assertTrue(typeof(A(-10000, 0)) == 'undefined');
 assertTrue(typeof(A(-1)) == 'undefined');
 assertTrue(typeof(A(-1, 0)) == 'undefined');
 assertTrue(typeof(A(10000)) == 'undefined');
 assertTrue(typeof(A(10000, 0)) == 'undefined');
 
 // String access.
-assertEquals(0, A('0'));
-assertEquals(0, A('0',1));
+assertEquals('0', A('0'));
+assertEquals('0', A('0',1));
 assertEquals(2, A('1',2));
 assertEquals(2, A('1',2,3,4,5));
 assertEquals(5, A('4',2,3,4,5));
-assertTrue(typeof A('1') == 'undefined');
-assertTrue(typeof A('3',2,1) == 'undefined');
+assertEquals('undefined', typeof A('1'));
+assertEquals('undefined', typeof A('3',2,1));
 assertEquals(A, A('callee'));
 assertEquals(1, A('length'));
 assertEquals(2, A('length',2));
 assertEquals(5, A('length',2,3,4,5));
 assertEquals({}.toString, A('toString'));
 assertEquals({}.isPrototypeOf, A('isPrototypeOf'));
-assertTrue(typeof A('xxx') == 'undefined');
+assertEquals('undefined', typeof A('xxx'));
 
 // Object access.
 function O(key) {
   return { toString: function() { return key; } };
 }
 
-assertEquals(0, A(O(0)));
-assertEquals(0, A(O(0),1));
+var O0 = O(0);
+assertSame(O0, A(O0));
+assertSame(O0, A(O0,1));
 assertEquals(2, A(O(1),2));
 assertEquals(2, A(O(1),2,3,4,5));
 assertEquals(5, A(O(4),2,3,4,5));
 assertTrue(typeof A(O(1)) == 'undefined');
 assertTrue(typeof A(O(3),2,1) == 'undefined');
 
-assertEquals(0, A(O('0')));
-assertEquals(0, A(O('0'),1));
+O0 = O('0');
+assertSame(O0, A(O0));
+assertSame(O0, A(O0,1));
 assertEquals(2, A(O('1'),2));
 assertEquals(2, A(O('1'),2,3,4,5));
 assertEquals(5, A(O('4'),2,3,4,5));
 assertTrue(typeof A(O('1')) == 'undefined');
 assertTrue(typeof A(O('3'),2,1) == 'undefined');
 assertEquals(A, A(O('callee')));
 assertEquals(1, A(O('length')));
 assertEquals(2, A(O('length'),2));
 assertEquals(5, A(O('length'),2,3,4,5));
 assertEquals({}.toString, A(O('toString')));
 assertEquals({}.isPrototypeOf, A(O('isPrototypeOf')));
 assertTrue(typeof A(O('xxx')) == 'undefined');
 
 // Make sure that out-of-bounds access do lookups in the
 // prototype chain.
 Object.prototype[5] = 42;
 assertEquals(42, A(5));
 Object.prototype[-5] = 87;
 assertEquals(87, A(-5));