New issue
Advanced search Search tips

Issue 782901 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove unused or stale items from webaudio wpt tests

Project Member Reported by rtoy@chromium.org, Nov 8 2017

Issue description

Remove js/lodash.js because not used anywhere even though it's loaded
a in several files. (And it's 134K!)

Remove the stale copy of the spec.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 9 2017

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

commit ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad
Author: Raymond Toy <rtoy@chromium.org>
Date: Thu Nov 09 20:05:46 2017

Remove old or unused items from wpt/webaudio

Remove the old stale copy of the spec and remove js/lodash.js which
is loaded in several places but not actually used for anything.

Bug:  782901 
Change-Id: Idfdbc9f1cf2d9edf190d649168074f4d4ec66c73
Reviewed-on: https://chromium-review.googlesource.com/759225
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515256}
[delete] https://crrev.com/71144b8fcd63ba305c7114e272181acf97e88323/third_party/WebKit/LayoutTests/external/wpt/webaudio/js/lodash.js
[delete] https://crrev.com/71144b8fcd63ba305c7114e272181acf97e88323/third_party/WebKit/LayoutTests/external/wpt/webaudio/specification.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audiobuffer-interface/idl-test.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audiodestinationnode-interface/idl-test.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-audioparam-interface/idl-test.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-delaynode-interface/idl-test.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-gainnode-interface/idl-test.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-gainnode-interface/test-gainnode.html
[modify] https://crrev.com/ad4a09f2fa8cd5f19712a783f5f4d243b298e5ad/third_party/WebKit/LayoutTests/external/wpt/webaudio/the-audio-api/the-mediaelementaudiosourcenode-interface/mediaElementAudioSourceToScriptProcessorTest.html

Comment 2 by rtoy@chromium.org, Nov 10 2017

Status: Fixed (was: Started)

Sign in to add a comment