diff options
| author | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-12 00:04:11 +0000 |
|---|---|---|
| committer | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-12 00:04:11 +0000 |
| commit | 513d2813014bd25f2b356e76dc55d864aa6b2a4e (patch) | |
| tree | 155e08141fb324119949b9c9b920f31076a98101 /webkit/plugins/npapi/webplugin.h | |
| parent | 222053a63ef6b3ae69ab9bc0d0871210cd41fa24 (diff) | |
| download | chromium_src-513d2813014bd25f2b356e76dc55d864aa6b2a4e.zip chromium_src-513d2813014bd25f2b356e76dc55d864aa6b2a4e.tar.gz chromium_src-513d2813014bd25f2b356e76dc55d864aa6b2a4e.tar.bz2 | |
Don't retain NPNVWindowNPObject and NPNVPluginElementNPObject twice
Both NPN_GetValue and WebPluginProxy take a reference on access, with
the latter conflating the caller's reference and its internal reference.
To be consistent with WebPluginImpl, let WebPluginProxy only manage its
internal reference and leave the plugin's reference to NPN_GetValue.
Document this in the WebPlugin interface.
Also release plugin-side proxies when WebPluginProxy is destroyed.
R=ananta
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6693052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88773 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/plugins/npapi/webplugin.h')
| -rw-r--r-- | webkit/plugins/npapi/webplugin.h | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/webkit/plugins/npapi/webplugin.h b/webkit/plugins/npapi/webplugin.h index 8b43eb4..986d828 100644 --- a/webkit/plugins/npapi/webplugin.h +++ b/webkit/plugins/npapi/webplugin.h @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -93,10 +93,12 @@ class WebPlugin { virtual void Invalidate() = 0; virtual void InvalidateRect(const gfx::Rect& rect) = 0; - // Returns the NPObject for the browser's window object. + // Returns the NPObject for the browser's window object. Does not + // take a reference. virtual NPObject* GetWindowScriptNPObject() = 0; - // Returns the DOM element that loaded the plugin. + // Returns the DOM element that loaded the plugin. Does not take a + // reference. virtual NPObject* GetPluginElement() = 0; // Cookies |