MediaRecorder: Adding BlobEvent and connecting it in MediaRecorderHandler

Source/modules/mediarecorder/BlobEvent.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 BlobEvent_h
+#define BlobEvent_h
+
+#include "core/fileapi/Blob.h"
+#include "modules/EventModules.h"
+#include "modules/ModulesExport.h"
+#include "wtf/text/AtomicString.h"
+
+namespace blink {
+
+class Blob;
+class BlobEventInit;
+
+class MODULES_EXPORT BlobEvent final : public Event {
+    DEFINE_WRAPPERTYPEINFO();
+public:
+    ~BlobEvent() override {}
+
+    static PassRefPtrWillBeRawPtr<BlobEvent> create();
+    static PassRefPtrWillBeRawPtr<BlobEvent> create(const AtomicString& type);

[Peter Beverloo, 2015/09/22 14:03:42]

You can remove this create() method and the associated constructor, they're
never called.

[mcasas, 2015/09/22 16:18:48]

Oops, you're right. Done.

+    static PassRefPtrWillBeRawPtr<BlobEvent> create(const AtomicString& type, const BlobEventInit& initializer);
+    static PassRefPtrWillBeRawPtr<BlobEvent> create(const AtomicString& type, bool canBubble, bool cancelable, Blob*);

[Peter Beverloo, 2015/09/22 14:03:42]

As this is only being called by MediaRecorder, you may want to remove the
canBubble and cancelable arguments and force them to |false| in the constructor.
MediaRecorder has no reason to be able to change that behaviour.

[mcasas, 2015/09/22 16:18:48]

Done.

+
+    Blob* data() const { return m_blob.get(); }
+
+    // Event
+    const AtomicString& interfaceName() const final;
+
+    DECLARE_VIRTUAL_TRACE();
+
+private:
+    BlobEvent() {}
+    BlobEvent(const AtomicString& type);
+    BlobEvent(const AtomicString& type, const BlobEventInit& initializer);
+    BlobEvent(const AtomicString& type, bool canBubble, bool cancelable, Blob*);
+
+    PersistentWillBeMember<Blob> m_blob;
+};
+
+} // namespace blink
+
+#endif // BlobEvent_h