Use size_t in GCIdleTimeHandler to fix undefined behaviour.

test/heap-unittests/heap-unittest.cc

Index: test/heap-unittests/heap-unittest.cc
diff --git a/test/heap-unittests/heap-unittest.cc b/test/heap-unittests/heap-unittest.cc
index 220d8f2dffb7350579e9d489780eb58cc81f53d5..db331f91e068089b78c748a197547cd4864b22ac 100644
--- a/test/heap-unittests/heap-unittest.cc
+++ b/test/heap-unittests/heap-unittest.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include <climits>
+#include <limits>
 
 #include "src/heap/gc-idle-time-handler.h"
 
@@ -12,33 +12,37 @@ namespace v8 {
 namespace internal {
 
 TEST(EstimateMarkingStepSizeTest, EstimateMarkingStepSizeInitial) {
-  intptr_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(1, 0);
-  EXPECT_EQ(static_cast<intptr_t>(
-                GCIdleTimeHandler::kInitialConservativeMarkingSpeed *
-                GCIdleTimeHandler::kConservativeTimeRatio),
-            step_size);
+  size_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(1, 0);
+  EXPECT_EQ(
+      static_cast<size_t>(GCIdleTimeHandler::kInitialConservativeMarkingSpeed *
+                          GCIdleTimeHandler::kConservativeTimeRatio),
+      step_size);
 }
 
 
 TEST(EstimateMarkingStepSizeTest, EstimateMarkingStepSizeNonZero) {
-  intptr_t marking_speed_in_bytes_per_millisecond = 100;
-  intptr_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(
+  size_t marking_speed_in_bytes_per_millisecond = 100;
+  size_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(
       1, marking_speed_in_bytes_per_millisecond);
-  EXPECT_EQ(static_cast<intptr_t>(marking_speed_in_bytes_per_millisecond *
-                                  GCIdleTimeHandler::kConservativeTimeRatio),
+  EXPECT_EQ(static_cast<size_t>(marking_speed_in_bytes_per_millisecond *
+                                GCIdleTimeHandler::kConservativeTimeRatio),
             step_size);
 }
 
 
 TEST(EstimateMarkingStepSizeTest, EstimateMarkingStepSizeOverflow1) {
-  intptr_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(10, INT_MAX);
-  EXPECT_EQ(INT_MAX, step_size);
+  size_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(
+      10, std::numeric_limits<size_t>::max());
+  EXPECT_EQ(static_cast<size_t>(GCIdleTimeHandler::kMaximumMarkingStepSize),
+            step_size);
 }
 
 
 TEST(EstimateMarkingStepSizeTest, EstimateMarkingStepSizeOverflow2) {
-  intptr_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(INT_MAX, 10);
-  EXPECT_EQ(INT_MAX, step_size);
+  size_t step_size = GCIdleTimeHandler::EstimateMarkingStepSize(
+      std::numeric_limits<size_t>::max(), 10);
+  EXPECT_EQ(static_cast<size_t>(GCIdleTimeHandler::kMaximumMarkingStepSize),
+            step_size);
 }
 
 }  // namespace internal