summaryrefslogtreecommitdiffstats
path: root/ppapi/native_client
diff options
context:
space:
mode:
authorteravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-07-09 23:39:22 +0000
committerteravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-07-09 23:39:22 +0000
commitb32761dcb8b4118458ed3331af8ce87058c0c795 (patch)
treeb3cbcdde0549f2f9032e70264adfaf47f602e15e /ppapi/native_client
parentfede7212df489fd072a868d87434d6119323b978 (diff)
downloadchromium_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')
-rw-r--r--ppapi/native_client/src/trusted/plugin/pnacl_resources.cc1
-rw-r--r--ppapi/native_client/src/trusted/plugin/pnacl_resources.h2
-rw-r--r--ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.cc2
-rw-r--r--ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_shim.c6
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,