Tidy up to enable C4800 on Windows

minidump/minidump_module_writer_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 140 matching lines @@
     const char* expected_debug_name,
     uint32_t expected_debug_type,
     bool expected_debug_utf16) {
   if (expected_debug_name) {
     EXPECT_NE(0u, misc_record->Rva);
     const IMAGE_DEBUG_MISC* misc_debug_record =
         MinidumpWritableAtLocationDescriptor<IMAGE_DEBUG_MISC>(file_contents,
                                                                *misc_record);
     ASSERT_TRUE(misc_debug_record);
     EXPECT_EQ(expected_debug_type, misc_debug_record->DataType);
-    EXPECT_EQ(expected_debug_utf16,
-              static_cast<bool>(misc_debug_record->Unicode));
+    EXPECT_EQ(expected_debug_utf16, misc_debug_record->Unicode != 0);
     EXPECT_EQ(0u, misc_debug_record->Reserved[0]);
     EXPECT_EQ(0u, misc_debug_record->Reserved[1]);
     EXPECT_EQ(0u, misc_debug_record->Reserved[2]);
 
     // Check for the NUL terminator.
     size_t bytes_available =
         misc_debug_record->Length - offsetof(IMAGE_DEBUG_MISC, Data);
     EXPECT_EQ('\0', misc_debug_record->Data[bytes_available - 1]);
     std::string observed_data(
         reinterpret_cast<const char*>(misc_debug_record->Data));
@@ skipping 578 matching lines @@
   minidump_file_writer.AddStream(module_list_writer.Pass());
 
   StringFile string_file;
   ASSERT_DEATH_CHECK(minidump_file_writer.WriteEverything(&string_file),
                      "name_");
 }
 
 }  // namespace
 }  // namespace test
 }  // namespace crashpad