Fix more -Wsign-compare warnings in heap, mips, base, etc.

test/cctest/libplatform/test-tracing.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 #include <limits>
 
 #include "include/libplatform/v8-tracing.h"
 #include "src/tracing/trace-event.h"
 #include "test/cctest/cctest.h"
 
 namespace v8 {
@@ skipping 20 matching lines @@
 TEST(TestTraceObject) {
   TraceObject trace_object;
   uint8_t category_enabled_flag = 41;
   trace_object.Initialize('X', &category_enabled_flag, "Test.Trace",
                           "Test.Scope", 42, 123, 0, nullptr, nullptr, nullptr,
                           nullptr, 0);
   CHECK_EQ('X', trace_object.phase());
   CHECK_EQ(category_enabled_flag, *trace_object.category_enabled_flag());
   CHECK_EQ(std::string("Test.Trace"), std::string(trace_object.name()));
   CHECK_EQ(std::string("Test.Scope"), std::string(trace_object.scope()));
-  CHECK_EQ(0, trace_object.duration());
-  CHECK_EQ(0, trace_object.cpu_duration());
+  CHECK_EQ(0u, trace_object.duration());
+  CHECK_EQ(0u, trace_object.cpu_duration());
 }
 
 class ConvertableToTraceFormatMock : public v8::ConvertableToTraceFormat {
  public:
   explicit ConvertableToTraceFormatMock(int value) : value_(value) {}
   void AppendAsTraceFormat(std::string* out) const override {
     *out += "[" + std::to_string(value_) + "," + std::to_string(value_) + "]";
   }
 
  private:
@@ skipping 126 matching lines @@
   TraceConfig* trace_config = new TraceConfig();
   trace_config->AddIncludedCategory("v8");
   tracing_controller.StartTracing(trace_config);
 
   TRACE_EVENT0("v8", "v8.Test");
   // cat category is not included in default config
   TRACE_EVENT0("cat", "v8.Test2");
   TRACE_EVENT0("v8", "v8.Test3");
   tracing_controller.StopTracing();
 
-  CHECK_EQ(2, writer->events().size());
+  CHECK_EQ(2u, writer->events().size());
   CHECK_EQ(std::string("v8.Test"), writer->events()[0]);
   CHECK_EQ(std::string("v8.Test3"), writer->events()[1]);
 
   i::V8::SetPlatformForTesting(old_platform);
 }
 
 void GetJSONStrings(std::vector<std::string>& ret, std::string str,
                     std::string param, std::string start_delim,
                     std::string end_delim) {
   size_t pos = str.find(param);
@@ skipping 87 matching lines @@
     tracing_controller.StopTracing();
   }
 
   std::string trace_str = stream.str();
 
   std::vector<std::string> all_args, all_names, all_cats;
   GetJSONStrings(all_args, trace_str, "\"args\"", "{", "}");
   GetJSONStrings(all_names, trace_str, "\"name\"", "\"", "\"");
   GetJSONStrings(all_cats, trace_str, "\"cat\"", "\"", "\"");
 
-  CHECK_EQ(all_args.size(), 24);
+  CHECK_EQ(all_args.size(), 24u);
   CHECK_EQ(all_args[0], "\"aa\":11");
   CHECK_EQ(all_args[1], "\"bb\":22");
   CHECK_EQ(all_args[2], "\"cc\":33");
   CHECK_EQ(all_args[3], "\"dd\":44");
   CHECK_EQ(all_args[4], "\"ee\":-55");
   CHECK_EQ(all_args[5], "\"ff\":-66");
   CHECK_EQ(all_args[6], "\"gg\":-77");
   CHECK_EQ(all_args[7], "\"hh\":-88");
   CHECK_EQ(all_args[8], "\"ii1\":true");
   CHECK_EQ(all_args[9], "\"ii2\":false");
@@ skipping 90 matching lines @@
 
   CHECK_EQ(1, observer.enabled_count);
   CHECK_EQ(1, observer.disabled_count);
 
   i::V8::SetPlatformForTesting(old_platform);
 }
 
 }  // namespace tracing
 }  // namespace platform
 }  // namespace v8