diff options
| author | steveblock@chromium.org <steveblock@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-14 20:22:25 +0000 |
|---|---|---|
| committer | steveblock@chromium.org <steveblock@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-14 20:22:25 +0000 |
| commit | ece20f57928501899c46a8a73119f88b562c2fba (patch) | |
| tree | b9b5aa7075d4e3558ad6c292342741578f931987 /content/renderer/java/java_bridge_channel.h | |
| parent | 6c158192e64a230aa18f2ba4dd647d5b1a5a8b1d (diff) | |
| download | chromium_src-ece20f57928501899c46a8a73119f88b562c2fba.zip chromium_src-ece20f57928501899c46a8a73119f88b562c2fba.tar.gz chromium_src-ece20f57928501899c46a8a73119f88b562c2fba.tar.bz2 | |
Fix JavaBridgeDispatcher to ref-count JavaBridgeChannel
JavaBridgeChannel inherits from NPChannelBase, which is ref-counted, so we
should use scoped_refptr, not scoped_ptr.
Review URL: https://chromiumcodereview.appspot.com/10356149
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136965 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/renderer/java/java_bridge_channel.h')
| -rw-r--r-- | content/renderer/java/java_bridge_channel.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/content/renderer/java/java_bridge_channel.h b/content/renderer/java/java_bridge_channel.h index 7bf500c..1700703 100644 --- a/content/renderer/java/java_bridge_channel.h +++ b/content/renderer/java/java_bridge_channel.h @@ -14,7 +14,6 @@ class JavaBridgeChannel : public NPChannelBase { static JavaBridgeChannel* GetJavaBridgeChannel( const IPC::ChannelHandle& channel_handle, base::MessageLoopProxy* ipc_message_loop); - virtual ~JavaBridgeChannel() {} // NPChannelBase implementation: virtual int GenerateRouteID() OVERRIDE; @@ -23,7 +22,9 @@ class JavaBridgeChannel : public NPChannelBase { virtual bool OnControlMessageReceived(const IPC::Message& msg) OVERRIDE; private: - JavaBridgeChannel() {} + JavaBridgeChannel(); + // This class is ref-counted. + virtual ~JavaBridgeChannel(); static NPChannelBase* ClassFactory() { return new JavaBridgeChannel(); } |