diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-02 21:45:56 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-02 21:45:56 +0000 |
| commit | 02cd2bfab054f3f21f3d4972831933f0e588290d (patch) | |
| tree | 5e5e1b958f2435236d105b50349bdb66cbf94f14 /chrome/browser/aeropeek_manager.cc | |
| parent | d5b20e5d794c1472870bf7ba31d55377d2be99a5 (diff) | |
| download | chromium_src-02cd2bfab054f3f21f3d4972831933f0e588290d.zip chromium_src-02cd2bfab054f3f21f3d4972831933f0e588290d.tar.gz chromium_src-02cd2bfab054f3f21f3d4972831933f0e588290d.tar.bz2 | |
Changes semantics of TabSelectedAt. For multi-selection the TabStrip
is going to still have the notion of the selected tab, but I need some
way to notify observers when other aspects of the selection
change. For example, if you shift click to extend the selection the
'selected tab' won't change. I'm changing TabSelectedAt so that if the
selected TabContents hasn't changed the old/new are the same.
This change only adds the necessary code to ignore the new
semantics. Code that actually invokes TabSelectedAt will come later.
BUG=30572
TEST=none
Review URL: http://codereview.chromium.org/6591097
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76619 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/aeropeek_manager.cc')
| -rw-r--r-- | chrome/browser/aeropeek_manager.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/chrome/browser/aeropeek_manager.cc b/chrome/browser/aeropeek_manager.cc index af6a054..4d1159d 100644 --- a/chrome/browser/aeropeek_manager.cc +++ b/chrome/browser/aeropeek_manager.cc @@ -1123,6 +1123,9 @@ void AeroPeekManager::TabSelectedAt(TabContentsWrapper* old_contents, TabContentsWrapper* new_contents, int index, bool user_gesture) { + if (old_contents == new_contents) + return; + // Deactivate the old window in the thumbnail list and activate the new one // to synchronize the thumbnail list with TabStrip. if (old_contents) { |