Remove BrowserList::GetLastActive usage in TaskManager. Just use browser::Navigate() instead.

chrome/browser/task_manager/task_manager.cc

 // Copyright (c) 2012 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/task_manager.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/i18n/number_formatting.h"
 #include "base/i18n/rtl.h"
 #include "base/process_util.h"
 #include "base/rand_util.h"
 #include "base/string_number_conversions.h"
 #include "base/stringprintf.h"
 #include "base/threading/thread.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/background/background_contents_service.h"
 #include "chrome/browser/background/background_contents_service_factory.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/task_manager/task_manager_resource_providers.h"
 #include "chrome/browser/task_manager/task_manager_worker_resource_provider.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_list.h"
-#include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/common/chrome_view_type.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_child_process_host.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_view_host_delegate.h"
 #include "content/public/browser/resource_request_info.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 1082 matching lines @@
 void TaskManager::ModelChanged() {
   model_->ModelChanged();
 }
 
 // static
 TaskManager* TaskManager::GetInstance() {
   return Singleton<TaskManager>::get();
 }
 
 void TaskManager::OpenAboutMemory() {
-  Browser* browser = BrowserList::GetLastActive();
-  OpenURLParams params(
-      GURL(chrome::kChromeUIMemoryURL), Referrer(), NEW_FOREGROUND_TAB,
-      content::PAGE_TRANSITION_LINK, false);
-
-  if (!browser) {
-    // On OS X, the task manager can be open without any open browser windows.
-    if (!g_browser_process || !g_browser_process->profile_manager())
-      return;
-    Profile* profile =
-        g_browser_process->profile_manager()->GetLastUsedProfile();
-    if (!profile)
-      return;
-    browser = Browser::Create(profile);
-    browser->OpenURL(params);
-  } else {
-    browser->OpenURL(params);
-
-    // In case the browser window is minimized, show it. If |browser| is a
-    // non-tabbed window, the call to OpenURL above will have opened a
-    // TabContentsWrapper in a tabbed browser, so we need to grab it with
-    // GetLastActive before the call to show().
-    if (!browser->is_type_tabbed()) {
-      browser = BrowserList::GetLastActive();
-      DCHECK(browser);
-    }
-  }
-  browser->window()->Show();
+  browser::NavigateParams params(NULL,
+                                 GURL(chrome::kChromeUIMemoryURL),
+                                 content::PAGE_TRANSITION_LINK);
+  params.disposition = NEW_FOREGROUND_TAB;
+  browser::Navigate(&params);
 }
 
 bool TaskManagerModel::GetAndCacheMemoryMetrics(base::ProcessHandle handle,
                                                 MemoryUsageEntry* usage) const {
   MetricsMap::const_iterator iter = metrics_map_.find(handle);
   if (iter == metrics_map_.end())
     return false;
 
   if (!iter->second->GetMemoryBytes(&usage->first, &usage->second))
     return false;
@@ skipping 44 matching lines @@
     // Count the number of extensions with background pages (including
     // incognito).
     count += CountExtensionBackgroundPagesForProfile(profile);
     if (profile->HasOffTheRecordProfile()) {
       count += CountExtensionBackgroundPagesForProfile(
           profile->GetOffTheRecordProfile());
     }
   }
   return count;
 }