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

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

Issue 10537062: TabContentsWrapper -> TabContents, part 13. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fiz Created 8 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/ui/tabs/tab_strip_model_order_controller.h" 5 #include "chrome/browser/ui/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.h"
8 #include "content/public/browser/web_contents.h" 8 #include "content/public/browser/web_contents.h"
9 9
10 using content::NavigationController; 10 using content::NavigationController;
11 11
12 /////////////////////////////////////////////////////////////////////////////// 12 ///////////////////////////////////////////////////////////////////////////////
13 // TabStripModelOrderController, public: 13 // TabStripModelOrderController, public:
14 14
15 TabStripModelOrderController::TabStripModelOrderController( 15 TabStripModelOrderController::TabStripModelOrderController(
16 TabStripModel* tabstrip) 16 TabStripModel* tabstrip)
17 : tabstrip_(tabstrip), 17 : tabstrip_(tabstrip),
18 insertion_policy_(TabStripModel::INSERT_AFTER) { 18 insertion_policy_(TabStripModel::INSERT_AFTER) {
19 tabstrip_->AddObserver(this); 19 tabstrip_->AddObserver(this);
20 } 20 }
21 21
22 TabStripModelOrderController::~TabStripModelOrderController() { 22 TabStripModelOrderController::~TabStripModelOrderController() {
23 tabstrip_->RemoveObserver(this); 23 tabstrip_->RemoveObserver(this);
24 } 24 }
25 25
26 int TabStripModelOrderController::DetermineInsertionIndex( 26 int TabStripModelOrderController::DetermineInsertionIndex(
27 TabContentsWrapper* new_contents, 27 TabContents* new_contents,
28 content::PageTransition transition, 28 content::PageTransition transition,
29 bool foreground) { 29 bool foreground) {
30 int tab_count = tabstrip_->count(); 30 int tab_count = tabstrip_->count();
31 if (!tab_count) 31 if (!tab_count)
32 return 0; 32 return 0;
33 33
34 // NOTE: TabStripModel enforces that all non-mini-tabs occur after mini-tabs, 34 // NOTE: TabStripModel enforces that all non-mini-tabs occur after mini-tabs,
35 // so we don't have to check here too. 35 // so we don't have to check here too.
36 if (transition == content::PAGE_TRANSITION_LINK && 36 if (transition == content::PAGE_TRANSITION_LINK &&
37 tabstrip_->active_index() != -1) { 37 tabstrip_->active_index() != -1) {
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 105
106 // No opener set, fall through to the default handler... 106 // No opener set, fall through to the default handler...
107 int selected_index = tabstrip_->active_index(); 107 int selected_index = tabstrip_->active_index();
108 if (selected_index >= (tab_count - 1)) 108 if (selected_index >= (tab_count - 1))
109 return selected_index - 1; 109 return selected_index - 1;
110 110
111 return selected_index; 111 return selected_index;
112 } 112 }
113 113
114 void TabStripModelOrderController::ActiveTabChanged( 114 void TabStripModelOrderController::ActiveTabChanged(
115 TabContentsWrapper* old_contents, 115 TabContents* old_contents,
116 TabContentsWrapper* new_contents, 116 TabContents* new_contents,
117 int index, 117 int index,
118 bool user_gesture) { 118 bool user_gesture) {
119 NavigationController* old_opener = NULL; 119 NavigationController* old_opener = NULL;
120 if (old_contents) { 120 if (old_contents) {
121 int index = tabstrip_->GetIndexOfTabContents(old_contents); 121 int index = tabstrip_->GetIndexOfTabContents(old_contents);
122 if (index != TabStripModel::kNoTab) { 122 if (index != TabStripModel::kNoTab) {
123 old_opener = tabstrip_->GetOpenerOfTabContentsAt(index); 123 old_opener = tabstrip_->GetOpenerOfTabContentsAt(index);
124 124
125 // Forget any group/opener relationships that need to be reset whenever 125 // Forget any group/opener relationships that need to be reset whenever
126 // selection changes (see comment in TabStripModel::AddTabContentsAt). 126 // selection changes (see comment in TabStripModel::AddTabContentsAt).
(...skipping 14 matching lines...) Expand all
141 141
142 /////////////////////////////////////////////////////////////////////////////// 142 ///////////////////////////////////////////////////////////////////////////////
143 // TabStripModelOrderController, private: 143 // TabStripModelOrderController, private:
144 144
145 int TabStripModelOrderController::GetValidIndex( 145 int TabStripModelOrderController::GetValidIndex(
146 int index, int removing_index) const { 146 int index, int removing_index) const {
147 if (removing_index < index) 147 if (removing_index < index)
148 index = std::max(0, index - 1); 148 index = std::max(0, index - 1);
149 return index; 149 return index;
150 } 150 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/tabs/tab_strip_model_order_controller.h ('k') | chrome/browser/ui/tabs/tab_strip_model_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698