diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-11 17:42:43 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-11 17:42:43 +0000 |
| commit | 82d7c2e9709a4624c08b451bed9625218ab4fb5c (patch) | |
| tree | a9e87fec44065fd7f5bde22d6a397f4e967f9969 /ppapi/proxy | |
| parent | 24321934209c4277acf9f7bb5b71d5d164162cda (diff) | |
| download | chromium_src-82d7c2e9709a4624c08b451bed9625218ab4fb5c.zip chromium_src-82d7c2e9709a4624c08b451bed9625218ab4fb5c.tar.gz chromium_src-82d7c2e9709a4624c08b451bed9625218ab4fb5c.tar.bz2 | |
Don't send ReleaseResource notifications to the browser for fonts. They have
no corresponding host resource so this would spew a lot of errors.
Review URL: http://codereview.chromium.org/6982027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84997 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy')
| -rw-r--r-- | ppapi/proxy/plugin_resource_tracker.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ppapi/proxy/plugin_resource_tracker.cc b/ppapi/proxy/plugin_resource_tracker.cc index 0abae90..24d1281 100644 --- a/ppapi/proxy/plugin_resource_tracker.cc +++ b/ppapi/proxy/plugin_resource_tracker.cc @@ -161,8 +161,10 @@ void PluginResourceTracker::ReleasePluginResourceRef( // dispatcher can be NULL if the plugin held on to a resource after the // instance was destroyed. In that case the browser-side resource has - // already been freed correctly on the browser side. - if (notify_browser_on_release && dispatcher) { + // already been freed correctly on the browser side. The host_resource + // will be NULL for proxy-only resources, which we obviously don't need to + // tell the host about. + if (notify_browser_on_release && dispatcher && !host_resource.is_null()) { dispatcher->Send(new PpapiHostMsg_PPBCore_ReleaseResource( INTERFACE_ID_PPB_CORE, host_resource)); } |