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

Side by Side Diff: chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc

Issue 9854043: Fix some grammar in chrome/browser/ui/views/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 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/views/tabs/browser_tab_strip_controller.h" 5 #include "chrome/browser/ui/views/tabs/browser_tab_strip_controller.h"
6 6
7 #include "base/auto_reset.h" 7 #include "base/auto_reset.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "chrome/browser/extensions/extension_tab_helper.h" 9 #include "chrome/browser/extensions/extension_tab_helper.h"
10 #include "chrome/browser/favicon/favicon_tab_helper.h" 10 #include "chrome/browser/favicon/favicon_tab_helper.h"
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 model_->AddObserver(this); 140 model_->AddObserver(this);
141 141
142 notification_registrar_.Add(this, 142 notification_registrar_.Add(this,
143 chrome::NOTIFICATION_TAB_CLOSEABLE_STATE_CHANGED, 143 chrome::NOTIFICATION_TAB_CLOSEABLE_STATE_CHANGED,
144 content::NotificationService::AllSources()); 144 content::NotificationService::AllSources());
145 } 145 }
146 146
147 BrowserTabStripController::~BrowserTabStripController() { 147 BrowserTabStripController::~BrowserTabStripController() {
148 // When we get here the TabStrip is being deleted. We need to explicitly 148 // When we get here the TabStrip is being deleted. We need to explicitly
149 // cancel the menu, otherwise it may try to invoke something on the tabstrip 149 // cancel the menu, otherwise it may try to invoke something on the tabstrip
150 // from it's destructor. 150 // from its destructor.
151 if (context_menu_contents_.get()) 151 if (context_menu_contents_.get())
152 context_menu_contents_->Cancel(); 152 context_menu_contents_->Cancel();
153 153
154 model_->RemoveObserver(this); 154 model_->RemoveObserver(this);
155 } 155 }
156 156
157 void BrowserTabStripController::InitFromModel(TabStrip* tabstrip) { 157 void BrowserTabStripController::InitFromModel(TabStrip* tabstrip) {
158 tabstrip_ = tabstrip; 158 tabstrip_ = tabstrip;
159 // Walk the model, calling our insertion observer method for each item within 159 // Walk the model, calling our insertion observer method for each item within
160 // it. 160 // it.
(...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
478 478
479 void BrowserTabStripController::StopHighlightTabsForCommand( 479 void BrowserTabStripController::StopHighlightTabsForCommand(
480 TabStripModel::ContextMenuCommand command_id, 480 TabStripModel::ContextMenuCommand command_id,
481 BaseTab* tab) { 481 BaseTab* tab) {
482 if (command_id == TabStripModel::CommandCloseTabsToRight || 482 if (command_id == TabStripModel::CommandCloseTabsToRight ||
483 command_id == TabStripModel::CommandCloseOtherTabs) { 483 command_id == TabStripModel::CommandCloseOtherTabs) {
484 // Just tell all Tabs to stop pulsing - it's safe. 484 // Just tell all Tabs to stop pulsing - it's safe.
485 tabstrip_->StopAllHighlighting(); 485 tabstrip_->StopAllHighlighting();
486 } 486 }
487 } 487 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/browser_view_layout.cc ('k') | chrome/browser/ui/views/tabs/tab_drag_controller2.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698