Make most browser accelerators not repeat.

chrome/browser/ui/views/accelerator_table_unittest.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 <stddef.h>
 
 #include <set>
 
 #include "build/build_config.h"
 #include "chrome/browser/ui/views/accelerator_table.h"
@@ skipping 27 matching lines @@
            accelerator_list.begin(); it != accelerator_list.end(); ++it) {
     const AcceleratorMapping& entry = *it;
     EXPECT_TRUE(accelerators.insert(entry).second)
         << "Duplicated accelerator: " << entry.keycode << ", "
         << (entry.modifiers & ui::EF_SHIFT_DOWN) << ", "
         << (entry.modifiers & ui::EF_CONTROL_DOWN) << ", "
         << (entry.modifiers & ui::EF_ALT_DOWN);
   }
 }
 
+TEST(AcceleratorTableTest, CheckDuplicatedRepeatableCommandIds) {
+  std::set<int> commands;
+  const std::vector<int> command_list(GetRepeatableCommandIds());
+  for (int command : command_list) {
+    EXPECT_TRUE(commands.insert(command).second)
+        << "Duplicated command ID " << command;
+  }
+}
+
 #if defined(OS_CHROMEOS)
 TEST(AcceleratorTableTest, CheckDuplicatedAcceleratorsAsh) {
   std::set<AcceleratorMapping, Cmp> accelerators;
   const std::vector<AcceleratorMapping> accelerator_list(GetAcceleratorList());
   for (std::vector<AcceleratorMapping>::const_iterator it =
            accelerator_list.begin(); it != accelerator_list.end(); ++it) {
     const AcceleratorMapping& entry = *it;
     accelerators.insert(entry);
   }
   for (size_t i = 0; i < ash::kAcceleratorDataLength; ++i) {
@@ skipping 16 matching lines @@
     EXPECT_TRUE(accelerators.insert(entry).second)
         << "Duplicated accelerator: " << entry.keycode << ", "
         << (entry.modifiers & ui::EF_SHIFT_DOWN) << ", "
         << (entry.modifiers & ui::EF_CONTROL_DOWN) << ", "
         << (entry.modifiers & ui::EF_ALT_DOWN);
   }
 }
 #endif  // USE_ASH
 
 }  // namespace chrome