Fix assertSame for unit testing harness.

test/mjsunit/mjsunit.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 205 matching lines @@
       objectClass == "Boolean" || objectClass == "Date") {
       if (a.valueOf() !== b.valueOf()) return false;
     }
     return deepObjectEquals(a, b);
   }
 
 
   assertSame = function assertSame(expected, found, name_opt) {
     if (found === expected) {
       if (expected !== 0 || (1 / expected) == (1 / found)) return;
-    } else if (isNaN(expected) && isNaN(found)) {
+    } else if ((expected !== expected) && (found !== found)) {

[rossberg, 2011/10/28 09:05:18]

Somebody should pull IEEE to court for their NaN semantics.
Somebody should also pull to court the person responsible for the semantics of
isNaN in JavaScript.

       return;
     }
     fail(PrettyPrint(expected), found, name_opt);
   };
 
 
   assertEquals = function assertEquals(expected, found, name_opt) {
     if (!deepEquals(found, expected)) {
       fail(PrettyPrint(expected), found, name_opt);
     }
@@ skipping 109 matching lines @@
     // Fix this when we ditch the old test runner.
     var message = "Fail" + "ure: unreachable";
     if (name_opt) {
       message += " - " + name_opt;
     }
     throw new MjsUnitAssertionError(message);
   };
 
 })();