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

Unified Diff: chrome/browser/ui/views/tabs/tab_strip_unittest.cc

Issue 2368653002: Remove pre-MD code from browser/ui/views/tabs/tab.cc (Closed)
Patch Set: fix compile Created 4 years, 3 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
Index: chrome/browser/ui/views/tabs/tab_strip_unittest.cc
diff --git a/chrome/browser/ui/views/tabs/tab_strip_unittest.cc b/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
index e4c6cd0529a97e169b29169972475cbb7f32629d..b82e59839e32463d06c288952f961dbe8bb7ff26 100644
--- a/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
+++ b/chrome/browser/ui/views/tabs/tab_strip_unittest.cc
@@ -306,26 +306,6 @@ TEST_F(TabStripTest, VisibilityInOverflow) {
EXPECT_TRUE(tab_strip_->tab_at(tab_strip_->tab_count() - 1)->visible());
}
-TEST_F(TabStripTest, ImmersiveMode) {
- // Immersive mode defaults to off.
- EXPECT_FALSE(tab_strip_->IsImmersiveStyle());
-
- // Tab strip defaults to normal tab height.
- int normal_height = Tab::GetMinimumInactiveSize().height();
- EXPECT_EQ(normal_height, tab_strip_->GetPreferredSize().height());
-
- // Tab strip can toggle immersive mode.
- tab_strip_->SetImmersiveStyle(true);
- EXPECT_TRUE(tab_strip_->IsImmersiveStyle());
-
- // Now tabs have the immersive height.
- int immersive_height = Tab::GetImmersiveHeight();
- EXPECT_EQ(immersive_height, tab_strip_->GetPreferredSize().height());
-
- // Sanity-check immersive tabs are shorter than normal tabs.
- EXPECT_LT(immersive_height, normal_height);
-}
-
// Creates a tab strip in stacked layout mode and verifies the correctness
// of hit tests against the visible/occluded regions of a tab and the tab
// close button of the active tab.

Powered by Google App Engine
This is Rietveld 408576698