chrome: Extract CommandUpdaterDelegate into its own header file.

chrome/browser/command_updater.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/command_updater.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/observer_list.h"
 #include "base/stl_util.h"
 #include "chrome/browser/command_observer.h"
-
-CommandUpdater::CommandUpdaterDelegate::~CommandUpdaterDelegate() {
-}
+#include "chrome/browser/command_updater_delegate.h"
 
 class CommandUpdater::Command {
  public:
   bool enabled;
   ObserverList<CommandObserver> observers;
 
   Command() : enabled(true) {}
 };
 
-CommandUpdater::CommandUpdater(CommandUpdaterDelegate* handler)
-    : delegate_(handler) {
+CommandUpdater::CommandUpdater(CommandUpdaterDelegate* delegate)
+    : delegate_(delegate) {
 }
 
 CommandUpdater::~CommandUpdater() {
   STLDeleteContainerPairSecondPointers(commands_.begin(), commands_.end());
 }
 
+bool CommandUpdater::SupportsCommand(int id) const {
+  return commands_.find(id) != commands_.end();
+}
+
 bool CommandUpdater::IsCommandEnabled(int id) const {
   const CommandMap::const_iterator command(commands_.find(id));
   if (command == commands_.end())
     return false;
   return command->second->enabled;
 }
 
-bool CommandUpdater::SupportsCommand(int id) const {
-  return commands_.find(id) != commands_.end();
-}
-
 bool CommandUpdater::ExecuteCommand(int id) {
   return ExecuteCommandWithDisposition(id, CURRENT_TAB);
 }
 
 bool CommandUpdater::ExecuteCommandWithDisposition(
     int id,
     WindowOpenDisposition disposition) {
   if (SupportsCommand(id) && IsCommandEnabled(id)) {
     delegate_->ExecuteCommandWithDisposition(id, disposition);
     return true;
   }
   return false;
 }
 
-CommandObserver::~CommandObserver() {
+void CommandUpdater::AddCommandObserver(int id, CommandObserver* observer) {
+  GetCommand(id, true)->observers.AddObserver(observer);
+}
+
+void CommandUpdater::RemoveCommandObserver(int id, CommandObserver* observer) {
+  GetCommand(id, false)->observers.RemoveObserver(observer);
+}
+
+void CommandUpdater::RemoveCommandObserver(CommandObserver* observer) {
+  for (CommandMap::const_iterator it = commands_.begin();
+       it != commands_.end();
+       ++it) {
+    Command* command = it->second;
+    if (command)
+      command->observers.RemoveObserver(observer);
+  }
 }
 
 void CommandUpdater::UpdateCommandEnabled(int id, bool enabled) {
   Command* command = GetCommand(id, true);
   if (command->enabled == enabled)
     return;  // Nothing to do.
   command->enabled = enabled;
   FOR_EACH_OBSERVER(CommandObserver, command->observers,
                     EnabledStateChangedForCommand(id, enabled));
 }
 
 CommandUpdater::Command* CommandUpdater::GetCommand(int id, bool create) {
   bool supported = SupportsCommand(id);
   if (supported)
     return commands_[id];
   DCHECK(create);
   Command* command = new Command;
   commands_[id] = command;
   return command;
 }
-
-void CommandUpdater::AddCommandObserver(int id, CommandObserver* observer) {
-  GetCommand(id, true)->observers.AddObserver(observer);
-}
-
-void CommandUpdater::RemoveCommandObserver(int id, CommandObserver* observer) {
-  GetCommand(id, false)->observers.RemoveObserver(observer);
-}
-
-void CommandUpdater::RemoveCommandObserver(CommandObserver* observer) {
-  for (CommandMap::const_iterator it = commands_.begin();
-       it != commands_.end();
-       ++it) {
-    Command* command = it->second;
-    if (command)
-      command->observers.RemoveObserver(observer);
-  }
-}