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 MODULES_EXPORT BlobEvent final : public Event {
+    DEFINE_WRAPPERTYPEINFO();
+public:
+    virtual ~BlobEvent() {}

[Peter Beverloo, 2015/09/18 13:07:26]

~BlobEvent() override {}

(The base class already defines it.)

[mcasas, 2015/09/21 15:24:15]

Done.

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

[Peter Beverloo, 2015/09/18 13:07:26]

nit: You could consider inlining the bodies of the static methods, since they
just defer to the constructors anyway. (This is quite common.)

[mcasas, 2015/09/21 15:24:15]

Yeah, I'd love to, but in previous CLs I was instructed to
not inline them (see e.g. MediaRecorderErrorEvent in this 
folder) by esprehn@. 

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

OK. I disagree, but don't feel strongly either so let's just go with it.

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