diff options
author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-08 16:31:46 +0000 |
---|---|---|
committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-08 16:31:46 +0000 |
commit | 465faa29046328890a224677db522f1aece8cad0 (patch) | |
tree | 5cf23803cc13d27c71c05c4532a5fc434e6d7e4d /ppapi/proxy/ppb_testing_proxy.cc | |
parent | a313e51c562c3d3400d2bd14231f23e9ca699857 (diff) | |
download | chromium_src-465faa29046328890a224677db522f1aece8cad0.zip chromium_src-465faa29046328890a224677db522f1aece8cad0.tar.gz chromium_src-465faa29046328890a224677db522f1aece8cad0.tar.bz2 |
Rent syncemove all uses of the global Dispatcher Get function.
This reqired reworking how plugin->host GetInterface works. Previously,
interface requests were symmetric where each side would first do a
SupportsInterface to see if the remote side supports the interface, then create
the proxy. Since the plugin may talk to multiple renderers, we don't know where
to send these requests. The solution is to make the assumption that the
renderer always supports all PPB interfaces (which is possible since the proxy
is compiled with the executable).
This also adds some better lookup for interfaces to avoid having multiple lists
of interfaces. We now have a list of interfaces and factory functions in
dispatcher.cc.
Add some additional testing infrastructure for the dispatchers with simple tests.
Review URL: http://codereview.chromium.org/6286070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74121 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/ppb_testing_proxy.cc')
-rw-r--r-- | ppapi/proxy/ppb_testing_proxy.cc | 21 |
1 files changed, 15 insertions, 6 deletions
diff --git a/ppapi/proxy/ppb_testing_proxy.cc b/ppapi/proxy/ppb_testing_proxy.cc index b65bae7..ca15151 100644 --- a/ppapi/proxy/ppb_testing_proxy.cc +++ b/ppapi/proxy/ppb_testing_proxy.cc @@ -70,6 +70,11 @@ const PPB_Testing_Dev testing_interface = { &GetLiveObjectsForInstance }; +InterfaceProxy* CreateTestingProxy(Dispatcher* dispatcher, + const void* target_interface) { + return new PPB_Testing_Proxy(dispatcher, target_interface); +} + } // namespace PPB_Testing_Proxy::PPB_Testing_Proxy(Dispatcher* dispatcher, @@ -80,12 +85,16 @@ PPB_Testing_Proxy::PPB_Testing_Proxy(Dispatcher* dispatcher, PPB_Testing_Proxy::~PPB_Testing_Proxy() { } -const void* PPB_Testing_Proxy::GetSourceInterface() const { - return &testing_interface; -} - -InterfaceID PPB_Testing_Proxy::GetInterfaceId() const { - return INTERFACE_ID_PPB_TESTING; +// static +const InterfaceProxy::Info* PPB_Testing_Proxy::GetInfo() { + static const Info info = { + &testing_interface, + PPB_TESTING_DEV_INTERFACE, + INTERFACE_ID_PPB_TESTING, + false, + &CreateTestingProxy, + }; + return &info; } bool PPB_Testing_Proxy::OnMessageReceived(const IPC::Message& msg) { |