diff options
Diffstat (limited to 'chrome/browser/views/tabs/tab_strip.cc')
-rw-r--r-- | chrome/browser/views/tabs/tab_strip.cc | 24 |
1 files changed, 2 insertions, 22 deletions
diff --git a/chrome/browser/views/tabs/tab_strip.cc b/chrome/browser/views/tabs/tab_strip.cc index 68fc477..56bf512 100644 --- a/chrome/browser/views/tabs/tab_strip.cc +++ b/chrome/browser/views/tabs/tab_strip.cc @@ -253,7 +253,8 @@ TabStrip::TabStrip(TabStripModel* model) } TabStrip::~TabStrip() { - model_->RemoveObserver(this); + // TODO(beng): (1031854) Restore this line once XPFrame/VistaFrame are dead. + // model_->RemoveObserver(this); // TODO(beng): remove this if it doesn't work to fix the TabSelectedAt bug. drag_controller_.reset(NULL); @@ -328,15 +329,6 @@ gfx::Rect TabStrip::GetNewTabButtonBounds() { return newtab_button_->bounds(); } -void TabStrip::InitFromModel() { - // Walk the model, calling our insertion observer method for each item within - // it. - for (int i = 0; i < model_->count(); ++i) { - TabInsertedAt(model_->GetTabContentsAt(i), i, - i == model_->selected_index()); - } -} - //////////////////////////////////////////////////////////////////////////////// // TabStrip, BaseTabStrip implementation: @@ -856,18 +848,6 @@ bool TabStrip::IsCommandEnabledForTab( return false; } -bool TabStrip::IsCommandCheckedForTab( - TabStripModel::ContextMenuCommand command_id, const Tab* tab) const { - // TODO(beng): move to TabStripModel, see note in IsTabPinned. - if (command_id == TabStripModel::CommandTogglePinned) - return IsTabPinned(tab); - - int index = GetModelIndexOfTab(tab); - if (model_->ContainsIndex(index)) - return model_->IsContextMenuCommandChecked(index, command_id); - return false; -} - void TabStrip::ExecuteCommandForTab( TabStripModel::ContextMenuCommand command_id, Tab* tab) { int model_index = GetModelIndexOfTab(tab); |