Make PlatformFileForTransit a class on Windows.

components/nacl/renderer/manifest_service_channel.cc

 // Copyright 2014 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 "components/nacl/renderer/manifest_service_channel.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 71 matching lines @@
 }
 
 void ManifestServiceChannel::DidOpenResource(IPC::Message* reply,
                                              base::File file,
                                              uint64_t token_lo,
                                              uint64_t token_hi) {
   ppapi::proxy::SerializedHandle handle;
   if (file.IsValid()) {
     IPC::PlatformFileForTransit file_for_transit;
 #if defined(OS_WIN)
+    HANDLE raw_handle = INVALID_HANDLE_VALUE;
     bool ok = content::BrokerDuplicateHandle(

[Mark Seaborn, 2016/03/29 19:24:09]

So this case would get converted by a later change?

[erikchen, 2016/03/29 20:57:42]

yes. Trying to keep this change as small as possible.

-        file.TakePlatformFile(),
-        peer_pid_,
-        &file_for_transit,
+        file.TakePlatformFile(), peer_pid_, &raw_handle,
         0,  // desired_access is 0 since we're using DUPLICATE_SAME_ACCESS.
         DUPLICATE_SAME_ACCESS | DUPLICATE_CLOSE_SOURCE);
-    if (ok)
+    if (ok) {
+      file_for_transit = IPC::PlatformFileForTransit(raw_handle, peer_pid_);
       handle.set_file_handle(file_for_transit, PP_FILEOPENFLAG_READ, 0);
+    }
 #else
     file_for_transit = base::FileDescriptor(std::move(file));
     handle.set_file_handle(file_for_transit, PP_FILEOPENFLAG_READ, 0);
 #endif
   }
   PpapiHostMsg_OpenResource::WriteReplyParams(reply,
                                               handle,
                                               token_lo,
                                               token_hi);
   Send(reply);
 }
 
 }  // namespace nacl