Change custom NaN check to use isnan to fix pixel array...

src/objects.cc

 // Copyright 2006-2009 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 7042 matching lines @@
 }
 
 
 Object* PixelArray::SetValue(uint32_t index, Object* value) {
   uint8_t clamped_value = 0;
   if (index < static_cast<uint32_t>(length())) {
     int int_value = 0;
     if (value->IsSmi()) {
       int_value = Smi::cast(value)->value();
     } else if (value->IsHeapNumber()) {
-      static const DoubleRepresentation nan(OS::nan_value());
-      DoubleRepresentation double_value = HeapNumber::cast(value)->value();
-      if (nan.bits != double_value.bits) {
-        int_value = static_cast<int>(double_value.value + 0.5);
-      } else {
-        // NaN clamps to zero.
-        int_value = 0;
+      double double_value = HeapNumber::cast(value)->value();
+      if (!isnan(double_value)) {
+        // NaN clamps to zero (default). Other doubles are rounded to
+        // the nearest integer.
+        int_value = static_cast<int>(double_value + 0.5);
       }
-    } else if (value->IsUndefined()) {
-      int_value = 0;
     } else {
-      // All other types have been converted to a number type further up in the
-      // call chain.
-      UNREACHABLE();
+      // Clamp undefined to zero (default). All other types have been
+      // converted to a number type further up in the call chain.
+      ASSERT(value->IsUndefined());
     }
     if (int_value < 0) {
       clamped_value = 0;
     } else if (int_value > 255) {
       clamped_value = 255;
     } else {
       clamped_value = static_cast<uint8_t>(int_value);
     }
     set(index, clamped_value);
   }
@@ skipping 910 matching lines @@
   if (break_point_objects()->IsUndefined()) return 0;
   // Single beak point.
   if (!break_point_objects()->IsFixedArray()) return 1;
   // Multiple break points.
   return FixedArray::cast(break_point_objects())->length();
 }
 #endif
 
 
 } }  // namespace v8::internal