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 93 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 waits for it to release all its resources.
+  void CloseBrowserSynchronously(Browser* browser);
+
+  // Closes the browser without waiting for it to release all its resources.
+  // WARNING: This may leave tasks posted, but not yet run, in the message
+  // loops. Prefer CloseBrowserSynchronously over this method.
+  void CloseBrowserWithoutWaiting(Browser* browser);

[Robert Sesek, 2015/07/09 15:07:05]

Why "WithoutWaiting" instead of "Asynchronously" ?

[erikchen, 2015/07/09 23:35:43]

I plead the fifth. Changed the name to CloseBrowserAsynchronously

+
+  // Closes all browsers. No guarantees are made about the destruction of
+  // outstanding resources.
+  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 105 matching lines @@
   base::mac::ScopedNSAutoreleasePool* autorelease_pool_;
   scoped_ptr<ScopedBundleSwizzlerMac> bundle_swizzler_;
 #endif  // OS_MACOSX
 
 #if defined(OS_WIN)
   scoped_ptr<base::win::ScopedCOMInitializer> com_initializer_;
 #endif
 };
 
 #endif  // CHROME_TEST_BASE_IN_PROCESS_BROWSER_TEST_H_