Index: chrome/test/base/in_process_browser_test.cc |
diff --git a/chrome/test/base/in_process_browser_test.cc b/chrome/test/base/in_process_browser_test.cc |
index db081c2a728f315d229c9beb2c67e4e23466bc4e..1ddd6aa66cd88a53e71a8ff51cca35e67a53d7af 100644 |
--- a/chrome/test/base/in_process_browser_test.cc |
+++ b/chrome/test/base/in_process_browser_test.cc |
@@ -93,38 +93,6 @@ namespace { |
// Passed as value of kTestType. |
const char kBrowserTestType[] = "browser"; |
-// A BrowserListObserver that makes sure that all browsers created are on the |
-// |allowed_desktop_|. |
-class SingleDesktopTestObserver : public chrome::BrowserListObserver, |
- public base::NonThreadSafe { |
- public: |
- explicit SingleDesktopTestObserver(chrome::HostDesktopType allowed_desktop); |
- ~SingleDesktopTestObserver() override; |
- |
- // chrome::BrowserListObserver: |
- void OnBrowserAdded(Browser* browser) override; |
- |
- private: |
- chrome::HostDesktopType allowed_desktop_; |
- |
- DISALLOW_COPY_AND_ASSIGN(SingleDesktopTestObserver); |
-}; |
- |
-SingleDesktopTestObserver::SingleDesktopTestObserver( |
- chrome::HostDesktopType allowed_desktop) |
- : allowed_desktop_(allowed_desktop) { |
- BrowserList::AddObserver(this); |
-} |
- |
-SingleDesktopTestObserver::~SingleDesktopTestObserver() { |
- BrowserList::RemoveObserver(this); |
-} |
- |
-void SingleDesktopTestObserver::OnBrowserAdded(Browser* browser) { |
- CHECK(CalledOnValidThread()); |
- CHECK_EQ(browser->host_desktop_type(), allowed_desktop_); |
-} |
- |
} // namespace |
// Library used for testing accessibility. |
@@ -550,16 +518,7 @@ void InProcessBrowserTest::RunTestOnMainThreadLoop() { |
// Pump startup related events. |
content::RunAllPendingInMessageLoop(); |
- chrome::HostDesktopType active_desktop = chrome::GetActiveDesktop(); |
- // Self-adds/removes itself from the BrowserList observers. |
- scoped_ptr<SingleDesktopTestObserver> single_desktop_test_observer; |
- if (!multi_desktop_test_) { |
- single_desktop_test_observer.reset( |
- new SingleDesktopTestObserver(active_desktop)); |
- } |
- |
- const BrowserList* active_browser_list = |
- BrowserList::GetInstance(active_desktop); |
+ const BrowserList* active_browser_list = BrowserList::GetInstance(); |
if (!active_browser_list->empty()) { |
browser_ = active_browser_list->get(0); |
#if defined(USE_ASH) |
@@ -628,12 +587,8 @@ void InProcessBrowserTest::RunTestOnMainThreadLoop() { |
content::RunAllPendingInMessageLoop(); |
QuitBrowsers(); |
- // All BrowserLists should be empty at this point. |
- for (chrome::HostDesktopType t = chrome::HOST_DESKTOP_TYPE_FIRST; |
- t < chrome::HOST_DESKTOP_TYPE_COUNT; |
- t = static_cast<chrome::HostDesktopType>(t + 1)) { |
- CHECK(BrowserList::GetInstance(t)->empty()) << t; |
- } |
+ // BrowserList should be empty at this point. |
+ CHECK(BrowserList::GetInstance()->empty()); |
} |
void InProcessBrowserTest::QuitBrowsers() { |