diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-20 23:09:28 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-20 23:09:28 +0000 |
| commit | ac4b54de7f49f4f0910b952888eb44fc55737195 (patch) | |
| tree | 00fc8b7d9bb51ca71ee34d3befb20be85acc2a3c /ppapi/proxy/plugin_dispatcher_unittest.cc | |
| parent | d365a681dce7dca73ecbd682529a98b5505e9cdb (diff) | |
| download | chromium_src-ac4b54de7f49f4f0910b952888eb44fc55737195.zip chromium_src-ac4b54de7f49f4f0910b952888eb44fc55737195.tar.gz chromium_src-ac4b54de7f49f4f0910b952888eb44fc55737195.tar.bz2 | |
Rename InterfaceID to ApiID and move the file.
This was originally in the proxy and had a 1:1 correspondence with an
interface. Then we reused this for other stuff and then merged some interfaces
into larger APIs (ppapi/thunk/*_api.h) so the name was no longer accurate.
It was wrong to be in the proxy directory since directories at a "lower level"
than the proxy (ppapi/shared_impl and webkit/plugins/ppapi) depended on it.
This renames to ApiID (I avoided APIID since it looks like a define) which is
the proper description of the class, and moved it to shared_impl. This fixes
the deps since there are no longer any bad dependencies on the proxy directory.
TEST=it compiles
BUG=none
Review URL: http://codereview.chromium.org/8333004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106619 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/plugin_dispatcher_unittest.cc')
| -rw-r--r-- | ppapi/proxy/plugin_dispatcher_unittest.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/ppapi/proxy/plugin_dispatcher_unittest.cc b/ppapi/proxy/plugin_dispatcher_unittest.cc index 8a32026..a2999d0 100644 --- a/ppapi/proxy/plugin_dispatcher_unittest.cc +++ b/ppapi/proxy/plugin_dispatcher_unittest.cc @@ -52,7 +52,7 @@ class PluginDispatcherTest : public PluginProxyTest { public: PluginDispatcherTest() {} - bool HasTargetProxy(InterfaceID id) { + bool HasTargetProxy(ApiID id) { return !!plugin_dispatcher()->proxies_[id].get(); } }; @@ -71,12 +71,12 @@ TEST_F(PluginDispatcherTest, SupportsInterface) { TEST_F(PluginDispatcherTest, PPBCreation) { // Sending a PPB message out of the blue should create a target proxy for // that interface in the plugin. - EXPECT_FALSE(HasTargetProxy(INTERFACE_ID_PPB_AUDIO)); + EXPECT_FALSE(HasTargetProxy(API_ID_PPB_AUDIO)); PpapiMsg_PPBAudio_NotifyAudioStreamCreated audio_msg( - INTERFACE_ID_PPB_AUDIO, HostResource(), 0, + API_ID_PPB_AUDIO, HostResource(), 0, IPC::PlatformFileForTransit(), base::SharedMemoryHandle(), 0); plugin_dispatcher()->OnMessageReceived(audio_msg); - EXPECT_TRUE(HasTargetProxy(INTERFACE_ID_PPB_AUDIO)); + EXPECT_TRUE(HasTargetProxy(API_ID_PPB_AUDIO)); } } // namespace proxy |