diff options
author | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-31 19:55:11 +0000 |
---|---|---|
committer | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-31 19:55:11 +0000 |
commit | 7445a61840911ec3c4786bcbcc79b9b9fc8029c5 (patch) | |
tree | 63da5faf10947a74086bdac0ced03d3ffaa17d33 /ppapi/host/resource_host.cc | |
parent | 4ae74e10f25b8e92765eb7eff1b41126c51fce20 (diff) | |
download | chromium_src-7445a61840911ec3c4786bcbcc79b9b9fc8029c5.zip chromium_src-7445a61840911ec3c4786bcbcc79b9b9fc8029c5.tar.gz chromium_src-7445a61840911ec3c4786bcbcc79b9b9fc8029c5.tar.bz2 |
PPAPI: Remove references to content from ppapi/host
As jam noted here: https://chromiumcodereview.appspot.com/16057021/
we should not reference "content" classes in ppapi/host code.
This replaces "AsPepperFoo{Browser}Host" with "IsFooHost". Now, you have to
follow the IsFooHost check with a static_cast to PepperFoo{Browser}Host*". Note that there are two hosts that return true for IsFileSystemHost, one in the browser, one in the renderer. But it seems pretty clear which one to use in any given context.
BUG=
Review URL: https://chromiumcodereview.appspot.com/21026004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214808 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/host/resource_host.cc')
-rw-r--r-- | ppapi/host/resource_host.cc | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/ppapi/host/resource_host.cc b/ppapi/host/resource_host.cc index 6e08c01..4e12260 100644 --- a/ppapi/host/resource_host.cc +++ b/ppapi/host/resource_host.cc @@ -48,21 +48,16 @@ void ResourceHost::SendReply(const ReplyMessageContext& context, host_->SendReply(context, msg); } -content::PepperFileRefHost* ResourceHost::AsPepperFileRefHost() { - return NULL; +bool ResourceHost::IsFileRefHost() { + return false; } -content::PepperFileSystemBrowserHost* -ResourceHost::AsPepperFileSystemBrowserHost() { - return NULL; +bool ResourceHost::IsFileSystemHost() { + return false; } -content::PepperFileSystemHost* ResourceHost::AsPepperFileSystemHost() { - return NULL; -} - -content::PepperGraphics2DHost* ResourceHost::AsPepperGraphics2DHost() { - return NULL; +bool ResourceHost::IsGraphics2DHost() { + return false; } void ResourceHost::AddFilter(scoped_refptr<ResourceMessageFilter> filter) { |