ServiceWorker: Introduce ServiceWorkerMessageEvent to replace MessageEvent (3/3).

Source/modules/serviceworkers/ServiceWorkerMessageEvent.h

+// 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.
+
+#ifndef ServiceWorkerMessageEvent_h
+#define ServiceWorkerMessageEvent_h
+
+#include "core/dom/DOMArrayBuffer.h"
+#include "core/dom/MessagePort.h"
+#include "modules/EventModules.h"
+#include "modules/ModulesExport.h"
+#include "modules/serviceworkers/ServiceWorkerMessageEventInit.h"
+
+namespace blink {
+
+class MODULES_EXPORT ServiceWorkerMessageEvent final : public Event {
+    DEFINE_WRAPPERTYPEINFO();
+public:
+    static PassRefPtrWillBeRawPtr<ServiceWorkerMessageEvent> create()
+    {
+        return adoptRefWillBeNoop(new ServiceWorkerMessageEvent);
+    }
+
+    static PassRefPtrWillBeRawPtr<ServiceWorkerMessageEvent> create(const AtomicString& type, const ServiceWorkerMessageEventInit& initializer)
+    {
+        return adoptRefWillBeNoop(new ServiceWorkerMessageEvent(type, initializer));
+    }
+
+    static PassRefPtrWillBeRawPtr<ServiceWorkerMessageEvent> create(PassOwnPtrWillBeRawPtr<MessagePortArray> ports, PassRefPtr<SerializedScriptValue> data, const ServiceWorkerOrMessagePort& source)
+    {
+        return adoptRefWillBeNoop(new ServiceWorkerMessageEvent(data, String(), String(), source, ports));
+    }
+
+    virtual ~ServiceWorkerMessageEvent();
+
+    const String& origin() const { return m_origin; }
+    const String& lastEventId() const { return m_lastEventId; }
+    MessagePortArray ports() const { return m_ports ? *m_ports : MessagePortArray(); }
+    void source(ServiceWorkerOrMessagePort& result) const { result = m_source; }
+
+    virtual const AtomicString& interfaceName() const override;
+
+    ScriptValue data() const { return m_data; }
+    SerializedScriptValue* serializedData() const { return m_serializedData.get(); }

[bashi, 2015/05/22 00:23:36]

Could you explain why do you need this?

The reason why CustomEvent has serializedDetail() is that CustomEvent has
initCustomEvent() which takes a SerializedScriptValue. We expose CustomEvent as
WebDOMCustomEvent and blink embedder (i.e. chrome's content layer) uses
initCustomEvent() to initialize it.

ServiceWorkerMessageEvent doesn't provide initXXX() method, and if we can remove
create() which takes SerializedScriptValue (see my comment on
ServiceWorkerContainer.cpp), I think we can remove m_serializedData.

+
+    DECLARE_VIRTUAL_TRACE();
+
+private:
+    ServiceWorkerMessageEvent();
+    ServiceWorkerMessageEvent(const AtomicString& type, const ServiceWorkerMessageEventInit& initializer);
+    ServiceWorkerMessageEvent(PassRefPtr<SerializedScriptValue> data, const String& origin, const String& lastEventId, const ServiceWorkerOrMessagePort& source, PassOwnPtrWillBeRawPtr<MessagePortArray> ports);
+
+    ScriptValue m_data;
+    RefPtr<SerializedScriptValue> m_serializedData;
+    String m_origin;
+    String m_lastEventId;
+    ServiceWorkerOrMessagePort m_source;
+    OwnPtrWillBeMember<MessagePortArray> m_ports;
+};
+
+} // namespace blink
+
+#endif // ServiceWorkerMessageEvent_h