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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-large-endtime.html

Issue 2658703002: Convert AudioParam Audit tests to testharness (Closed)
Patch Set: Rebase test Created 3 years, 11 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/AudioParam/audioparam-large-endtime.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-large-endtime.html b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-large-endtime.html
index f97527aa5e3df2db64f504765bc5117198a377da..0951d34b30c491c7754ab1d4083fd3d54f9a3772 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-large-endtime.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-large-endtime.html
@@ -1,7 +1,8 @@
<!doctype html>
<html>
<head>
- <script src="../../resources/js-test.js"></script>
+ <script src="../../resources/testharness.js"></script>
+ <script src="../../resources/testharnessreport.js"></script>
<script src="../resources/audit-util.js"></script>
<script src="../resources/audio-testing.js"></script>
<title>AudioParam with Huge End Time</title>
@@ -9,9 +10,6 @@
<body>
<script>
- description("Test AudioParam with Huge Time Value")
- window.jsTestIsAsync = true;
-
var sampleRate = 48000;
// Render for some small (but fairly arbitrary) time.
var renderDuration = 0.125;
@@ -30,7 +28,9 @@
graph.source.start();
graph.context.startRendering().then(function (buffer) {
- testPassed("linearRampToValue(0.1, " + largeTime + ") successfully rendered.");
+ Should("linearRampToValue(0.1, " + largeTime + ")", true)
+ .summarize("successfully rendered",
+ "unsuccessfully rendered");
}).then(done);
});
@@ -41,12 +41,13 @@
graph.source.start();
graph.context.startRendering().then(function (buffer) {
- testPassed("exponentialRampToValue(0.1, " + largeTime + ") successfully rendered.");
+ Should("exponentialRampToValue(0.1, " + largeTime + ")", true)
+ .summarize("successfully rendered",
+ "unsuccessfully rendered");
}).then(done);
});
audit.defineTask("finish", function (done) {
- finishJSTest();
done();
});

Powered by Google App Engine
This is Rietveld 408576698