Chromium Code Reviews| Index: ppapi/native_client/src/trusted/plugin/pnacl_resources.cc |
| =================================================================== |
| --- ppapi/native_client/src/trusted/plugin/pnacl_resources.cc (revision 115018) |
| +++ ppapi/native_client/src/trusted/plugin/pnacl_resources.cc (working copy) |
| @@ -11,6 +11,7 @@ |
| #include "native_client/src/shared/platform/nacl_check.h" |
| #include "native_client/src/trusted/desc/nacl_desc_wrapper.h" |
| #include "native_client/src/trusted/plugin/browser_interface.h" |
| +#include "native_client/src/trusted/plugin/manifest.h" |
| #include "native_client/src/trusted/plugin/plugin.h" |
| #include "native_client/src/trusted/plugin/plugin_error.h" |
| #include "native_client/src/trusted/plugin/pnacl_coordinator.h" |
| @@ -43,15 +44,22 @@ |
| // Schedule the downloads. |
| CHECK(resource_urls_.size() > 0); |
| for (size_t i = 0; i < resource_urls_.size(); ++i) { |
| - const nacl::string& full_url = resource_base_url_ + resource_urls_[i]; |
| + nacl::string full_url; |
| + ErrorInfo error_info; |
| + if (!manifest_->ResolveURL(resource_urls_[i], &full_url, &error_info)) { |
| + coordinator_->ReportNonPpapiError(nacl::string("failed to resolve ") + |
| + resource_urls_[i] + ": " + |
| + error_info.message() + "\n"); |
|
elijahtaylor (use chromium)
2011/12/20 17:59:11
Sorry, what I meant was a C 'break', not a line br
sehr (please use chromium)
2011/12/21 00:52:38
Doh, of course. Fixed.
|
| + } |
| pp::CompletionCallback ready_callback = |
| callback_factory_.NewCallback(&PnaclResources::ResourceReady, |
| resource_urls_[i], |
| full_url); |
| if (!plugin_->StreamAsFile(full_url, |
| + manifest_->PermitsExtensionUrls(), |
| ready_callback.pp_completion_callback())) { |
| - coordinator_->ReportNonPpapiError( |
| - nacl::string("failed to download ") + resource_urls_[i] + "\n"); |
| + coordinator_->ReportNonPpapiError(nacl::string("failed to download ") + |
| + resource_urls_[i] + "\n"); |
| break; |
| } |
| } |