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

Unified Diff: ppapi/proxy/ppb_surface_3d_proxy.h

Issue 7629017: Add a unified resource tracker shared between the proxy and the impl. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments Created 9 years, 4 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/proxy/ppb_surface_3d_proxy.h
diff --git a/ppapi/proxy/ppb_surface_3d_proxy.h b/ppapi/proxy/ppb_surface_3d_proxy.h
index 0fa0d4cc04cbc71b5b142a80e89dfb2543a96501..ae2ec8fc3a4c25f01b09f19767f6ca428860f78f 100644
--- a/ppapi/proxy/ppb_surface_3d_proxy.h
+++ b/ppapi/proxy/ppb_surface_3d_proxy.h
@@ -12,8 +12,8 @@
#include "ppapi/c/pp_instance.h"
#include "ppapi/cpp/completion_callback.h"
#include "ppapi/proxy/interface_proxy.h"
-#include "ppapi/proxy/plugin_resource.h"
#include "ppapi/proxy/proxy_non_thread_safe_ref_count.h"
+#include "ppapi/shared_impl/resource.h"
#include "ppapi/thunk/ppb_surface_3d_api.h"
struct PPB_Surface3D_Dev;
@@ -23,13 +23,13 @@ namespace proxy {
class Context3D;
-class Surface3D : public PluginResource,
+class Surface3D : public ppapi::Resource,
public ppapi::thunk::PPB_Surface3D_API {
public:
explicit Surface3D(const ppapi::HostResource& host_resource);
virtual ~Surface3D();
- // ResourceObjectBase overrides.
+ // Resource overrides.
virtual PPB_Surface3D_API* AsPPB_Surface3D_API() OVERRIDE;
// PPB_Surface3D_API implementation.
@@ -51,11 +51,7 @@ class Surface3D : public PluginResource,
Context3D* context() const { return context_; }
- void set_resource(PP_Resource resource) { resource_ = resource; }
- PP_Resource resource() const { return resource_; }
-
private:
- PP_Resource resource_;
Context3D* context_;
// In the plugin, this is the current callback set for Flushes. When the

Powered by Google App Engine
This is Rietveld 408576698