Extract duplicated code in browser_tests for closing a browser.

chrome/test/base/in_process_browser_test.h

 // 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.
 
 #ifndef CHROME_TEST_BASE_IN_PROCESS_BROWSER_TEST_H_
 #define CHROME_TEST_BASE_IN_PROCESS_BROWSER_TEST_H_
 
 #include "base/compiler_specific.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/ref_counted.h"
@@ skipping 90 matching lines @@
   // BrowserMain. BrowserMain ends up invoking RunTestOnMainThreadLoop.
   void SetUp() override;
 
   // Restores state configured in SetUp.
   void TearDown() override;
 
  protected:
   // Returns the browser created by CreateBrowser.
   Browser* browser() const { return browser_; }
 
+  // Closes the given browser and synchronously flushes all of its outstanding
+  // resources.
+  void CloseBrowserSynchronously(Browser* browser);
+
+  // Closes all browsers. No guarantees are made about the destruction of
+  // outstanding resources.

[erikchen, 2015/07/08 22:05:44]

There is only one caller of this method, and I just replicated its code, with
the addition of the autoreleasepool drain.

+  void CloseAllBrowsers();
+
   // Convenience methods for adding tabs to a Browser.
   void AddTabAtIndexToBrowser(Browser* browser,
                               int index,
                               const GURL& url,
                               ui::PageTransition transition,
                               bool check_navigation_success);
   void AddTabAtIndex(int index,
                      const GURL& url,
                      ui::PageTransition transition);
 
@@ skipping 104 matching lines @@
 #if defined(OS_MACOSX)
   base::mac::ScopedNSAutoreleasePool* autorelease_pool_;
 #endif  // OS_MACOSX
 
 #if defined(OS_WIN)
   scoped_ptr<base::win::ScopedCOMInitializer> com_initializer_;
 #endif
 };
 
 #endif  // CHROME_TEST_BASE_IN_PROCESS_BROWSER_TEST_H_