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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-method-chaining.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-method-chaining.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-method-chaining.html b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-method-chaining.html
index 0fac244e92869026dcb42d7b3efbb32357283613..f3195add1b822215e6d4084d23313c761ab48839 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-method-chaining.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-method-chaining.html
@@ -2,7 +2,8 @@
<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>
<script src="../resources/audioparam-testing.js"></script>
@@ -10,8 +11,6 @@
<body>
<script>
- description('Test method chaining feature of AudioParam automation methods.');
- window.jsTestIsAsync = true;
var sampleRate = 44100;
@@ -29,10 +28,9 @@
];
function verifyReturnedParam(config) {
- if (config.source === config.returned)
- testPassed('The return value of ' + config.desc + ' matches the source AudioParam.');
- else
- testFailed('The return value of ' + config.desc + ' does NOT match source AudioParam.');
+ Should('The return value of ' + config.desc + ' matches the source AudioParam',
+ config.source === config.returned)
+ .beEqualTo(true);
}
var audit = Audit.createTaskRunner();
@@ -131,7 +129,6 @@
});
audit.defineTask('finish', function (done) {
- finishJSTest();
done();
});

Powered by Google App Engine
This is Rietveld 408576698