diff options
author | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-28 17:29:03 +0000 |
---|---|---|
committer | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-28 17:29:03 +0000 |
commit | 17acbb549e69322632c56e0e6c93a3d0c5c3dda1 (patch) | |
tree | 56084ef5cad6b4738722d7a71386e0cea9361146 | |
parent | fc72076a7fa4c7260312fcb092294ab6e2a39821 (diff) | |
download | chromium_src-17acbb549e69322632c56e0e6c93a3d0c5c3dda1.zip chromium_src-17acbb549e69322632c56e0e6c93a3d0c5c3dda1.tar.gz chromium_src-17acbb549e69322632c56e0e6c93a3d0c5c3dda1.tar.bz2 |
Speculative fix for a leak in SyncChannel.
This leak is intermittent and I can't reproduce it reliably on my machine but after this fix I haven't been able to reproduce it at all.
BUG=16128
TEST=no test possible
Review URL: http://codereview.chromium.org/176013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24740 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | ipc/ipc_channel_proxy.h | 2 | ||||
-rw-r--r-- | ipc/ipc_sync_channel.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/ipc/ipc_channel_proxy.h b/ipc/ipc_channel_proxy.h index e884818..a2d55be 100644 --- a/ipc/ipc_channel_proxy.h +++ b/ipc/ipc_channel_proxy.h @@ -93,7 +93,7 @@ class ChannelProxy : public Message::Sender { Channel::Listener* listener, MessageFilter* filter, MessageLoop* ipc_thread_loop); - ~ChannelProxy() { + virtual ~ChannelProxy() { Close(); } diff --git a/ipc/ipc_sync_channel.h b/ipc/ipc_sync_channel.h index f5d2add..bfc9eac 100644 --- a/ipc/ipc_sync_channel.h +++ b/ipc/ipc_sync_channel.h @@ -36,7 +36,7 @@ class SyncChannel : public ChannelProxy, Channel::Listener* listener, MessageFilter* filter, MessageLoop* ipc_message_loop, bool create_pipe_now, base::WaitableEvent* shutdown_event); - ~SyncChannel(); + virtual ~SyncChannel(); virtual bool Send(Message* message); virtual bool SendWithTimeout(Message* message, int timeout_ms); |