Cleanup IPC::ChannelProxy to use SingleThreadTaskRunner

content/common/child_thread.cc

Index: content/common/child_thread.cc
diff --git a/content/common/child_thread.cc b/content/common/child_thread.cc
index 3f00f93e120f259a43f9d308c7810df563b551a6..71a40389b91d245d6c42a9cbb76a7341a4a4249e 100644
--- a/content/common/child_thread.cc
+++ b/content/common/child_thread.cc
@@ -75,6 +75,9 @@ ChildThread::~ChildThread() {
   // releases the reference count to this channel.
   channel_->Close();
 
+  // TODO(sergeyu): The comment below is not up to date. Figure out if

[jam, 2012/07/01 22:19:41]

no need for the TODO, I'm looking into this :)
http://codereview.chromium.org/10702048/

[Sergey Ulanov, 2012/07/02 18:44:34]

Done.

+  // we still need this call and update the comment or remove it.
+  //
   // The ChannelProxy object caches a pointer to the IPC thread, so need to
   // reset it as it's not guaranteed to outlive this object.
   // NOTE: this also has the side-effect of not closing the main IPC channel to
@@ -83,7 +86,7 @@ ChildThread::~ChildThread() {
   // until this process is shut down, and the OS closes the handle
   // automatically.  We used to watch the object handle on Windows to do this,
   // but it wasn't possible to do so on POSIX.
-  channel_->ClearIPCMessageLoop();
+  channel_->ClearIPCTaskRunner();
 }
 
 void ChildThread::OnChannelError() {