Revert 284766 "Pepper: Simplify TempFile in trusted plugin."

trunk/src/ppapi/native_client/src/trusted/plugin/temporary_file.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 "ppapi/native_client/src/trusted/plugin/temporary_file.h"
 
 #include "native_client/src/include/portability_io.h"
 #include "native_client/src/shared/platform/nacl_check.h"
 #include "native_client/src/trusted/service_runtime/include/sys/stat.h"
 
 #include "ppapi/cpp/core.h"
 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/module.h"
 #include "ppapi/c/private/pp_file_handle.h"
 
 #include "ppapi/native_client/src/trusted/plugin/plugin.h"
 #include "ppapi/native_client/src/trusted/plugin/utility.h"
 
 namespace plugin {
 
-TempFile::TempFile(Plugin* plugin, PP_FileHandle handle)
-    : plugin_(plugin), internal_handle_(handle) { }
+TempFile::TempFile(Plugin* plugin) : plugin_(plugin),
+                                     internal_handle_(PP_kInvalidFileHandle) {
+}
 
 TempFile::~TempFile() { }
 
 int32_t TempFile::Open(bool writeable) {
+  // TODO(teravest): Clean up this Open() behavior; this is really confusing as
+  // written.
+  if (internal_handle_ == PP_kInvalidFileHandle) {
+    internal_handle_ =
+        plugin_->nacl_interface()->CreateTemporaryFile(plugin_->pp_instance());
+  }
+
   if (internal_handle_ == PP_kInvalidFileHandle) {
     PLUGIN_PRINTF(("TempFile::Open failed w/ PP_kInvalidFileHandle\n"));
     return PP_ERROR_FAILED;
   }
 
 #if NACL_WINDOWS
   HANDLE handle = internal_handle_;
 
   //////// Now try the posix view.
   int rdwr_flag = writeable ? _O_RDWR : _O_RDONLY;
@@ skipping 43 matching lines @@
 
 PP_FileHandle TempFile::TakeFileHandle() {
   PP_FileHandle to_return = internal_handle_;
   internal_handle_ = PP_kInvalidFileHandle;
   read_wrapper_.release();
   write_wrapper_.release();
   return to_return;
 }
 
 }  // namespace plugin