Issue 651090 link

Starred by 2 users

Issue metadata

Status:	Fixed
Owner:	rtoy@chromium.org
Closed:	Oct 2016
Components:	Blink>WebAudio
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

Add 'required' syntax for required dictionary elements.

Project Member Reported by rtoy@chromium.org, Sep 28 2016

Issue description

See https://github.com/WebAudio/web-audio-api/issues/981

Project Member

Comment 1 by bugdroid1@chromium.org, Oct 11 2016

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

commit ad65d1a7dc4dcbd4578752439edd441f7fa7e81e
Author: rtoy <rtoy@chromium.org>
Date: Tue Oct 11 17:53:18 2016

Mark required dictionary members as required

Update code to remove explicit tests for required members and update
tests accordingly.

BUG= 651090 
TEST=constructor/iirfilter.html

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

[modify] https://crrev.com/ad65d1a7dc4dcbd4578752439edd441f7fa7e81e/third_party/WebKit/LayoutTests/webaudio/constructor/iirfilter.html
[modify] https://crrev.com/ad65d1a7dc4dcbd4578752439edd441f7fa7e81e/third_party/WebKit/Source/modules/webaudio/IIRFilterOptions.idl
[modify] https://crrev.com/ad65d1a7dc4dcbd4578752439edd441f7fa7e81e/third_party/WebKit/Source/modules/webaudio/MediaElementAudioSourceOptions.idl

Comment 2 by rtoy@chromium.org, Oct 11 2016

Owner: rtoy@chromium.org
Status: Started (was: Available)

Comment 3 by rtoy@chromium.org, Oct 17 2016

Status: Fixed (was: Started)

►

Issue 651090 link

Issue metadata

Add 'required' syntax for required dictionary elements.

Issue description

Comment 1 by bugdroid1@chromium.org, Oct 11 2016

Comment 1 Deleted

Comment 2 by rtoy@chromium.org, Oct 11 2016

Comment 2 Deleted

Comment 3 by rtoy@chromium.org, Oct 17 2016

Comment 3 Deleted

Sign in to add a comment