Implement a proxy for Pepper FileIO.

ppapi/tests/test_url_loader.cc

 // Copyright (c) 2011 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/tests/test_url_loader.h"
 
 #include <stdio.h>
 #include <string.h>
 #include <string>
 
@@ skipping 53 matching lines @@
 }
 
 bool TestURLLoader::Init() {
   if (!InitTestingInterface()) {
     instance_->AppendError("Testing interface not available");
     return false;
   }
 
   const PPB_FileIO* file_io_interface = static_cast<const PPB_FileIO*>(
       pp::Module::Get()->GetBrowserInterface(PPB_FILEIO_INTERFACE));
-  if (testing_interface_->IsOutOfProcess() && file_io_interface) {
-    instance_->AppendError(
-        "FileIO interface is now supported by ppapi proxy: update this test!");
-  } else if (!testing_interface_->IsOutOfProcess() && !file_io_interface) {
+  if (!file_io_interface)
     instance_->AppendError("FileIO interface not available");
-  }
 
   file_io_trusted_interface_ = static_cast<const PPB_FileIOTrusted*>(
       pp::Module::Get()->GetBrowserInterface(PPB_FILEIOTRUSTED_INTERFACE));
   url_loader_trusted_interface_ = static_cast<const PPB_URLLoaderTrusted*>(
       pp::Module::Get()->GetBrowserInterface(PPB_URLLOADERTRUSTED_INTERFACE));
-  if  (testing_interface_->IsOutOfProcess() && file_io_trusted_interface_) {
-    instance_->AppendError(
-        "FileIOTrusted interface is now supported by ppapi proxy: "
-        "update this test!");
-  } else if (!testing_interface_->IsOutOfProcess()) {
+  if (!testing_interface_->IsOutOfProcess()) {
     // Trusted interfaces are not supported under NaCl.
 #if !(defined __native_client__)
     if (!file_io_trusted_interface_)
       instance_->AppendError("FileIOTrusted interface not available");
     if (!url_loader_trusted_interface_)
       instance_->AppendError("URLLoaderTrusted interface not available");
 #else
     if (file_io_trusted_interface_)
       instance_->AppendError("FileIOTrusted interface is supported by NaCl");
     if (url_loader_trusted_interface_)
@@ skipping 363 matching lines @@
 
   rv = loader.FinishStreamingToFile(callback);
   if (force_async_ && rv != PP_OK_COMPLETIONPENDING)
     return ReportError("URLLoader::FinishStreamingToFile force_async", rv);
   if (rv == PP_OK_COMPLETIONPENDING)
     rv = callback.WaitForResult();
   if (rv != PP_OK)
     return ReportError("URLLoader::FinishStreamingToFile", rv);
 
   // FileIO is not yet supported by ppapi/proxy.
+  pp::FileIO reader(instance_);
+  rv = reader.Open(body, PP_FILEOPENFLAG_READ, callback);
+  if (force_async_ && rv != PP_OK_COMPLETIONPENDING)
+    return ReportError("FileIO::Open force_async", rv);
+  if (rv == PP_OK_COMPLETIONPENDING)
+    rv = callback.WaitForResult();
+  if (rv != PP_OK)
+    return ReportError("FileIO::Open", rv);
+
+  std::string data;
+  std::string error = ReadEntireFile(&reader, &data);
+  if (!error.empty())
+    return error;
+
+  std::string expected_body = "hello\n";
+  if (data.size() != expected_body.size())
+    return "ReadEntireFile returned unexpected content length";
+  if (data != expected_body)
+    return "ReadEntireFile returned unexpected content";
+
+  // FileIOTrusted is not supported by NaCl or ppapi/proxy.
   if (!testing_interface_->IsOutOfProcess()) {
-    pp::FileIO reader(instance_);
-    rv = reader.Open(body, PP_FILEOPENFLAG_READ, callback);
-    if (force_async_ && rv != PP_OK_COMPLETIONPENDING)
-      return ReportError("FileIO::Open force_async", rv);
-    if (rv == PP_OK_COMPLETIONPENDING)
-      rv = callback.WaitForResult();
-    if (rv != PP_OK)
-      return ReportError("FileIO::Open", rv);
-
-    std::string data;
-    std::string error = ReadEntireFile(&reader, &data);
-    if (!error.empty())
-      return error;
-
-    std::string expected_body = "hello\n";
-    if (data.size() != expected_body.size())
-      return "ReadEntireFile returned unexpected content length";
-    if (data != expected_body)
-      return "ReadEntireFile returned unexpected content";
-
-    // FileIOTrusted is not supported by NaCl or ppapi/proxy.
-    if (!testing_interface_->IsOutOfProcess()) {
 #if !(defined __native_client__)
-      int32_t file_descriptor = file_io_trusted_interface_->GetOSFileDescriptor(
-          reader.pp_resource());
-      if (file_descriptor < 0)
-        return "FileIO::GetOSFileDescriptor() returned a bad file descriptor.";
+    int32_t file_descriptor = file_io_trusted_interface_->GetOSFileDescriptor(
+        reader.pp_resource());
+    if (file_descriptor < 0)
+      return "FileIO::GetOSFileDescriptor() returned a bad file descriptor.";
 #endif
-    }
   }
   PASS();
 }
 
 // Untrusted, unintended cross-origin requests should fail.
 std::string TestURLLoader::TestUntrustedSameOriginRestriction() {
   pp::URLRequestInfo request(instance_);
   std::string cross_origin_url = GetReachableCrossOriginURL();
   request.SetURL(cross_origin_url);
 
@@ skipping 325 matching lines @@
   if (!error.empty())
     return error;
   if (body != "hello\n")
     return ReportError("Couldn't read data", rv);
 
   PASS();
 }
 
 // TODO(viettrungluu): Add tests for FollowRedirect,
 // Get{Upload,Download}Progress, Close (including abort tests if applicable).