Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(98)

Unified Diff: third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html

Issue 2768983002: Fix duplicate test names in WebAudio tests (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html b/third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html
index d84ce0f7c69d180feb96571abf477f6b88ce78d5..596c3f559ab39784c62f63192f3e137bad8b5633 100644
--- a/third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html
+++ b/third_party/WebKit/LayoutTests/webaudio/Analyser/handle-silent-inputs.html
@@ -19,14 +19,14 @@
label: 'connected',
description: 'Test handling of silent inputs'
}, function (task, should) {
- tester(should, false).then(task.done.bind(task));
+ tester(should, false, '0').then(task.done.bind(task));
});
audit.define({
label: 'auto-pull',
description: 'Test handling of silent inputs'
}, function(task, should) {
- tester(should, true).then(task.done.bind(task));
+ tester(should, true, '1').then(task.done.bind(task));
});
audit.define({
@@ -100,7 +100,7 @@
audit.run();
- function tester(should, isAutoPullTest) {
+ function tester(should, isAutoPullTest, prefix) {
// Connect an oscillator to an analyser for testing the time data of the
// analyser after the oscillator stops.
let context = new OfflineAudioContext(1, renderFrames, sampleRate);
@@ -131,7 +131,7 @@
.then(context.resume.bind(context));
return context.startRendering().then(buffer => {
- should(timeData, 'Analyser time data at time ' + dataTime)
+ should(timeData, prefix + ': Analyser time data at time ' + dataTime)
.beConstantValueOf(0);
});
}

Powered by Google App Engine
This is Rietveld 408576698