Print Preview: Make print preview tab modal.

chrome/browser/ui/browser_list_unittest.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/printing/background_printing_manager.h"
 #include "chrome/browser/ui/browser_list.h"
 #if defined(TOUCH_UI)
 // TODO(alicet): clean up dependency on defaults when max tab count is removed.
 #include "chrome/browser/defaults.h"
 #endif
@@ skipping 140 matching lines @@
       EXPECT_EQ(browser3.get(), iterator.browser());
     else
       EXPECT_TRUE(false);
   }
 
   // Close all remaining tabs to keep all the destructors happy.
   browser2->CloseAllTabs();
   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 comment before commit

[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());
 
   // Create more browsers/windows.
   scoped_ptr<Browser> browser2(new Browser(Browser::TYPE_TABBED, profile()));
   scoped_ptr<Browser> browser3(new Browser(Browser::TYPE_TABBED, profile()));
 
   scoped_ptr<TestBrowserWindow> window2(new TestBrowserWindow(browser2.get()));
   scoped_ptr<TestBrowserWindow> window3(new TestBrowserWindow(browser3.get()));
@@ skipping 60 matching lines @@
   EXPECT_EQ(6U, CountAllTabs());
 
   // Delete all tabs to clean up.
   for (std::vector<TabContentsWrapper*>::iterator it = owned_tabs.begin();
        it != owned_tabs.end(); ++it) {
     delete *it;
   }
 
   EXPECT_EQ(0U, CountAllTabs());
 }
+#endif