Fifth patch in getting rid of caching MessageLoop pointers.

chrome/browser/task_manager.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/task_manager.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/compiler_specific.h"
 #include "base/i18n/number_formatting.h"
@@ skipping 40 matching lines @@
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // TaskManagerModel class
 ////////////////////////////////////////////////////////////////////////////////
 
 // static
 int TaskManagerModel::goats_teleported_ = 0;
 
 TaskManagerModel::TaskManagerModel(TaskManager* task_manager)
-    : ui_loop_(MessageLoop::current()),
-      update_state_(IDLE) {
+    : update_state_(IDLE) {
 
   TaskManagerBrowserProcessResourceProvider* browser_provider =
       new TaskManagerBrowserProcessResourceProvider(task_manager);
   browser_provider->AddRef();
   providers_.push_back(browser_provider);
   TaskManagerTabContentsResourceProvider* wc_provider =
       new TaskManagerTabContentsResourceProvider(task_manager);
   wc_provider->AddRef();
   providers_.push_back(wc_provider);
   TaskManagerChildProcessResourceProvider* child_process_provider =
@@ skipping 682 matching lines @@
 }
 
 void TaskManagerModel::OnBytesRead(URLRequestJob* job, int byte_count) {
   int render_process_host_child_id = -1, routing_id = -1;
   ResourceDispatcherHost::RenderViewForRequest(job->request(),
                                                &render_process_host_child_id,
                                                &routing_id);
   // This happens in the IO thread, post it to the UI thread.
   int origin_child_id =
       chrome_browser_net::GetOriginProcessUniqueIDForRequest(job->request());
-  ui_loop_->PostTask(FROM_HERE,
-                     NewRunnableMethod(
-                         this,
-                         &TaskManagerModel::BytesRead,
-                         BytesReadParam(origin_child_id,
-                                        render_process_host_child_id,
-                                        routing_id, byte_count)));
+  ChromeThread::PostTask(
+      ChromeThread::UI, FROM_HERE,
+      NewRunnableMethod(
+          this,
+          &TaskManagerModel::BytesRead,
+          BytesReadParam(origin_child_id,
+          render_process_host_child_id,
+          routing_id, byte_count)));
 }
 
 bool TaskManagerModel::GetProcessMetricsForRow(
     int row, base::ProcessMetrics** proc_metrics) const {
   DCHECK(row < ResourceCount());
   *proc_metrics = NULL;
 
   MetricsMap::const_iterator iter =
       metrics_map_.find(resources_[row]->GetProcess());
   if (iter == metrics_map_.end())
@@ skipping 77 matching lines @@
   // or popup, we can only have one tab, hence we need to process this in a
   // tabbed browser window. Currently, |browser| is pointing to the application,
   // popup window. Therefore, we have to retrieve the last active tab again,
   // since a new window has been used.
   if (browser->type() & Browser::TYPE_APP_POPUP) {
     browser = BrowserList::GetLastActive();
     DCHECK(browser);
   }
   browser->window()->Show();
 }