diff options
Diffstat (limited to 'chrome/browser/tabs')
-rw-r--r-- | chrome/browser/tabs/default_tab_handler.cc | 8 | ||||
-rw-r--r-- | chrome/browser/tabs/default_tab_handler.h | 2 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model.cc | 18 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model.h | 5 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model_delegate.h | 6 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model_observer.cc | 4 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model_observer.h | 4 | ||||
-rw-r--r-- | chrome/browser/tabs/tab_strip_model_unittest.cc | 2 |
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: |