commit 67012993060a3f5d2a4786f996fc92fa0449c87b
author fayang <fayang@google.com> Tue Jul 30 12:02:42 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jul 30 12:03:33 2019 -0700
tree 23347b632bf7c576df034d145b2a3cac62163bd8
parent 118c8accd58fde6fa562b0134b9425a926738325

Fix comparison of signed to unsigned in htt2_priority_write_scheduler_* and spdy_intrusive_list_test.

gfe-relnote: (n/a) Test only change, not protected.
PiperOrigin-RevId: 260766208
Change-Id: Ibb269b9d84209a9997c43a8312d6decfb087a8e2

spdy/core/http2_priority_write_scheduler.h

diff --git a/spdy/core/http2_priority_write_scheduler.h b/spdy/core/http2_priority_write_scheduler.h
index f89dc8d..f64d92b 100644
--- a/spdy/core/http2_priority_write_scheduler.h
+++ b/spdy/core/http2_priority_write_scheduler.h
@@ -406,7 +406,7 @@
   }
 
   if (stream_info->parent == new_parent &&
-      (!exclusive || new_parent->children.size() == 1)) {
+      (!exclusive || new_parent->children.size() == 1u)) {
     // If the new parent is already the stream's parent, and exclusivity (if
     // specified) is already satisfied, we are done.
     return;
@@ -723,8 +723,8 @@
 template <typename StreamIdType>
 bool Http2PriorityWriteScheduler<StreamIdType>::ValidateInvariantsForTests()
     const {
-  int total_streams = 0;
-  int streams_visited = 0;
+  size_t total_streams = 0;
+  size_t streams_visited = 0;
   // Iterate through all streams in the map.
   for (const auto& kv : all_stream_infos_) {
     ++total_streams;