win: Disable some more warnings when not building with Crashpad's common.gypi

client/simple_string_dictionary_test.cc

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 154 matching lines @@
   // We then verify that it iterates through exactly the number of key/value
   // pairs we expect, and that they match one-for-one with what we would expect.
   // The ordering of the iteration does not matter...
 
   // used to keep track of number of occurrences found for key/value pairs
   int count[kDictionaryCapacity];
   memset(count, 0, sizeof(count));
 
   int totalCount = 0;
 
-  const SimpleStringDictionary::Entry* entry;
-  while ((entry = iter.Next())) {
+  for (;;) {
+    const SimpleStringDictionary::Entry* entry = iter.Next();
+    if (!entry)
+      break;
     totalCount++;
 
     // Extract keyNumber from a string of the form key<keyNumber>
     int keyNumber;
     sscanf(entry->key, "key%d", &keyNumber);
 
     // Extract valueNumber from a string of the form value<valueNumber>
     int valueNumber;
     sscanf(entry->value, "value%d", &valueNumber);
 
@@ skipping 102 matching lines @@
   ASSERT_DEATH_CHECK(map.GetValueForKey(nullptr), "key");
   map.RemoveKey("hi");
   EXPECT_EQ(0u, map.GetCount());
 }
 
 #endif
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad