First patch in making destructors of refcounted objects private.

ipc/ipc_sync_channel.h

Index: ipc/ipc_sync_channel.h
===================================================================
--- ipc/ipc_sync_channel.h	(revision 30973)
+++ ipc/ipc_sync_channel.h	(working copy)
@@ -61,8 +61,6 @@
                 MessageLoop* ipc_thread,
                 base::WaitableEvent* shutdown_event);
 
-    ~SyncContext();
-
     // Adds information about an outgoing sync message to the context so that
     // we know how to deserialize the reply.
     void Push(IPC::SyncMessage* sync_msg);
@@ -97,6 +95,7 @@
     }
 
    private:
+    ~SyncContext();

[M-A Ruel, 2009/11/05 20:31:28]

virtual

[jam, 2009/11/05 21:52:37]

no need, nothing derives from this class.

[awong, 2009/11/05 21:58:39]

It's already virtual because it's inheriting from a class that has a virtual
destructor.  Making it virtual here is just good documentation.  (Also the style
guide says to. :) )

Same with the other childless classes that are descendants of others with
virtual destructors.

     // IPC::ChannelProxy methods that we override.
 
     // Called on the listener thread.