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

Unified Diff: ppapi/proxy/plugin_resource.h

Issue 7587011: Remove the old type system for proxied resources. These were no longer being used. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: New one 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
« no previous file with comments | « ppapi/proxy/mock_resource.cc ('k') | ppapi/proxy/plugin_resource.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/plugin_resource.h
diff --git a/ppapi/proxy/plugin_resource.h b/ppapi/proxy/plugin_resource.h
index 3abe9cbbb703a6b3a352ae212243729d9c817958..10dbf72b8d082cb6ae07d1ff0bac1dcbe0895179 100644
--- a/ppapi/proxy/plugin_resource.h
+++ b/ppapi/proxy/plugin_resource.h
@@ -12,53 +12,14 @@
#include "ppapi/proxy/plugin_resource_tracker.h"
#include "ppapi/shared_impl/resource_object_base.h"
-// If you inherit from resource, make sure you add the class name here.
-#define FOR_ALL_PLUGIN_RESOURCES(F) \
- F(Audio) \
- F(AudioConfig) \
- F(Broker) \
- F(Buffer) \
- F(Context3D) \
- F(FileChooser) \
- F(FileRef) \
- F(FileSystem) \
- F(FlashMenu) \
- F(FlashNetConnector) \
- F(Font) \
- F(Graphics2D) \
- F(Graphics3D) \
- F(ImageData) \
- F(MockResource) \
- F(PrivateFontFile) \
- F(Surface3D) \
- F(URLLoader) \
- F(URLRequestInfo) \
- F(URLResponseInfo) \
- F(VideoCapture) \
- F(VideoDecoder)
-
namespace pp {
namespace proxy {
-// Forward declaration of Resource classes.
-#define DECLARE_RESOURCE_CLASS(RESOURCE) class RESOURCE;
-FOR_ALL_PLUGIN_RESOURCES(DECLARE_RESOURCE_CLASS)
-#undef DECLARE_RESOURCE_CLASS
-
class PluginResource : public ::ppapi::ResourceObjectBase {
public:
PluginResource(const HostResource& resource);
virtual ~PluginResource();
- // Returns NULL if the resource is invalid or is a different type.
- template<typename T> static T* GetAs(PP_Resource res) {
- PluginResource* resource =
- PluginResourceTracker::GetInstance()->GetResourceObject(res);
- return resource ? resource->Cast<T>() : NULL;
- }
-
- template <typename T> T* Cast() { return NULL; }
-
PP_Instance instance() const { return host_resource_.instance(); }
// Returns the host resource ID for sending to the host process.
@@ -69,14 +30,6 @@ class PluginResource : public ::ppapi::ResourceObjectBase {
PluginDispatcher* GetDispatcher();
private:
- // Type-specific getters for individual resource types. These will return
- // NULL if the resource does not match the specified type. Used by the Cast()
- // function.
- #define DEFINE_TYPE_GETTER(RESOURCE) \
- virtual RESOURCE* As##RESOURCE();
- FOR_ALL_PLUGIN_RESOURCES(DEFINE_TYPE_GETTER)
- #undef DEFINE_TYPE_GETTER
-
// The resource ID in the host that this object corresponds to. Inside the
// plugin we'll remap the resource IDs so we can have many host processes
// each independently generating resources (which may conflict) but the IDs
@@ -86,14 +39,6 @@ class PluginResource : public ::ppapi::ResourceObjectBase {
DISALLOW_COPY_AND_ASSIGN(PluginResource);
};
-// Cast() specializations.
-#define DEFINE_RESOURCE_CAST(Type) \
- template <> inline Type* PluginResource::Cast<Type>() { \
- return As##Type(); \
- }
-FOR_ALL_PLUGIN_RESOURCES(DEFINE_RESOURCE_CAST)
-#undef DEFINE_RESOURCE_CAST
-
} // namespace proxy
} // namespace pp
« no previous file with comments | « ppapi/proxy/mock_resource.cc ('k') | ppapi/proxy/plugin_resource.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698