Fixes for re-enabling more MSVC level 4 warnings: content/renderer/ edition

content/renderer/pepper/pepper_broker.cc

 // Copyright (c) 2012 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 "content/renderer/pepper/pepper_broker.h"
 
 #include "build/build_config.h"
 #include "content/renderer/pepper/pepper_proxy_channel_delegate_impl.h"
 #include "content/renderer/pepper/plugin_module.h"
 #include "content/renderer/pepper/ppb_broker_impl.h"
@@ skipping 70 matching lines @@
 
 // Does not take ownership of the local pipe.
 int32_t PepperBrokerDispatcherWrapper::SendHandleToBroker(
     PP_Instance instance,
     base::SyncSocket::Handle handle) {
   IPC::PlatformFileForTransit foreign_socket_handle =
       dispatcher_->ShareHandleWithRemote(handle, false);
   if (foreign_socket_handle == IPC::InvalidPlatformFileForTransit())
     return PP_ERROR_FAILED;
 
-  int32_t result;
+  int32_t result = PP_ERROR_FAILED;
   if (!dispatcher_->Send(new PpapiMsg_ConnectToPlugin(
           instance, foreign_socket_handle, &result))) {
     // The plugin did not receive the handle, so it must be closed.
     // The easiest way to clean it up is to just put it in an object
     // and then close it. This failure case is not performance critical.
     // The handle could still leak if Send succeeded but the IPC later failed.
     base::SyncSocket temp_socket(
         IPC::PlatformFileForTransitToPlatformFile(foreign_socket_handle));
     return PP_ERROR_FAILED;
   }
@@ skipping 145 matching lines @@
 
   // TOOD(ddorwin): Change the IPC to asynchronous: Queue an object containing
   // client and plugin_socket.release(), then return.
   // That message handler will then call client->BrokerConnected() with the
   // saved pipe handle.
   // Temporarily, just call back.
   client->BrokerConnected(ppapi::PlatformFileToInt(plugin_handle), result);
 }
 
 }  // namespace content