MacViews: Fix duplicate definition of ExtensionKeyBindingRegistry::SetShortcutHandlingSuspended

chrome/browser/extensions/extension_keybinding_registry.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/extensions/extension_keybinding_registry.h"
 
 #include "base/values.h"
 #include "chrome/browser/extensions/active_tab_permission_granter.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/command.h"
@@ skipping 10 matching lines @@
 
 namespace extensions {
 
 ExtensionKeybindingRegistry::ExtensionKeybindingRegistry(
     content::BrowserContext* context,
     ExtensionFilter extension_filter,
     Delegate* delegate)
     : browser_context_(context),
       extension_filter_(extension_filter),
       delegate_(delegate),
-      extension_registry_observer_(this) {
+      extension_registry_observer_(this),
+      shortcut_handling_suspended_(false) {
   extension_registry_observer_.Add(ExtensionRegistry::Get(browser_context_));
 
   Profile* profile = Profile::FromBrowserContext(browser_context_);
   registrar_.Add(this,
                  extensions::NOTIFICATION_EXTENSION_COMMAND_ADDED,
                  content::Source<Profile>(profile->GetOriginalProfile()));
   registrar_.Add(this,
                  extensions::NOTIFICATION_EXTENSION_COMMAND_REMOVED,
                  content::Source<Profile>(profile->GetOriginalProfile()));
 }
 
 ExtensionKeybindingRegistry::~ExtensionKeybindingRegistry() {
 }
 
+void ExtensionKeybindingRegistry::set_shortcut_handling_suspended(

[Finnur, 2015/01/16 10:05:05]

nit: Should not be unix_hacker style (calls virtual function).

[Devlin, 2015/01/16 15:49:55]

I probably caused this confusion; I only meant that the accessor should be
hacker_style (related: it [the accessor] should also be defined in the header).

[Andre, 2015/01/16 18:40:15]

Done.

+    bool suspended) {
+  shortcut_handling_suspended_ = suspended;
+  OnShortcutHandlingSuspended(suspended);
+}
+
+bool ExtensionKeybindingRegistry::shortcut_handling_suspended() const {
+  return shortcut_handling_suspended_;
+}
+
 void ExtensionKeybindingRegistry::RemoveExtensionKeybinding(
     const Extension* extension,
     const std::string& command_name) {
   EventTargets::iterator it = event_targets_.begin();
   while (it != event_targets_.end()) {
     TargetList& target_list = it->second;
     TargetList::iterator target = target_list.begin();
     while (target != target_list.end()) {
       if (target->first == extension->id() &&
           (command_name.empty() || command_name == target->second))
@@ skipping 192 matching lines @@
     if (extension_id.empty() || it->first == extension_id) {
       CommandExecuted(it->first, it->second);
       executed = true;
     }
   }
 
   return executed;
 }
 
 }  // namespace extensions