Issue 673965 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	mcasas@chromium.org
Closed:	Dec 2016
Cc:	emir...@chromium.org hta@chromium.org niklase@chromium.org mcasas@chromium.org
Components:	Blink>MediaRecording
EstimatedDays:	----
NextAction:	----
OS:	All
Pri:	2
Type:	Bug
M-57

MediaRecorder: only throw exception from requestData() if state is inactive

Project Member Reported by mcasas@chromium.org, Dec 14 2016

Issue description

... hence allowing for requestData() in pause state:

https://github.com/w3c/mediacapture-record/issues/100

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/29c40203a88f3e540ca1985d12dcd4de09a07317

commit 29c40203a88f3e540ca1985d12dcd4de09a07317
Author: mcasas <mcasas@chromium.org>
Date: Wed Dec 14 02:59:17 2016

MediaRecoder: only throw exception from requestData() if state is inactive

Simple change to update to the Spec and FF implementation.

LayoutTest updated (note that we're still checking that requestData()
throws when the state==inactive).

BUG= 673965 

Review-Url: https://codereview.chromium.org/2571093002
Cr-Commit-Position: refs/heads/master@{#438407}

[modify] https://crrev.com/29c40203a88f3e540ca1985d12dcd4de09a07317/third_party/WebKit/LayoutTests/fast/mediarecorder/MediaRecorder-events-and-exceptions.html
[modify] https://crrev.com/29c40203a88f3e540ca1985d12dcd4de09a07317/third_party/WebKit/Source/modules/mediarecorder/MediaRecorder.cpp