Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(896)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/extensions/extension_browsertest.cc

Issue 126175: Close all browsers during InProcessBrowserTest cleanup, not just the main one... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/test/in_process_browser_test.h » ('j') | chrome/test/in_process_browser_test.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_browsertest.cc
===================================================================
--- chrome/browser/extensions/extension_browsertest.cc (revision 18385)
+++ chrome/browser/extensions/extension_browsertest.cc (working copy)
@@ -163,11 +163,4 @@
// TODO(mpcomplete): wait for uninstall to complete?
browser()->profile()->GetExtensionsService()->UninstallExtension(kGoodCrxId);
-
- // Close our incognito window.
- Browser* otr_browser = BrowserList::FindBrowserWithType(
- browser()->profile()->GetOffTheRecordProfile(),
- Browser::TYPE_NORMAL);
- if (otr_browser)
- otr_browser->CloseAllTabs();
}
« no previous file with comments | « no previous file | chrome/test/in_process_browser_test.h » ('j') | chrome/test/in_process_browser_test.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698