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

Unified Diff: chrome/browser/ui/browser_command_controller.cc

Issue 11308259: chrome: Extract CommandUpdaterDelegate into its own header file. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: FakeCommandObserver Created 8 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/browser_command_controller.cc
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc
index f6ec7f5b1b43f3730b298c2525afae8b6d7402b3..65094759fa995af425fc06a0d3151746f3120684 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -282,11 +282,11 @@ void BrowserCommandController::LoadingStateChanged(bool is_loading,
}
////////////////////////////////////////////////////////////////////////////////
-// BrowserCommandController,
-// CommandUpdater::CommandUpdaterDelegate implementation:
+// BrowserCommandController, CommandUpdaterDelegate implementation:
void BrowserCommandController::ExecuteCommandWithDisposition(
- int id, WindowOpenDisposition disposition) {
+ int id,
+ WindowOpenDisposition disposition) {
// No commands are enabled if there is not yet any selected tab.
// TODO(pkasting): It seems like we should not need this, because either
// most/all commands should not have been enabled yet anyway or the ones that

Powered by Google App Engine
This is Rietveld 408576698