Extract duplicated code in browser_tests for closing a browser.

chrome/test/base/in_process_browser_test.h

Index: chrome/test/base/in_process_browser_test.h
diff --git a/chrome/test/base/in_process_browser_test.h b/chrome/test/base/in_process_browser_test.h
index e516f4ead1ae3bbd538eb9276b2ec54bb09d6b6f..494386a45e2541bee6f8d85f65076dceb7761d81 100644
--- a/chrome/test/base/in_process_browser_test.h
+++ b/chrome/test/base/in_process_browser_test.h
@@ -111,6 +111,18 @@ class InProcessBrowserTest : public content::BrowserTestBase {
   // 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,