Remove CreatePlatformFile from content shell.

chrome/common/local_discovery/local_discovery_messages.h

 // Copyright 2013 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.
 
 // Defines local discovery messages between the browser and utility process.
 
+// Multiple-included file, no traditional include guard.
 #include <vector>
 
-#include "base/file_descriptor_posix.h"
 #include "chrome/common/local_discovery/service_discovery_client.h"
 #include "ipc/ipc_message_macros.h"
 
 #ifndef CHROME_COMMON_LOCAL_DISCOVERY_LOCAL_DISCOVERY_MESSAGES_H_
 #define CHROME_COMMON_LOCAL_DISCOVERY_LOCAL_DISCOVERY_MESSAGES_H_
 
 #if defined(OS_POSIX)
+#include "base/file_descriptor_posix.h"
+#endif
+

[Tom Sepez, 2014/03/12 23:02:16]

nit: two ifdef posix blocks back-to-back can be combined.

[rvargas (doing something else), 2014/03/12 23:22:00]

Right. I though about that but it feels wrong to mix the #include block with a
definition of a local struct, especially considering the include guards being
not standard in this file.

+#if defined(OS_POSIX)
 struct LocalDiscoveryMsg_SocketInfo {
   LocalDiscoveryMsg_SocketInfo()
       : address_family(net::ADDRESS_FAMILY_UNSPECIFIED),
         interface_index(0) {
   }
 
   base::FileDescriptor descriptor;
   net::AddressFamily address_family;
   uint32 interface_index;
 };
@@ skipping 98 matching lines @@
     local_discovery::ServiceResolver::RequestStatus /* status */,
     local_discovery::ServiceDescription /* description */)
 
 // Notifies browser process about local domain resolution results.
 IPC_MESSAGE_CONTROL4(
     LocalDiscoveryHostMsg_LocalDomainResolverCallback,
     uint64 /* id */,
     bool /* success */,
     net::IPAddressNumber /* ip_address_ipv4 */,
     net::IPAddressNumber /* ip_address_ipv6 */)