Relands "Removed InProcessBrowserTest::CleanUpOnMainThread()"

chrome/browser/sessions/session_restore_browsertest_chromeos.cc

Index: chrome/browser/sessions/session_restore_browsertest_chromeos.cc
diff --git a/chrome/browser/sessions/session_restore_browsertest_chromeos.cc b/chrome/browser/sessions/session_restore_browsertest_chromeos.cc
index b8961e93c1270b8e23a5a9a5aad9d9c950bbe876..a0302cb65394b3b3c696134cb8584e0c024d24ed 100644
--- a/chrome/browser/sessions/session_restore_browsertest_chromeos.cc
+++ b/chrome/browser/sessions/session_restore_browsertest_chromeos.cc
@@ -34,19 +34,6 @@ class SessionRestoreTestChromeOS : public InProcessBrowserTest {
     InProcessBrowserTest::SetUpCommandLine(command_line);
   }
 
-  virtual void SetUpOnMainThread() OVERRIDE {
-    InProcessBrowserTest::SetUpOnMainThread();
-  }
-
-  virtual void TearDownOnMainThread() OVERRIDE {
-    InProcessBrowserTest::TearDownOnMainThread();
-    for (std::list<Browser*>::iterator iter = browser_list_.begin();
-         iter != browser_list_.end(); ++iter) {
-      CloseBrowserSynchronously(*iter);
-    }
-    browser_list_.clear();
-  }

[stevenjb, 2014/07/25 16:52:48]

This lgtm. As mentioned in email, this was introduced ages ago to ensure a clean
shutdown. If it is no longer necessary and makes the tests happy, removing this
seems fine.

-
   Browser* CreateBrowserWithParams(Browser::CreateParams params) {
     Browser* browser = new Browser(params);
     AddBlankTabAndShow(browser);