MediaRecorder Blink part

public/platform/WebMediaRecorderHandler.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 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() = default;
+    virtual bool initialize(WebMediaRecorderHandlerClient* client,
+                            const WebMediaStream& stream,
+                            const WebString& mimeType) = 0;
+    virtual bool start() = 0;

[esprehn, 2015/08/21 08:46:13]

I usually suggest not making these pure virtual and adding default methods
instead. It means adding a new method is simpler, otherwise you need add the
method here as a non-pure virtual, then add it in Chromium, then switch this
back to pure virtual. (3 patches)

[mcasas, 2015/08/21 18:50:05]

Done inlined, please instruct otherwise.

+    virtual bool start(int timeslice) = 0;
+    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