Revert of Pepper: Refactor OpenManifestEntry.

components/nacl/renderer/ppb_nacl_private_impl.cc

 // 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.
 
 #include "components/nacl/renderer/ppb_nacl_private_impl.h"
 
 #include <numeric>
 #include <string>
 #include <vector>
 
@@ skipping 1062 matching lines @@
     *pp_full_url = ppapi::StringVar::StringToPPVar(full_url);
     *pp_uses_nonsfi_mode = PP_FromBool(uses_nonsfi_mode);
     return PP_TRUE;
   }
 
   if (load_manager)
     load_manager->ReportLoadError(error_info.error, error_info.string);
   return PP_FALSE;
 }
 
-// TODO(teravest): Refactor DownloadFile/DownloadNexe out to their own file;
-// this one is getting way too messy.
-void DownloadFile(PP_Instance instance,
-                  const std::string& url,
-                  struct PP_NaClFileInfo* file_info,
-                  struct PP_CompletionCallback callback);
-
-void OpenManifestEntry(PP_Instance instance,
-                       PP_Bool is_helper_process,
-                       const char* key,
-                       PP_NaClFileInfo* out_file_info,
-                       PP_CompletionCallback callback) {
-  std::string resolved_url;
-
+PP_Bool ManifestResolveKey(PP_Instance instance,
+                           PP_Bool is_helper_process,
+                           const char* key,
+                           PP_Var* pp_full_url,
+                           PP_PNaClOptions* pnacl_options) {
   // For "helper" processes (llc and ld), we resolve keys manually as there is
   // no existing .nmf file to parse.
   if (PP_ToBool(is_helper_process)) {
+    pnacl_options->translate = PP_FALSE;
     // We can only resolve keys in the files/ namespace.
     const std::string kFilesPrefix = "files/";
     std::string key_string(key);
     if (key_string.find(kFilesPrefix) == std::string::npos) {
       nacl::NexeLoadManager* load_manager = GetNexeLoadManager(instance);
       if (load_manager)
         load_manager->ReportLoadError(PP_NACL_ERROR_MANIFEST_RESOLVE_URL,
                                       "key did not start with files/");
-      ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
-          FROM_HERE,
-          base::Bind(callback.func, callback.user_data,
-                     static_cast<int32_t>(PP_ERROR_FAILED)));
-      return;
+      return PP_FALSE;
     }
     std::string key_basename = key_string.substr(kFilesPrefix.length());
-    resolved_url =
+    std::string pnacl_url =
         std::string(kPNaClTranslatorBaseUrl) + GetSandboxArch() + "/" +
         key_basename;
-  } else {
-    JsonManifestMap::iterator it = g_manifest_map.Get().find(instance);
-    if (it == g_manifest_map.Get().end()) {
-      ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
-          FROM_HERE,
-          base::Bind(callback.func, callback.user_data,
-                     static_cast<int32_t>(PP_ERROR_FAILED)));
-      return;
-    }
-
-    PP_PNaClOptions pnacl_options;
-    bool ok = it->second->ResolveKey(key, &resolved_url, &pnacl_options);
-    // We don't support OpenManifestEntry for files that require PNaCl
-    // translation.
-    if (!ok || pnacl_options.translate) {
-      ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
-          FROM_HERE,
-          base::Bind(callback.func, callback.user_data,
-                     static_cast<int32_t>(PP_ERROR_FAILED)));
-      return;
-    }
+    *pp_full_url = ppapi::StringVar::StringToPPVar(pnacl_url);
+    return PP_TRUE;
   }
 
