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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/browser_list_unittest.cc

Issue 8136027: Print Preview: Make print preview tab modal. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: do not listen for F1 key in JS Created 9 years, 2 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
« chrome/browser/resources/print_preview/print_preview.js ('K') | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm » ('j') | chrome/browser/ui/webui/print_preview_handler.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_list_unittest.cc
===================================================================
--- chrome/browser/ui/browser_list_unittest.cc (revision 105366)
+++ chrome/browser/ui/browser_list_unittest.cc (working copy)
@@ -158,6 +158,8 @@
browser3->CloseAllTabs();
}
+#if 0
+// TODO(thestig) Fix or remove this test. http://crbug.com/XXXXX
vandebo (ex-Chrome) 2011/10/14 07:14:05 At least file the bug, if you don't resolve the co
Lei Zhang 2011/10/14 20:31:27 Done.
TEST_F(BrowserListTest, TabContentsIteratorBackgroundPrinting) {
// Make sure we have 1 window to start with.
EXPECT_EQ(1U, BrowserList::size());
@@ -238,3 +240,4 @@
EXPECT_EQ(0U, CountAllTabs());
}
+#endif
« chrome/browser/resources/print_preview/print_preview.js ('K') | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm » ('j') | chrome/browser/ui/webui/print_preview_handler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698