Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(853)

Unified Diff: ppapi/native_client/src/trusted/plugin/pnacl_resources.h

Issue 315583005: Pepper: Simplify error reporting in PnaclResources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/native_client/src/trusted/plugin/pnacl_resources.h
diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_resources.h b/ppapi/native_client/src/trusted/plugin/pnacl_resources.h
index a96cb17feb7385c8fe030549e4b85ae21d5fbc5a..cca3b5443a8bbdbc97929054f36f8b3fe7d657e9 100644
--- a/ppapi/native_client/src/trusted/plugin/pnacl_resources.h
+++ b/ppapi/native_client/src/trusted/plugin/pnacl_resources.h
@@ -20,17 +20,14 @@
namespace plugin {
class Plugin;
-class PnaclCoordinator;
// Loads a list of resources, providing a way to get file descriptors for
// these resources. URLs for resources are resolved by the manifest
// and point to pnacl component filesystem resources.
class PnaclResources {
public:
- PnaclResources(Plugin* plugin,
- PnaclCoordinator* coordinator)
+ explicit PnaclResources(Plugin* plugin)
: plugin_(plugin),
- coordinator_(coordinator),
llc_file_handle_(PP_kInvalidFileHandle),
ld_file_handle_(PP_kInvalidFileHandle) {
}
@@ -56,8 +53,6 @@ class PnaclResources {
// The plugin requesting the resource loading.
Plugin* plugin_;
- // The coordinator responsible for reporting errors, etc.
- PnaclCoordinator* coordinator_;
// Tool names for llc and ld; read from the resource info file.
nacl::string llc_tool_name_;
« no previous file with comments | « ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc ('k') | ppapi/native_client/src/trusted/plugin/pnacl_resources.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698