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

Side by Side Diff: chrome/browser/tabs/tab_strip_model_order_controller.cc

Issue 7215003: Multi-tab: Adding new Notification when tab selection changes (again). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebasing Created 9 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/tabs/tab_strip_model_order_controller.h" 5 #include "chrome/browser/tabs/tab_strip_model_order_controller.h"
6 6
7 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 7 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
8 8
9 /////////////////////////////////////////////////////////////////////////////// 9 ///////////////////////////////////////////////////////////////////////////////
10 // TabStripModelOrderController, public: 10 // TabStripModelOrderController, public:
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 return selected_index - 1; 104 return selected_index - 1;
105 105
106 return selected_index; 106 return selected_index;
107 } 107 }
108 108
109 void TabStripModelOrderController::ActiveTabChanged( 109 void TabStripModelOrderController::ActiveTabChanged(
110 TabContentsWrapper* old_contents, 110 TabContentsWrapper* old_contents,
111 TabContentsWrapper* new_contents, 111 TabContentsWrapper* new_contents,
112 int index, 112 int index,
113 bool user_gesture) { 113 bool user_gesture) {
114 if (old_contents == new_contents)
115 return;
116
117 NavigationController* old_opener = NULL; 114 NavigationController* old_opener = NULL;
118 if (old_contents) { 115 if (old_contents) {
119 int index = tabstrip_->GetIndexOfTabContents(old_contents); 116 int index = tabstrip_->GetIndexOfTabContents(old_contents);
120 if (index != TabStripModel::kNoTab) { 117 if (index != TabStripModel::kNoTab) {
121 old_opener = tabstrip_->GetOpenerOfTabContentsAt(index); 118 old_opener = tabstrip_->GetOpenerOfTabContentsAt(index);
122 119
123 // Forget any group/opener relationships that need to be reset whenever 120 // Forget any group/opener relationships that need to be reset whenever
124 // selection changes (see comment in TabStripModel::AddTabContentsAt). 121 // selection changes (see comment in TabStripModel::AddTabContentsAt).
125 if (tabstrip_->ShouldResetGroupOnSelect(old_contents)) 122 if (tabstrip_->ShouldResetGroupOnSelect(old_contents))
126 tabstrip_->ForgetGroup(old_contents); 123 tabstrip_->ForgetGroup(old_contents);
(...skipping 13 matching lines...) Expand all
140 137
141 /////////////////////////////////////////////////////////////////////////////// 138 ///////////////////////////////////////////////////////////////////////////////
142 // TabStripModelOrderController, private: 139 // TabStripModelOrderController, private:
143 140
144 int TabStripModelOrderController::GetValidIndex( 141 int TabStripModelOrderController::GetValidIndex(
145 int index, int removing_index) const { 142 int index, int removing_index) const {
146 if (removing_index < index) 143 if (removing_index < index)
147 index = std::max(0, index - 1); 144 index = std::max(0, index - 1);
148 return index; 145 return index;
149 } 146 }
OLDNEW
« no previous file with comments | « chrome/browser/tabs/tab_strip_model_observer.cc ('k') | chrome/browser/tabs/tab_strip_model_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698