Implement receiving side of web notification inline replies

third_party/WebKit/Source/modules/notifications/NotificationData.cpp

 // Copyright 2015 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 "modules/notifications/NotificationData.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/SerializedScriptValue.h"
 #include "bindings/core/v8/SerializedScriptValueFactory.h"
 #include "core/dom/ExecutionContext.h"
@@ skipping 75 matching lines @@
 
     const size_t maxActions = Notification::maxActions();
     for (const NotificationAction& action : options.actions()) {
         if (actions.size() >= maxActions)
             break;
 
         WebNotificationAction webAction;
         webAction.action = action.action();
         webAction.title = action.title();
 
+        if (action.type() == "button")
+            webAction.type = WebNotificationAction::Button;
+        else if (action.type() == "text")
+            webAction.type = WebNotificationAction::Text;
+        else
+            NOTREACHED() << "Unknown action type: " << action.type();
+
+        if (action.hasPlaceholder() && webAction.type == WebNotificationAction::Button) {
+            exceptionState.throwTypeError("Notifications of type \"button\" cannot specify a placeholder.");

[Peter Beverloo, 2016/04/06 17:22:00]

test?

See NotificationDataTest.cpp

[Nina, 2016/04/07 10:59:09]

Ah, this was only tested on the layout test. Added cpp unittest.

+            return WebNotificationData();
+        }
+
+        webAction.placeholder = action.placeholder();
+
         if (action.hasIcon() && !action.icon().isEmpty())
             webAction.icon = completeURL(executionContext, action.icon());
 
         actions.append(webAction);
     }
 
     webData.actions = actions;
 
     return webData;
 }
 
 } // namespace blink