-  // Hand off to DownloadFile to perform fetching the actual file.
-  DownloadFile(instance, resolved_url, out_file_info, callback);
+  JsonManifestMap::iterator it = g_manifest_map.Get().find(instance);
+  if (it == g_manifest_map.Get().end())
+    return PP_FALSE;
+
+  std::string full_url;
+  bool ok = it->second->ResolveKey(key, &full_url, pnacl_options);
+  if (ok)
+    *pp_full_url = ppapi::StringVar::StringToPPVar(full_url);
+  return PP_FromBool(ok);
 }
 
 PP_Bool GetPNaClResourceInfo(PP_Instance instance,
                              const char* filename,
                              PP_Var* llc_tool_name,
                              PP_Var* ld_tool_name) {
   NexeLoadManager* load_manager = GetNexeLoadManager(instance);
   DCHECK(load_manager);
   if (!load_manager)
     return PP_FALSE;
@@ skipping 292 matching lines @@
   int32_t pp_error = FileDownloaderToPepperError(status);
   if (pp_error == PP_OK) {
     file_info->handle = file;
     file_info->token_lo = 0;
     file_info->token_hi = 0;
   }
   callback.func(callback.user_data, pp_error);
 }
 
 void DownloadFile(PP_Instance instance,
-                  const std::string& url,
+                  const char* url,
                   struct PP_NaClFileInfo* file_info,
                   struct PP_CompletionCallback callback) {
+  CHECK(url);
   CHECK(file_info);
 
   NexeLoadManager* load_manager = GetNexeLoadManager(instance);
   DCHECK(load_manager);
   if (!load_manager) {
     ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
         FROM_HERE,
         base::Bind(callback.func, callback.user_data,
                    static_cast<int32_t>(PP_ERROR_FAILED)));
     return;
   }
 
   // Handle special PNaCl support files which are installed on the user's
   // machine.
-  if (url.find(kPNaClTranslatorBaseUrl, 0) == 0) {
-    PP_FileHandle handle = GetReadonlyPnaclFd(url.c_str());
+  std::string url_string(url);
+  if (url_string.find(kPNaClTranslatorBaseUrl, 0) == 0) {
+    PP_FileHandle handle = GetReadonlyPnaclFd(url);
     if (handle == PP_kInvalidFileHandle) {
       ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
           FROM_HERE,
           base::Bind(callback.func, callback.user_data,
                      static_cast<int32_t>(PP_ERROR_FAILED)));
       return;
     }
     // TODO(ncbray): enable the fast loading and validation paths for this type
     // of file.
     file_info->handle = handle;
@@ skipping 14 matching lines @@
         FROM_HERE,
         base::Bind(callback.func, callback.user_data,
                    static_cast<int32_t>(PP_ERROR_FAILED)));
     return;
   }
 
   // Try the fast path for retrieving the file first.
   uint64_t file_token_lo = 0;
   uint64_t file_token_hi = 0;
   PP_FileHandle file_handle = OpenNaClExecutable(instance,
-                                                 url.c_str(),
+                                                 url,
                                                  &file_token_lo,
                                                  &file_token_hi);
   if (file_handle != PP_kInvalidFileHandle) {
     file_info->handle = file_handle;
     file_info->token_lo = file_token_lo;
     file_info->token_hi = file_token_hi;
     ppapi::PpapiGlobals::Get()->GetMainThreadMessageLoop()->PostTask(
         FROM_HERE,
         base::Bind(callback.func, callback.user_data,
                    static_cast<int32_t>(PP_OK)));
@@ skipping 59 matching lines @@
   &SetExitStatus,
   &Vlog,
   &InitializePlugin,
   &GetNexeSize,
   &RequestNaClManifest,
   &GetManifestBaseURL,
   &ProcessNaClManifest,
   &GetManifestURLArgument,
   &DevInterfacesEnabled,
   &ManifestGetProgramURL,
-  &OpenManifestEntry,
+  &ManifestResolveKey,
   &GetPNaClResourceInfo,
   &GetCpuFeatureAttrs,
   &PostMessageToJavaScript,
   &DownloadNexe,
+  &DownloadFile
 };
 
 }  // namespace
 
 const PPB_NaCl_Private* GetNaClPrivateInterface() {
   return &nacl_interface;
 }
 
 }  // namespace nacl