Enable testing NotificationEvent.action

content/shell/browser/layout_test/layout_test_notification_manager.cc

Index: content/shell/browser/layout_test/layout_test_notification_manager.cc
diff --git a/content/shell/browser/layout_test/layout_test_notification_manager.cc b/content/shell/browser/layout_test/layout_test_notification_manager.cc
index 1e405add13afad290655388e3f551b99d5029087..6505a9d9b267b1331d43ecd5ba9cde2d887c26ae 100644
--- a/content/shell/browser/layout_test/layout_test_notification_manager.cc
+++ b/content/shell/browser/layout_test/layout_test_notification_manager.cc
@@ -106,12 +106,16 @@ bool LayoutTestNotificationManager::GetDisplayedPersistentNotifications(
   return false;
 }
 
-void LayoutTestNotificationManager::SimulateClick(const std::string& title) {
+void LayoutTestNotificationManager::SimulateClick(const std::string& title,
+                                                  int action_index) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   // First check for page-notifications with the given title.
   const auto& page_iterator = page_notifications_.find(title);
   if (page_iterator != page_notifications_.end()) {
+    // TODO(johnme): Implement this.
+    CHECK(action_index < 0) << "Simulating action clicks for page notifications"

[jochen (gone - plz use gerrit), 2015/08/07 07:12:33]

clusterfuzz will love this one... maybe just return?

[johnme, 2015/08/07 14:53:33]

Done (I removed the CHECK; for now there's actually no harm in firing a plain
NotificationClick; https://codereview.chromium.org/1275743003 will fix this and
add tests).

+                            << " is not yet implemented";
     page_iterator->second->NotificationClick();
     return;
   }
@@ -127,7 +131,7 @@ void LayoutTestNotificationManager::SimulateClick(const std::string& title) {
           notification.browser_context,
           notification.persistent_id,
           notification.origin,
-          -1 /* action_index */,
+          action_index,
           base::Bind(&OnEventDispatchComplete));
 }