MediaRecorder Blink part

public/platform/WebMediaRecorderHandler.h

Index: public/platform/WebMediaRecorderHandler.h
diff --git a/public/platform/WebMediaRecorderHandler.h b/public/platform/WebMediaRecorderHandler.h
new file mode 100644
index 0000000000000000000000000000000000000000..490b705becb9f532ee1f24fc9a31f23b11177871
--- /dev/null
+++ b/public/platform/WebMediaRecorderHandler.h
@@ -0,0 +1,40 @@
+// 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 WebMediaRecorderHandler_h
+#define WebMediaRecorderHandler_h
+
+#include "WebCommon.h"
+
+namespace blink {
+
+class WebMediaRecorderHandlerClient;
+class WebMediaStream;
+class WebString;
+
+// Platform interface of a MediaRecorder.
+class BLINK_PLATFORM_EXPORT WebMediaRecorderHandler {
+public:
+    virtual ~WebMediaRecorderHandler() { }

[mlamouri (slow - plz ping), 2015/08/13 13:31:48]

= default; instead of {} should do.

[mcasas, 2015/08/13 18:42:17]

Done.

+    virtual bool initialize(WebMediaRecorderHandlerClient* client,
+                            const WebMediaStream& stream,
+                            const WebString& mimeType) = 0;
+    virtual bool start() = 0;
+    virtual bool start(int timeslice) = 0;

[mlamouri (slow - plz ping), 2015/08/13 13:31:48]

I think the Chromium/Blink style guide do not recommend method override.

[mcasas, 2015/08/13 18:42:17]

Chromium is mildly against them [1] and in this case is 
needed to implement the W3C API...

[1]
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Over...

+    virtual void stop() = 0;
+    virtual void pause() = 0;
+    virtual void resume() = 0;
+
+    // MediaRecorder API canRecordMimeType() is a tristate in which the returned
+    // value 'probably' means that "the user agent is confident that mimeType
+    // represents a type that it can record" [1], but a number of reasons might
+    // prevent a firm answer at this stage, so a boolean is a better option,
+    // because "Implementors are encouraged to return "maybe" unless the type
+    // can be confidently established as being supported or not." [1].
+    // [1] http://w3c.github.io/mediacapture-record/MediaRecorder.html#methods
+    virtual bool canSupportMimeType(const WebString& mimeType) = 0;
+};
+
+} // namespace blink
+#endif

[mlamouri (slow - plz ping), 2015/08/13 13:31:48]

nit: empty line

[mcasas, 2015/08/13 18:42:17]

Done.