diff options
author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-09 23:39:22 +0000 |
---|---|---|
committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-09 23:39:22 +0000 |
commit | b32761dcb8b4118458ed3331af8ce87058c0c795 (patch) | |
tree | b3cbcdde0549f2f9032e70264adfaf47f602e15e /ppapi/native_client | |
parent | fede7212df489fd072a868d87434d6119323b978 (diff) | |
download | chromium_src-b32761dcb8b4118458ed3331af8ce87058c0c795.zip chromium_src-b32761dcb8b4118458ed3331af8ce87058c0c795.tar.gz chromium_src-b32761dcb8b4118458ed3331af8ce87058c0c795.tar.bz2 |
Pepper: Small PnaclResources cleanup
This fixes a place where the wrong URL is used for the ld process when
performing PNaCl translation. I don't believe the effect of this is severe,
though it will cause the wrong url to be displayed when it's running.
This also removes an unnecessary string constant usage in the trusted plugin.
BUG=
Review URL: https://codereview.chromium.org/373003005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282176 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/native_client')
4 files changed, 5 insertions, 6 deletions
diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_resources.cc b/ppapi/native_client/src/trusted/plugin/pnacl_resources.cc index 18eccea..13314e3 100644 --- a/ppapi/native_client/src/trusted/plugin/pnacl_resources.cc +++ b/ppapi/native_client/src/trusted/plugin/pnacl_resources.cc @@ -42,7 +42,6 @@ bool PnaclResources::ReadResourceInfo() { PP_Var pp_ld_tool_name_var; if (!plugin_->nacl_interface()->GetPnaclResourceInfo( plugin_->pp_instance(), - "chrome://pnacl-translator/pnacl.json", &pp_llc_tool_name_var, &pp_ld_tool_name_var)) { return false; diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_resources.h b/ppapi/native_client/src/trusted/plugin/pnacl_resources.h index ff70b12..c0f2968 100644 --- a/ppapi/native_client/src/trusted/plugin/pnacl_resources.h +++ b/ppapi/native_client/src/trusted/plugin/pnacl_resources.h @@ -59,5 +59,5 @@ class PnaclResources { PP_NaClFileInfo ld_file_info_; }; -} // namespace plugin; +} // namespace plugin #endif // NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_PNACL_RESOURCES_H_ diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.cc b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.cc index f6fea1e..639d494 100644 --- a/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.cc +++ b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.cc @@ -337,7 +337,7 @@ bool PnaclTranslateThread::RunLdSubprocess() { PP_NaClFileInfo ld_file_info = resources_->TakeLdFileInfo(); // On success, ownership of ld_file_info is transferred. nacl::scoped_ptr<NaClSubprocess> ld_subprocess( - plugin_->LoadHelperNaClModule(resources_->GetLlcUrl(), + plugin_->LoadHelperNaClModule(resources_->GetLdUrl(), ld_file_info, &error_info)); if (ld_subprocess.get() == NULL) { diff --git a/ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c b/ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c index 7adfbf9..f1fc368 100644 --- a/ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c +++ b/ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c @@ -3437,9 +3437,9 @@ static PP_Bool Pnacl_M25_PPB_NaCl_Private_GetManifestProgramURL(PP_Instance inst return iface->GetManifestProgramURL(instance, full_url, pnacl_options, uses_nonsfi_mode); } -static PP_Bool Pnacl_M25_PPB_NaCl_Private_GetPnaclResourceInfo(PP_Instance instance, const char* filename, struct PP_Var* llc_tool_name, struct PP_Var* ld_tool_name) { +static PP_Bool Pnacl_M25_PPB_NaCl_Private_GetPnaclResourceInfo(PP_Instance instance, struct PP_Var* llc_tool_name, struct PP_Var* ld_tool_name) { const struct PPB_NaCl_Private_1_0 *iface = Pnacl_WrapperInfo_PPB_NaCl_Private_1_0.real_iface; - return iface->GetPnaclResourceInfo(instance, filename, llc_tool_name, ld_tool_name); + return iface->GetPnaclResourceInfo(instance, llc_tool_name, ld_tool_name); } static void Pnacl_M25_PPB_NaCl_Private_GetCpuFeatureAttrs(struct PP_Var* _struct_result) { @@ -5251,7 +5251,7 @@ static const struct PPB_NaCl_Private_1_0 Pnacl_Wrappers_PPB_NaCl_Private_1_0 = { .ProcessNaClManifest = (void (*)(PP_Instance instance, const char* program_url))&Pnacl_M25_PPB_NaCl_Private_ProcessNaClManifest, .DevInterfacesEnabled = (PP_Bool (*)(PP_Instance instance))&Pnacl_M25_PPB_NaCl_Private_DevInterfacesEnabled, .GetManifestProgramURL = (PP_Bool (*)(PP_Instance instance, struct PP_Var* full_url, struct PP_PNaClOptions* pnacl_options, PP_Bool* uses_nonsfi_mode))&Pnacl_M25_PPB_NaCl_Private_GetManifestProgramURL, - .GetPnaclResourceInfo = (PP_Bool (*)(PP_Instance instance, const char* filename, struct PP_Var* llc_tool_name, struct PP_Var* ld_tool_name))&Pnacl_M25_PPB_NaCl_Private_GetPnaclResourceInfo, + .GetPnaclResourceInfo = (PP_Bool (*)(PP_Instance instance, struct PP_Var* llc_tool_name, struct PP_Var* ld_tool_name))&Pnacl_M25_PPB_NaCl_Private_GetPnaclResourceInfo, .GetCpuFeatureAttrs = (struct PP_Var (*)(void))&Pnacl_M25_PPB_NaCl_Private_GetCpuFeatureAttrs, .PostMessageToJavaScript = (void (*)(PP_Instance instance, const char* message))&Pnacl_M25_PPB_NaCl_Private_PostMessageToJavaScript, .DownloadNexe = (void (*)(PP_Instance instance, const char* url, struct PP_NaClFileInfo* file_info, struct PP_CompletionCallback callback))&Pnacl_M25_PPB_NaCl_Private_DownloadNexe, |