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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/note-grain-on-play.html

Issue 2895963003: Apply layout-test-tidy to LayoutTests/webaudio (Closed)
Patch Set: Created 3 years, 7 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/AudioBufferSource/note-grain-on-play.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/note-grain-on-play.html b/third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/note-grain-on-play.html
index c9fb5bfaa8ef69af11946c4808b39a626126f198..d673db1bc161f6241f1d1e4aea1803107436cf4f 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/note-grain-on-play.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioBufferSource/note-grain-on-play.html
@@ -1,18 +1,19 @@
-<!doctype>
+<!DOCTYPE html>
<html>
<head>
+ <title>
+ note-grain-on-play.html
+ </title>
<script src="../../resources/testharness.js"></script>
<script src="../../resources/testharnessreport.js"></script>
<script src="../resources/audit-util.js"></script>
<script src="../resources/audit.js"></script>
<script src="../resources/note-grain-on-testing.js"></script>
</head>
-
<body>
<div id="description"></div>
<div id="console"></div>
-
- <script>
+ <script id="layout-test-code">
let audit = Audit.createTaskRunner();
// To test noteGrainOn, a single ramp signal is created.
@@ -20,7 +21,7 @@
// different times, and we verify that the actual output
// consists of the correct section of the ramp at the correct
// time.
-
+
let linearRampBuffer;
// Array of the grain offset used for each ramp played.
@@ -29,92 +30,91 @@
// Verify the received signal is a ramp from the correct section
// of our ramp signal.
function verifyGrain(renderedData, startFrame, endFrame, grainIndex) {
- let grainOffsetFrame = timeToSampleFrame(grainOffsetTime[
- grainIndex], sampleRate);
- let grainFrameLength = endFrame - startFrame;
- let ramp = linearRampBuffer.getChannelData(0);
- let isCorrect = true;
-
- let expected;
- let actual;
- let frame;
-
- for (let k = 0; k < grainFrameLength; ++k) {
- if (renderedData[startFrame + k] != ramp[grainOffsetFrame +
- k]) {
- expected = ramp[grainOffsetFrame + k];
- actual = renderedData[startFrame + k];
- frame = startFrame + k;
- isCorrect = false;
- break;
- }
+ let grainOffsetFrame =
+ timeToSampleFrame(grainOffsetTime[grainIndex], sampleRate);
+ let grainFrameLength = endFrame - startFrame;
+ let ramp = linearRampBuffer.getChannelData(0);
+ let isCorrect = true;
+
+ let expected;
+ let actual;
+ let frame;
+
+ for (let k = 0; k < grainFrameLength; ++k) {
+ if (renderedData[startFrame + k] != ramp[grainOffsetFrame + k]) {
+ expected = ramp[grainOffsetFrame + k];
+ actual = renderedData[startFrame + k];
+ frame = startFrame + k;
+ isCorrect = false;
+ break;
}
- return {
- verified: isCorrect,
- expected: expected,
- actual: actual,
- frame: frame
- };
+ }
+ return {
+ verified: isCorrect,
+ expected: expected,
+ actual: actual,
+ frame: frame
+ };
}
function checkResult(buffer, should) {
- renderedData = buffer.getChannelData(0);
- let nSamples = renderedData.length;
-
- // Number of grains that we found that have incorrect data.
- let invalidGrainDataCount = 0;
-
- let startEndFrames = findStartAndEndSamples(renderedData);
-
- // Verify the start and stop times. Not strictly needed for
- // this test, but it's useful to know that if the ramp data
- // appears to be incorrect.
- verifyStartAndEndFrames(startEndFrames, should);
-
- // Loop through each of the rendered grains and check that
- // each grain contains our expected ramp.
- for (let k = 0; k < startEndFrames.start.length; ++k) {
- // Verify that the rendered data matches the expected
- // section of our ramp signal.
- let result = verifyGrain(renderedData, startEndFrames.start[
- k], startEndFrames.end[k], k);
- should(result.verified, "Pulse " + k +
- " contained the expected data")
+ renderedData = buffer.getChannelData(0);
+ let nSamples = renderedData.length;
+
+ // Number of grains that we found that have incorrect data.
+ let invalidGrainDataCount = 0;
+
+ let startEndFrames = findStartAndEndSamples(renderedData);
+
+ // Verify the start and stop times. Not strictly needed for
+ // this test, but it's useful to know that if the ramp data
+ // appears to be incorrect.
+ verifyStartAndEndFrames(startEndFrames, should);
+
+ // Loop through each of the rendered grains and check that
+ // each grain contains our expected ramp.
+ for (let k = 0; k < startEndFrames.start.length; ++k) {
+ // Verify that the rendered data matches the expected
+ // section of our ramp signal.
+ let result = verifyGrain(
+ renderedData, startEndFrames.start[k], startEndFrames.end[k], k);
+ should(result.verified, 'Pulse ' + k + ' contained the expected data')
.beTrue();
- }
- should(invalidGrainDataCount,
- "Number of grains that did not contain the expected data")
+ }
+ should(
+ invalidGrainDataCount,
+ 'Number of grains that did not contain the expected data')
.beEqualTo(0);
}
- audit.define({
- label: "note-grain-on-play",
- description: "Test noteGrainOn offset rendering"
- }, function(task, should) {
- // Create offline audio context.
- context = new OfflineAudioContext(2, sampleRate *
- renderTime, sampleRate);
-
- // Create a linear ramp for testing noteGrainOn.
- linearRampBuffer = createSignalBuffer(context,
- function(k) {
- // Want the ramp to start
- // with 1, not 0.
- return k + 1;
- });
-
- let grainInfo = playAllGrains(context, linearRampBuffer,
- numberOfTests);
-
- grainOffsetTime = grainInfo.grainOffsetTimes;
-
- context.startRendering()
- .then(function(audioBuffer) {
- checkResult(audioBuffer, should);
- task.done();
- });
- });
-
+ audit.define(
+ {
+ label: 'note-grain-on-play',
+ description: 'Test noteGrainOn offset rendering'
+ },
+ function(task, should) {
+ // Create offline audio context.
+ context =
+ new OfflineAudioContext(2, sampleRate * renderTime, sampleRate);
+
+ // Create a linear ramp for testing noteGrainOn.
+ linearRampBuffer = createSignalBuffer(context, function(k) {
+ // Want the ramp to start
+ // with 1, not 0.
+ return k + 1;
+ });
+
+ let grainInfo =
+ playAllGrains(context, linearRampBuffer, numberOfTests);
+
+ grainOffsetTime = grainInfo.grainOffsetTimes;
+
+ context.startRendering().then(function(audioBuffer) {
+ checkResult(audioBuffer, should);
+ task.done();
+ });
+ });
+
audit.run();
</script>
</body>

Powered by Google App Engine
This is Rietveld 408576698