Deprecating high-conflict accelerators

ash/accelerators/accelerator_table_unittest.cc

Index: ash/accelerators/accelerator_table_unittest.cc
diff --git a/ash/accelerators/accelerator_table_unittest.cc b/ash/accelerators/accelerator_table_unittest.cc
index b04c579594e443b1da57f8abec1ba59ba4800abd..c62f17347678391f40790e282ec2e8afcc2f216e 100644
--- a/ash/accelerators/accelerator_table_unittest.cc
+++ b/ash/accelerators/accelerator_table_unittest.cc
@@ -76,4 +76,20 @@ TEST(AcceleratorTableTest, CheckDuplicatedNonrepeatableActions) {
   }
 }
 
+TEST(AcceleratorTableTest, CheckDeprecatedAccelerators) {
+  std::set<AcceleratorAction> deprecated_actions;
+  for (size_t i = 0; i < kDeprecatedAcceleratorsLength; ++i) {
+    // A deprecated action can never appear twice in the list.
+    AcceleratorAction deprecated_action =
+        kDeprecatedAccelerators[i].deprecated_accelerator.action;
+    EXPECT_TRUE(deprecated_actions.insert(deprecated_action).second)
+        << "Duplicated action: " << deprecated_action << " at index: " << i;
+
+    // The UMA histogram name must be of the format "Ash.Accelerators.*"
+    std::string uma_histogram(kDeprecatedAccelerators[i].uma_histogram_name);
+    EXPECT_TRUE(uma_histogram.find("Ash.Accelerators.") != std::string::npos);
+    EXPECT_TRUE(uma_histogram.find("Ash.Accelerators.") == 0);
+  }
+}
+
 }  // namespace ash