Multi-tab: Fixing TabStripSelectionModel::SetSelectedIndex corner case.

chrome/browser/tabs/tab_strip_selection_model.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/tabs/tab_strip_selection_model.h"
 
 #include <algorithm>
 #include <valarray>
 
 #include "base/logging.h"
@@ skipping 41 matching lines @@
       i = selected_indices_.erase(i);
     else
       ++i;
   }
   DecrementFromImpl(index, &anchor_);
   DecrementFromImpl(index, &active_);
 }
 
 void TabStripSelectionModel::SetSelectedIndex(int index) {
   anchor_ = active_ = index;
-  SetSelectionFromAnchorTo(index);
+  selected_indices_.clear();
+  selected_indices_.push_back(index);

[sky, 2011/06/08 20:38:49]

Only push_back if index != kUnselectedIndex.

[dpapad, 2011/06/08 21:19:34]

Done.

 }
 
 bool TabStripSelectionModel::IsSelected(int index) const {
   return std::find(selected_indices_.begin(), selected_indices_.end(), index) !=
       selected_indices_.end();
 }
 
 void TabStripSelectionModel::AddIndexToSelection(int index) {
   if (!IsSelected(index)) {
     selected_indices_.push_back(index);
@@ skipping 59 matching lines @@
   anchor_ = active_ = kUnselectedIndex;
   SelectedIndices empty_selection;
   selected_indices_.swap(empty_selection);
 }
 
 void TabStripSelectionModel::Copy(const TabStripSelectionModel& source) {
   selected_indices_ = source.selected_indices_;
   active_ = source.active_;
   anchor_ = source.anchor_;
 }