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

Side by Side Diff: chrome/browser/tabs/tab_strip_model_observer.h

Issue 6933037: Multi-tab selection for Linux. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adding TabSelectionChanged callback and removing unnecessary method Created 9 years, 7 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 #ifndef CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_
6 #define CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_ 6 #define CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_
7 #pragma once 7 #pragma once
8 8
9 #include <set>
10
9 class TabContentsWrapper; 11 class TabContentsWrapper;
10 class TabStripModel; 12 class TabStripModel;
11 13
12 //////////////////////////////////////////////////////////////////////////////// 14 ////////////////////////////////////////////////////////////////////////////////
13 // 15 //
14 // TabStripModelObserver 16 // TabStripModelObserver
15 // 17 //
16 // Objects implement this interface when they wish to be notified of changes 18 // Objects implement this interface when they wish to be notified of changes
17 // to the TabStripModel. 19 // to the TabStripModel.
18 // 20 //
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 // 76 //
75 // TODO(sky): consider not overloading this. Instead rename this to 77 // TODO(sky): consider not overloading this. Instead rename this to
76 // TabActivatedAt (or something) and have TabSelectionChanged as well. 78 // TabActivatedAt (or something) and have TabSelectionChanged as well.
77 // TabSelectedAt. This requires renaming everyone to use new terms instead of 79 // TabSelectedAt. This requires renaming everyone to use new terms instead of
78 // selection. 80 // selection.
79 virtual void TabSelectedAt(TabContentsWrapper* old_contents, 81 virtual void TabSelectedAt(TabContentsWrapper* old_contents,
80 TabContentsWrapper* new_contents, 82 TabContentsWrapper* new_contents,
81 int index, 83 int index,
82 bool user_gesture); 84 bool user_gesture);
83 85
86 virtual void TabSelectionChanged(std::set<int>& indices_affected);
James Hawkins 2011/05/18 18:03:10 Document this method.
dpapad 2011/06/01 18:05:41 Done.
87
84 // The specified TabContents at |from_index| was moved to |to_index|. 88 // The specified TabContents at |from_index| was moved to |to_index|.
85 virtual void TabMoved(TabContentsWrapper* contents, 89 virtual void TabMoved(TabContentsWrapper* contents,
86 int from_index, 90 int from_index,
87 int to_index); 91 int to_index);
88 92
89 // The specified TabContents at |index| changed in some way. |contents| may 93 // The specified TabContents at |index| changed in some way. |contents| may
90 // be an entirely different object and the old value is no longer available 94 // be an entirely different object and the old value is no longer available
91 // by the time this message is delivered. 95 // by the time this message is delivered.
92 // 96 //
93 // See TabChangeType for a description of |change_type|. 97 // See TabChangeType for a description of |change_type|.
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 133
130 // Invoked when an active/selected tab at |index| is selected again (ie - the 134 // Invoked when an active/selected tab at |index| is selected again (ie - the
131 // active/foreground tab is clicked). 135 // active/foreground tab is clicked).
132 virtual void ActiveTabClicked(int index); 136 virtual void ActiveTabClicked(int index);
133 137
134 protected: 138 protected:
135 virtual ~TabStripModelObserver() {} 139 virtual ~TabStripModelObserver() {}
136 }; 140 };
137 141
138 #endif // CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_ 142 #endif // CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698