Enable warning 4389 as an error on windows builds. This will make...

chrome_frame/chrome_launcher.cc

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium 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 "chrome_frame/chrome_launcher.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 46 matching lines @@
           "What module is this? It's not in 'servers' or main output dir.";
       return new CommandLine(servers_path);
     }
   } else {
     NOTREACHED();
     return NULL;
   }
 }
 
 void SanitizeCommandLine(const CommandLine& original, CommandLine* sanitized) {
-  int num_sanitized_switches = 0;
+  size_t num_sanitized_switches = 0;
   for (int i = 0; i < arraysize(kAllowedSwitches); ++i) {
     const char* current_switch = kAllowedSwitches[i];
     if (original.HasSwitch(current_switch)) {
       ++num_sanitized_switches;
       std::wstring switch_value = original.GetSwitchValue(current_switch);
       if (0 == switch_value.length()) {
         sanitized->AppendSwitch(current_switch);
       } else {
         sanitized->AppendSwitchWithValue(current_switch, switch_value);
       }
@@ skipping 52 matching lines @@
 
   return result;
 }
 
 // Compile-time check to see that the type CfLaunchChromeProc is correct.
 #ifndef NODEBUG
 namespace {
 chrome_launcher::CfLaunchChromeProc cf_launch_chrome = CfLaunchChrome;
 }  // namespace
 #endif  // NODEBUG