summaryrefslogtreecommitdiffstats
path: root/chrome/browser/tabs
diff options
context:
space:
mode:
Diffstat (limited to 'chrome/browser/tabs')
-rw-r--r--chrome/browser/tabs/default_tab_handler.cc8
-rw-r--r--chrome/browser/tabs/default_tab_handler.h2
-rw-r--r--chrome/browser/tabs/tab_strip_model.cc18
-rw-r--r--chrome/browser/tabs/tab_strip_model.h5
-rw-r--r--chrome/browser/tabs/tab_strip_model_delegate.h6
-rw-r--r--chrome/browser/tabs/tab_strip_model_observer.cc4
-rw-r--r--chrome/browser/tabs/tab_strip_model_observer.h4
-rw-r--r--chrome/browser/tabs/tab_strip_model_unittest.cc2
8 files changed, 1 insertions, 48 deletions
diff --git a/chrome/browser/tabs/default_tab_handler.cc b/chrome/browser/tabs/default_tab_handler.cc
index 1524852..b81f0a2 100644
--- a/chrome/browser/tabs/default_tab_handler.cc
+++ b/chrome/browser/tabs/default_tab_handler.cc
@@ -115,10 +115,6 @@ void DefaultTabHandler::ToggleUseVerticalTabs() {
delegate_->AsBrowser()->ToggleUseVerticalTabs();
}
-void DefaultTabHandler::ToggleUseCompactNavigationBar() {
- delegate_->AsBrowser()->ToggleUseCompactNavigationBar();
-}
-
bool DefaultTabHandler::CanRestoreTab() {
return delegate_->AsBrowser()->CanRestoreTab();
}
@@ -135,10 +131,6 @@ bool DefaultTabHandler::UseVerticalTabs() const {
return delegate_->AsBrowser()->UseVerticalTabs();
}
-bool DefaultTabHandler::UseCompactNavigationBar() const {
- return delegate_->AsBrowser()->UseCompactNavigationBar();
-}
-
////////////////////////////////////////////////////////////////////////////////
// DefaultTabHandler, TabStripModelObserver implementation:
diff --git a/chrome/browser/tabs/default_tab_handler.h b/chrome/browser/tabs/default_tab_handler.h
index f59dd25..09f779a 100644
--- a/chrome/browser/tabs/default_tab_handler.h
+++ b/chrome/browser/tabs/default_tab_handler.h
@@ -52,12 +52,10 @@ class DefaultTabHandler : public TabHandler,
virtual void BookmarkAllTabs();
virtual bool CanCloseTab() const;
virtual void ToggleUseVerticalTabs();
- virtual void ToggleUseCompactNavigationBar();
virtual bool CanRestoreTab();
virtual void RestoreTab();
virtual bool LargeIconsPermitted() const;
virtual bool UseVerticalTabs() const;
- virtual bool UseCompactNavigationBar() const;
// Overridden from TabStripModelObserver:
virtual void TabInsertedAt(TabContentsWrapper* contents,
diff --git a/chrome/browser/tabs/tab_strip_model.cc b/chrome/browser/tabs/tab_strip_model.cc
index d1ded2f..fa632af 100644
--- a/chrome/browser/tabs/tab_strip_model.cc
+++ b/chrome/browser/tabs/tab_strip_model.cc
@@ -711,10 +711,6 @@ void TabStripModel::MoveTabPrevious() {
MoveTabContentsAt(active_index(), new_index, true);
}
-void TabStripModel::ActiveTabClicked(int index) {
- FOR_EACH_OBSERVER(TabStripModelObserver, observers_, ActiveTabClicked(index));
-}
-
// Context menu functions.
bool TabStripModel::IsContextMenuCommandEnabled(
int context_index, ContextMenuCommand command_id) const {
@@ -768,7 +764,6 @@ bool TabStripModel::IsContextMenuCommandEnabled(
delegate_->CanBookmarkAllTabs();
case CommandUseVerticalTabs:
- case CommandUseCompactNavigationBar:
return true;
case CommandSelectByDomain:
@@ -787,8 +782,6 @@ bool TabStripModel::IsContextMenuCommandChecked(
switch (command_id) {
case CommandUseVerticalTabs:
return delegate()->UseVerticalTabs();
- case CommandUseCompactNavigationBar:
- return delegate()->UseCompactNavigationBar();
default:
NOTREACHED();
break;
@@ -911,14 +904,6 @@ void TabStripModel::ExecuteContextMenuCommand(
break;
}
- case CommandUseCompactNavigationBar: {
- UserMetrics::RecordAction(
- UserMetricsAction("TabContextMenu_CompactNavigationBar"));
-
- delegate()->ToggleUseCompactNavigationBar();
- break;
- }
-
case CommandSelectByDomain:
case CommandSelectByOpener: {
std::vector<int> indices;
@@ -1044,9 +1029,6 @@ bool TabStripModel::ContextMenuCommandToBrowserCommand(int cmd_id,
case CommandUseVerticalTabs:
*browser_cmd = IDC_TOGGLE_VERTICAL_TABS;
break;
- case CommandUseCompactNavigationBar:
- *browser_cmd = IDC_COMPACT_NAVBAR;
- break;
default:
*browser_cmd = 0;
return false;
diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h
index c0d1e0b..f236e5f 100644
--- a/chrome/browser/tabs/tab_strip_model.h
+++ b/chrome/browser/tabs/tab_strip_model.h
@@ -411,10 +411,6 @@ class TabStripModel : public NotificationObserver {
void MoveTabNext();
void MoveTabPrevious();
- // Notifies the observers that the active/foreground tab at |index| was
- // reselected (ie - it was already active and was clicked again).
- void ActiveTabClicked(int index);
-
// View API //////////////////////////////////////////////////////////////////
// Context menu functions.
@@ -430,7 +426,6 @@ class TabStripModel : public NotificationObserver {
CommandTogglePinned,
CommandBookmarkAllTabs,
CommandUseVerticalTabs,
- CommandUseCompactNavigationBar,
CommandSelectByDomain,
CommandSelectByOpener,
CommandLast
diff --git a/chrome/browser/tabs/tab_strip_model_delegate.h b/chrome/browser/tabs/tab_strip_model_delegate.h
index 0279e84..bae327e 100644
--- a/chrome/browser/tabs/tab_strip_model_delegate.h
+++ b/chrome/browser/tabs/tab_strip_model_delegate.h
@@ -118,12 +118,6 @@ class TabStripModelDelegate {
// Toggles the use of the vertical tabstrip.
virtual void ToggleUseVerticalTabs() = 0;
- // Returns true if the compact navigation bar should be used.
- virtual bool UseCompactNavigationBar() const = 0;
-
- // Toggles the use of the compact navigation bar.
- virtual void ToggleUseCompactNavigationBar() = 0;
-
// Returns true if the tab strip can use large icons.
virtual bool LargeIconsPermitted() const = 0;
diff --git a/chrome/browser/tabs/tab_strip_model_observer.cc b/chrome/browser/tabs/tab_strip_model_observer.cc
index 0f5395b..f20746a 100644
--- a/chrome/browser/tabs/tab_strip_model_observer.cc
+++ b/chrome/browser/tabs/tab_strip_model_observer.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 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.
@@ -58,5 +58,3 @@ void TabStripModelObserver::TabBlockedStateChanged(TabContentsWrapper* contents,
void TabStripModelObserver::TabStripEmpty() {}
void TabStripModelObserver::TabStripModelDeleted() {}
-
-void TabStripModelObserver::ActiveTabClicked(int index) {}
diff --git a/chrome/browser/tabs/tab_strip_model_observer.h b/chrome/browser/tabs/tab_strip_model_observer.h
index ba9df25..af169ea 100644
--- a/chrome/browser/tabs/tab_strip_model_observer.h
+++ b/chrome/browser/tabs/tab_strip_model_observer.h
@@ -127,10 +127,6 @@ class TabStripModelObserver {
// must be dropped.
virtual void TabStripModelDeleted();
- // Invoked when an active/selected tab at |index| is selected again (ie - the
- // active/foreground tab is clicked).
- virtual void ActiveTabClicked(int index);
-
protected:
virtual ~TabStripModelObserver() {}
};
diff --git a/chrome/browser/tabs/tab_strip_model_unittest.cc b/chrome/browser/tabs/tab_strip_model_unittest.cc
index fc1c941..4876c81 100644
--- a/chrome/browser/tabs/tab_strip_model_unittest.cc
+++ b/chrome/browser/tabs/tab_strip_model_unittest.cc
@@ -130,8 +130,6 @@ class TabStripDummyDelegate : public TabStripModelDelegate {
virtual bool CanCloseTab() const { return true; }
virtual bool UseVerticalTabs() const { return false; }
virtual void ToggleUseVerticalTabs() {}
- virtual bool UseCompactNavigationBar() const { return false; }
- virtual void ToggleUseCompactNavigationBar() {}
virtual bool LargeIconsPermitted() const { return true; }
private: