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

Unified Diff: ppapi/proxy/host_dispatcher.h

Issue 7189045: Make o.o.p. proxy handle PPP_Instance versions 0.4 and 0.5. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make HostDispatcher remember plugin IFs by name, not ID Created 9 years, 6 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/dispatcher.cc ('k') | ppapi/proxy/host_dispatcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/host_dispatcher.h
diff --git a/ppapi/proxy/host_dispatcher.h b/ppapi/proxy/host_dispatcher.h
index 55ba56bc76b6593acc3611a4ee94623dede5c0f1..3d3ff98245cb317183d4d276d4e9c9dfb987dd4c 100644
--- a/ppapi/proxy/host_dispatcher.h
+++ b/ppapi/proxy/host_dispatcher.h
@@ -110,12 +110,14 @@ class HostDispatcher : public Dispatcher {
PP_Module pp_module_;
- enum PluginInterfaceSupport {
- INTERFACE_UNQUERIED = 0, // Must be 0 so memset(0) will clear the list.
+ enum PluginIFSupport {
INTERFACE_SUPPORTED,
INTERFACE_UNSUPPORTED
};
- PluginInterfaceSupport plugin_interface_support_[INTERFACE_ID_COUNT];
+ typedef std::map<std::string, PluginIFSupport> PluginIFSupportMap;
+ // Maps interface name to whether that interface is supported. If an interface
+ // name is not in the map, that implies that we haven't queried for it yet.
+ std::map<std::string, PluginIFSupport> plugin_if_support_;
brettw 2011/06/24 00:35:24 I guess I would have just done: std::map<std::stri
dmichael (off chromium) 2011/06/29 16:13:57 Good suggestion. Done.
// All target proxies currently created. These are ones that receive
// messages. They are created on demand when we receive messages.
« no previous file with comments | « ppapi/proxy/dispatcher.cc ('k') | ppapi/proxy/host_dispatcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698