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..44aee35b10affdb24e4a22b81fc51a8bdc14da56 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,15 @@ 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()) {
+    CHECK(action_index < 0) << "Simulating action clicks for page notifications"

[Peter Beverloo, 2015/08/04 21:08:33]

Please make sure there's a TODO + a bug for this.

[johnme, 2015/08/05 13:51:27]

Added a TODO. Plumbing action clicks to page notifications is a core part of
issue 513671 though (and this is how it'll be tested), so no need for an
additional bug.

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