Add an interface for nacl to create delete-on-close temp files,

chrome/browser/nacl_host/pnacl_file_host.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 "chrome/browser/nacl_host/pnacl_file_host.h"
 
 #include "base/bind.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
@@ skipping 58 matching lines @@
                                    true /* Close source */);
   if (target_desc == IPC::InvalidPlatformFileForTransit()) {
     NotifyRendererOfError(chrome_render_message_filter, reply_msg);
     return;
   }
   ChromeViewHostMsg_GetReadonlyPnaclFD::WriteReplyParams(
       reply_msg, target_desc);
   chrome_render_message_filter->Send(reply_msg);
 }
 
+void DoCreateTemporaryFile(
+    ChromeRenderMessageFilter* chrome_render_message_filter,
+    IPC::Message* reply_msg) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+
+  FilePath file_path;
+  if (!file_util::CreateTemporaryFile(&file_path)) {
+    NotifyRendererOfError(chrome_render_message_filter, reply_msg);
+    return;
+  }
+
+  base::PlatformFileError error;
+  base::PlatformFile file_handle = base::CreatePlatformFile(
+      file_path,
+      base::PLATFORM_FILE_CREATE_ALWAYS | base::PLATFORM_FILE_READ |
+      base::PLATFORM_FILE_WRITE | base::PLATFORM_FILE_TEMPORARY |
+      base::PLATFORM_FILE_DELETE_ON_CLOSE,
+      NULL, &error);
+
+  if (error != base::PLATFORM_FILE_OK) {
+    NotifyRendererOfError(chrome_render_message_filter, reply_msg);
+    return;
+  }
+
+  // Also unlink the temp file immediately on posix systems.
+#if defined(OS_POSIX)
+  if (!file_util::Delete(file_path, false)) {

[brettw, 2012/07/27 22:38:17]

Unless I'm mistaken, it looks like this is already done in CreatePlatformFile
when you specify DELETE_ON_CLOSE.

[jvoung - send to chromium..., 2012/07/28 01:30:05]

Oops make sense -- removed.

+    NotifyRendererOfError(chrome_render_message_filter, reply_msg);
+    return;
+  }
+#endif
+
+  // Send the reply!
+  // Do any DuplicateHandle magic that is necessary first.
+  IPC::PlatformFileForTransit target_desc =
+      IPC::GetFileHandleForProcess(file_handle,
+                                   chrome_render_message_filter->peer_handle(),
+                                   true);
+  if (target_desc == IPC::InvalidPlatformFileForTransit()) {
+    NotifyRendererOfError(chrome_render_message_filter, reply_msg);
+    return;
+  }
+
+  ChromeViewHostMsg_NaClCreateTemporaryFile::WriteReplyParams(
+      reply_msg, target_desc);
+  chrome_render_message_filter->Send(reply_msg);
+}
+
 }  // namespace
 
 namespace pnacl_file_host {
 
 void GetReadonlyPnaclFd(
     ChromeRenderMessageFilter* chrome_render_message_filter,
     const std::string& filename,
     IPC::Message* reply_msg) {
   if (!BrowserThread::PostTask(
           BrowserThread::FILE, FROM_HERE,
@@ skipping 39 matching lines @@
   }
   if (pnacl_dir.empty()) {
     return false;
   }
 
   FilePath full_path = pnacl_dir.Append(file_to_find);
   *file_to_open = full_path;
   return true;
 }
 
+void CreateTemporaryFile(
+    ChromeRenderMessageFilter* chrome_render_message_filter,
+    IPC::Message* reply_msg) {
+  if (!BrowserThread::PostTask(
+          BrowserThread::FILE, FROM_HERE,
+          base::Bind(&DoCreateTemporaryFile,
+                     make_scoped_refptr(chrome_render_message_filter),
+                     reply_msg))) {
+    NotifyRendererOfError(chrome_render_message_filter, reply_msg);
+  }
+}
+
 }  // namespace pnacl_file_host