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

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

Issue 291093005: Removes --enable-stacked-tab-strip flag (Stacked Tabs) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removes --enable-stacked-tab-strip flag (moved pref logic to browser_view_prefs) Created 6 years, 7 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/browser_tab_strip_controller.cc
diff --git a/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc b/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
index f88871dc5149f53c20676153dbd3832bc0dca539..7cb7394fb5aa9446e65189547e741ef0134f6cb8 100644
--- a/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
+++ b/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
@@ -59,25 +59,16 @@ TabRendererData::NetworkState TabContentsNetworkState(
return TabRendererData::NETWORK_STATE_LOADING;
}
-TabStripLayoutType DetermineTabStripLayout(
+bool DetermineTabStripLayoutStacked(
PrefService* prefs,
chrome::HostDesktopType host_desktop_type,
bool* adjust_layout) {
*adjust_layout = false;
- if (CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kEnableStackedTabStrip)) {
- return TAB_STRIP_LAYOUT_STACKED;
- }
// For ash, always allow entering stacked mode.
if (host_desktop_type != chrome::HOST_DESKTOP_TYPE_ASH)
- return TAB_STRIP_LAYOUT_SHRINK;
+ return false;
*adjust_layout = true;
- switch (prefs->GetInteger(prefs::kTabStripLayoutType)) {
- case TAB_STRIP_LAYOUT_STACKED:
- return TAB_STRIP_LAYOUT_STACKED;
- default:
- return TAB_STRIP_LAYOUT_SHRINK;
- }
+ return prefs->GetBoolean(prefs::kTabStripStackedLayout);
}
// Get the MIME type of the file pointed to by the url, based on the file's
@@ -201,7 +192,7 @@ BrowserTabStripController::BrowserTabStripController(Browser* browser,
local_pref_registrar_.Init(g_browser_process->local_state());
local_pref_registrar_.Add(
- prefs::kTabStripLayoutType,
+ prefs::kTabStripStackedLayout,
base::Bind(&BrowserTabStripController::UpdateLayoutType,
base::Unretained(this)));
}
@@ -392,15 +383,15 @@ bool BrowserTabStripController::IsIncognito() {
void BrowserTabStripController::LayoutTypeMaybeChanged() {
bool adjust_layout = false;
- TabStripLayoutType layout_type =
- DetermineTabStripLayout(g_browser_process->local_state(),
- browser_->host_desktop_type(), &adjust_layout);
- if (!adjust_layout || layout_type == tabstrip_->layout_type())
+ bool stacked_layout =
+ DetermineTabStripLayoutStacked(g_browser_process->local_state(),
+ browser_->host_desktop_type(),
+ &adjust_layout);
+ if (!adjust_layout || stacked_layout == tabstrip_->stacked_layout())
return;
- g_browser_process->local_state()->SetInteger(
- prefs::kTabStripLayoutType,
- static_cast<int>(tabstrip_->layout_type()));
+ g_browser_process->local_state()->SetBoolean(prefs::kTabStripStackedLayout,
+ tabstrip_->stacked_layout());
}
void BrowserTabStripController::OnStartedDraggingTabs() {
@@ -570,10 +561,12 @@ void BrowserTabStripController::AddTab(WebContents* contents,
void BrowserTabStripController::UpdateLayoutType() {
bool adjust_layout = false;
- TabStripLayoutType layout_type =
- DetermineTabStripLayout(g_browser_process->local_state(),
- browser_->host_desktop_type(), &adjust_layout);
- tabstrip_->SetLayoutType(layout_type, adjust_layout);
+ bool stacked_layout =
+ DetermineTabStripLayoutStacked(g_browser_process->local_state(),
+ browser_->host_desktop_type(),
+ &adjust_layout);
+ tabstrip_->set_adjust_layout(adjust_layout);
+ tabstrip_->SetStackedLayout(stacked_layout);
}
void BrowserTabStripController::OnFindURLMimeTypeCompleted(

Powered by Google App Engine
This is Rietveld 408576698