Add NotificationEvent.action property in SW events

Source/web/ServiceWorkerGlobalScopeProxy.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 59 matching lines @@
 
     // WebServiceWorkerContextProxy overrides:
     void setRegistration(WebServiceWorkerRegistration*) override;
     void dispatchActivateEvent(int) override;
     void dispatchCrossOriginMessageEvent(const WebCrossOriginServiceWorkerClient&, const WebString& message, const WebMessagePortChannelArray&) override;
     void dispatchFetchEvent(int, const WebServiceWorkerRequest&) override;
     void dispatchGeofencingEvent(int, WebGeofencingEventType, const WebString& regionID, const WebCircularGeofencingRegion&) override;
     void dispatchInstallEvent(int) override;
     void dispatchMessageEvent(const WebString& message, const WebMessagePortChannelArray&) override;
     void dispatchNotificationClickEvent(int, int64_t notificationID, const WebNotificationData&) override;
+    void dispatchNotificationClickEvent(int, int64_t notificationID, const WebNotificationData&, int actionIndex) override;

[Peter Beverloo, 2015/08/04 21:01:23]

Mm. Why did you decide to pass an int for the action index rather than passing
the action as a String? Now you need weird -1 values rather than passing the
empty string.

[johnme, 2015/08/05 11:56:22]

In the lower layers (ui::messagecenter::ButtonInfo etc) we don't know the
|action|s of the buttons, we only store their |title|s and get back a button
index. So passing the string would require a whole lot more plumbing.

I could define a kInvalidNotificationActionIndex constant in
content/public/common/platform_notification_data.h if you think that'd help?

     void dispatchPushEvent(int, const WebString& data) override;
     void dispatchServicePortConnectEvent(WebServicePortConnectEventCallbacks*, const WebURL& targetURL, const WebString& origin, WebServicePortID) override;
     void dispatchSyncEvent(int, const WebSyncRegistration&) override;
 
     // WorkerReportingProxy overrides:
     void reportException(const String& errorMessage, int lineNumber, int columnNumber, const String& sourceURL, int exceptionId) override;
     void reportConsoleMessage(PassRefPtrWillBeRawPtr<ConsoleMessage>) override;
     void postMessageToPageInspector(const String&) override;
     void postWorkerConsoleAgentEnabled() override { }
     void didEvaluateWorkerScript(bool success) override;
     void workerGlobalScopeStarted(WorkerGlobalScope*) override;
     void workerGlobalScopeClosed() override;
     void willDestroyWorkerGlobalScope() override;
     void workerThreadTerminated() override;
 
 private:
     ServiceWorkerGlobalScopeProxy(WebEmbeddedWorkerImpl&, Document&, WebServiceWorkerContextClient&);
 
     WebEmbeddedWorkerImpl& m_embeddedWorker;
     Document& m_document;
 
     WebServiceWorkerContextClient& m_client;
 
     ServiceWorkerGlobalScope* m_workerGlobalScope;
 };
 
 } // namespace blink
 
 #endif // ServiceWorkerGlobalScopeProxy_h