New issue
Advanced search Search tips

Issue 854220 link

Starred by 1 user
Status: Fixed
Owner:
rtoy@chromium.org
Closed: Jun 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove other webaudio IDL tests

Project Member Reported by rtoy@chromium.org, Jun 19 2018 
In some subdirectories of wpt/webaudio/the-audio-api there are IDL tests for some of the individual nodes (the-audiobuffer-interface for one).

But we have webaudio/idlharness.https.html that tests the entire IDL.

These subtests should probably be removed.
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 21 2018 

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

commit fb20841f876313b833080ef6add29e40a34d478f
Author: Raymond Toy <rtoy@chromium.org>
Date: Thu Jun 21 15:59:11 2018

Remove unneeded WebAudio IDL tests

The IDL tests in the subdirectories aren't really needed anymore since
we have webaudio/idlharness.https.html that covers everything in the
individual tests.

Bug:  854220 
Change-Id: I589f4972922dfc15c475ce20e7c4cb664b529920
Reviewed-on: https://chromium-review.googlesource.com/1107120
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569280}
[delete] https://crrev.com/308754d891da62cadc3ab326c55f0e4f6b14ea1c/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audiobuffer-interface/idl-test.html
[delete] https://crrev.com/308754d891da62cadc3ab326c55f0e4f6b14ea1c/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audiodestinationnode-interface/idl-test.html
[delete] https://crrev.com/308754d891da62cadc3ab326c55f0e4f6b14ea1c/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audioparam-interface/idl-test.html

Comment 2 by rtoy@chromium.org, Jun 22 2018

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

Sign in to add a comment