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

Unified Diff: chrome/browser/browser.cc

Issue 3056003: Attemp 2 at: (Closed)
Patch Set: Fix chromeos breakage Created 10 years, 5 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 | « chrome/browser/browser.h ('k') | chrome/browser/browser_window.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser.cc
diff --git a/chrome/browser/browser.cc b/chrome/browser/browser.cc
index 51c77643ad4c823a57080ed6812f5eba89b66acf..21ace5e2c36c7d11e229bf5110f48b41cdb30be0 100644
--- a/chrome/browser/browser.cc
+++ b/chrome/browser/browser.cc
@@ -1151,9 +1151,6 @@ bool Browser::SupportsWindowFeatureImpl(WindowFeature feature,
if (type() == TYPE_NORMAL || type() == TYPE_EXTENSION_APP)
features |= FEATURE_TABSTRIP;
- // TODO(aa): This is kinda a hack. The toolbar is not really there, it is
- // collapsed. We probably want to add a FEATURE_MINI_TOOLBAR to represent
- // the collapsed state.
if (type() == TYPE_NORMAL || type() == TYPE_EXTENSION_APP)
features |= FEATURE_TOOLBAR;
@@ -2367,10 +2364,6 @@ void Browser::ToggleUseVerticalTabs() {
UseVerticalTabsChanged();
}
-void Browser::SetToolbarVisibility(bool visible) {
- window()->SetToolbarCollapsedMode(!visible);
-}
-
///////////////////////////////////////////////////////////////////////////////
// Browser, TabStripModelObserver implementation:
@@ -2454,8 +2447,6 @@ void Browser::TabSelectedAt(TabContents* old_contents,
session_id(), tabstrip_model_.selected_index());
}
}
-
- window()->SetToolbarCollapsedMode(!tabstrip_model_.IsToolbarVisible(index));
}
void Browser::TabMoved(TabContents* contents,
« no previous file with comments | « chrome/browser/browser.h ('k') | chrome/browser/browser_window.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698