Simplify threading in browser thread by making only ChromeThread deal with di...

chrome/browser/renderer_host/resource_message_filter.cc

 // Copyright (c) 2009 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.
 
 #include "chrome/browser/renderer_host/resource_message_filter.h"
 
 #include "app/clipboard/clipboard.h"
 #include "app/gfx/native_widget_types.h"
 #include "base/command_line.h"
 #include "base/histogram.h"
@@ skipping 532 matching lines @@
 
 void ResourceMessageFilter::OnGetPlugins(bool refresh,
                                          IPC::Message* reply_msg) {
   // Schedule plugin loading on the file thread.
   // Note: it's possible that the only reference to this object is the task.  If
   // If the task executes on the file thread, and before it returns, the task it
   // posts to the IO thread runs, then this object will get destructed on the
   // file thread.  We need this object to be destructed on the IO thread, so do
   // the refcounting manually.
   AddRef();
-  ChromeThread::GetMessageLoop(ChromeThread::FILE)->PostTask(FROM_HERE,
+  ChromeThread::PostTask(
+      ChromeThread::FILE, FROM_HERE,
       NewRunnableFunction(&ResourceMessageFilter::OnGetPluginsOnFileThread,
           this, refresh, reply_msg));
 }
 
 void ResourceMessageFilter::OnGetPluginsOnFileThread(
     ResourceMessageFilter* filter,
     bool refresh,
     IPC::Message* reply_msg) {
   std::vector<WebPluginInfo> plugins;
   NPAPI::PluginList::Singleton()->GetPlugins(refresh, &plugins);
-
   ViewHostMsg_GetPlugins::WriteReplyParams(reply_msg, plugins);
-  // Note, we want to get to the IO thread now, but the file thread outlives it
-  // so we can't post a task to it directly as it might be in the middle of
-  // destruction.  So hop through the main thread, where the destruction of the
-  // IO thread happens and hence no race conditions exist.
-  filter->ui_loop()->PostTask(FROM_HERE,
-      NewRunnableFunction(&ResourceMessageFilter::OnNotifyPluginsLoaded,
-          filter, reply_msg));
-}
-
-void ResourceMessageFilter::OnNotifyPluginsLoaded(ResourceMessageFilter* filter,
-                                                  IPC::Message* reply_msg) {
-  ChromeThread::GetMessageLoop(ChromeThread::IO)->PostTask(FROM_HERE,
+  ChromeThread::PostTask(
+      ChromeThread::IO, FROM_HERE,
       NewRunnableMethod(filter, &ResourceMessageFilter::OnPluginsLoaded,
           reply_msg));
 }
 
 void ResourceMessageFilter::OnPluginsLoaded(IPC::Message* reply_msg) {
   Send(reply_msg);
   Release();
 }
 
 void ResourceMessageFilter::OnGetPluginPath(const GURL& url,
@@ skipping 534 matching lines @@
   *signed_public_key = keygen_handler->GenKeyAndSignChallenge();
 }
 
 #if defined(USE_TCMALLOC)
 void ResourceMessageFilter::OnRendererTcmalloc(base::ProcessId pid,
                                                const std::string& output) {
   ui_loop()->PostTask(FROM_HERE,
       NewRunnableFunction(AboutTcmallocRendererCallback, pid, output));
 }
 #endif