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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.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
« no previous file with comments | « third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessor-offlineaudiocontext.html ('k') | third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode-0-output-channels.html » ('j') | third_party/WebKit/LayoutTests/webaudio/tools/layout-test-tidy.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.html b/third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.html
index dbdec638448cb59ffc6f1d0bfb0fc903d8886303..55d6e9192040efff04c61bb6891daccf5d1ff5d5 100644
--- a/third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.html
+++ b/third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode.html
@@ -1,135 +1,139 @@
<!DOCTYPE html>
<html>
-<head>
-<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>
-</head>
-
-<body>
-<script>
-let audit = Audit.createTaskRunner();
-
-let sampleRate = 44100.0;
-let outputChannels = 6;
-let playbackTime = 0.0;
-
-// For the current implementation of ScriptProcessorNode, when it works with
-// OfflineAudioContext (which runs much faster than real-time) the
-// event.inputBuffer might be overwrite again before onaudioprocess ever get
-// chance to be called. We carefully arrange the renderLengthInFrames and
-// bufferSize to have exactly the same value to avoid this issue.
-let renderLengthInFrames = 512;
-let bufferSize = 512;
-
-let context;
-
-function createBuffer(context, length) {
- let audioBuffer = context.createBuffer(2, length, sampleRate);
- let n = audioBuffer.length;
- let dataL = audioBuffer.getChannelData(0);
- let dataR = audioBuffer.getChannelData(1);
-
- for (let i = 0; i < n; ++i) {
- dataL[i] = -1;
- dataR[i] = 1;
- }
-
- return audioBuffer;
-}
-
-function processAudioData(event, should) {
- playbackTime = event.playbackTime;
- let expectedTime = context.currentTime + (bufferSize / context.sampleRate);
- let allowedTimeGap = 0.0000001;
-
- // There may be a little time gap which is from different thread operation
- // between currentTime when main thread fires onaudioprocess() and currentTime
- // when read in JS since currentTime is continuously increasing on audio
- // thread.
-
- should(playbackTime, 'playbackTime').beCloseTo(expectedTime, {
- threshold: allowedTimeGap
- });
-
- buffer = event.outputBuffer;
- should(buffer.numberOfChannels, 'Number of output channels')
- .beEqualTo(outputChannels);
- should(buffer.length, 'Length of buffer').beEqualTo(bufferSize);
-
- buffer = event.inputBuffer;
- let bufferDataL = buffer.getChannelData(0);
- let bufferDataR = buffer.getChannelData(1);
-
- should(bufferDataL, 'Left channel').beConstantValueOf(-1);
- should(bufferDataR, 'Right channel').beConstantValueOf(1);
-}
-
-function doBufferSizeTest(size, should) {
- should(() => {
- context.createScriptProcessor(size, 1, 1);
- }, 'context.createScriptProcessor(' + size + ', 1, 1)').notThrow();
-}
-
-audit.define(
- {label: 'test', description: 'Basic ScriptProcessorNode properties'},
- (task, should) => {
- // Create offline audio context.
- context = new OfflineAudioContext(2, renderLengthInFrames, sampleRate);
-
- should(() => {
- context.createScriptProcessor(512, 0, 0);
- }, 'createScriptProcessor(512, 0, 0)').throw();
-
- should(() => {
- context.createScriptProcessor(512, 1, 0);
- }, 'context.createScriptProcessor(512, 1, 0)').notThrow();
-
- should(() => {
- context.createScriptProcessor(512, 2, 0);
- }, 'context.createScriptProcessor(512, 2, 0)').notThrow();
-
- should(() => {
- context.createScriptProcessor(512, 0, 1);
- }, 'context.createScriptProcessor(512, 0, 1)').notThrow();
-
- should(() => {
- context.createScriptProcessor(512, 0, 2);
- }, 'context.createScriptProcessor(512, 0, 2)').notThrow();
- should(() => {
- context.createScriptProcessor(511, 1, 1);
- }, 'context.createScriptProcessor(511, 1, 1)').throw();
-
- doBufferSizeTest(256, should);
- doBufferSizeTest(512, should);
- doBufferSizeTest(1024, should);
- doBufferSizeTest(2048, should);
- doBufferSizeTest(4096, should);
- doBufferSizeTest(8192, should);
- doBufferSizeTest(16384, should);
-
- let sourceBuffer = createBuffer(context, renderLengthInFrames);
-
- let bufferSource = context.createBufferSource();
- bufferSource.buffer = sourceBuffer;
-
- let jsnode = context.createScriptProcessor(bufferSize, 2, outputChannels);
-
- bufferSource.connect(jsnode);
- jsnode.connect(context.destination);
- jsnode.onaudioprocess = event => {
- processAudioData(event, should);
- };
-
- bufferSource.start(0);
-
- context.startRendering().then(() => task.done());
- ;
- });
-
-audit.run();
-</script>
-
-</body>
+ <head>
+ <title>
+ scriptprocessornode.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>
+ </head>
+ <body>
+ <script id="layout-test-code">
+ let audit = Audit.createTaskRunner();
+
+ let sampleRate = 44100.0;
+ let outputChannels = 6;
+ let playbackTime = 0.0;
+
+ // For the current implementation of ScriptProcessorNode, when it works
+ // with OfflineAudioContext (which runs much faster than real-time) the
+ // event.inputBuffer might be overwrite again before onaudioprocess ever
+ // get chance to be called. We carefully arrange the renderLengthInFrames
+ // and bufferSize to have exactly the same value to avoid this issue.
+ let renderLengthInFrames = 512;
+ let bufferSize = 512;
+
+ let context;
+
+ function createBuffer(context, length) {
+ let audioBuffer = context.createBuffer(2, length, sampleRate);
+ let n = audioBuffer.length;
+ let dataL = audioBuffer.getChannelData(0);
+ let dataR = audioBuffer.getChannelData(1);
+
+ for (let i = 0; i < n; ++i) {
+ dataL[i] = -1;
+ dataR[i] = 1;
+ }
+
+ return audioBuffer;
+ }
+
+ function processAudioData(event, should) {
+ playbackTime = event.playbackTime;
+ let expectedTime =
+ context.currentTime + (bufferSize / context.sampleRate);
+ let allowedTimeGap = 0.0000001;
+
+ // There may be a little time gap which is from different thread
+ // operation between currentTime when main thread fires onaudioprocess()
+ // and currentTime when read in JS since currentTime is continuously
+ // increasing on audio thread.
+
+ should(playbackTime, 'playbackTime').beCloseTo(expectedTime, {
+ threshold: allowedTimeGap
+ });
+
+ buffer = event.outputBuffer;
+ should(buffer.numberOfChannels, 'Number of output channels')
+ .beEqualTo(outputChannels);
+ should(buffer.length, 'Length of buffer').beEqualTo(bufferSize);
+
+ buffer = event.inputBuffer;
+ let bufferDataL = buffer.getChannelData(0);
+ let bufferDataR = buffer.getChannelData(1);
+
+ should(bufferDataL, 'Left channel').beConstantValueOf(-1);
+ should(bufferDataR, 'Right channel').beConstantValueOf(1);
+ }
+
+ function doBufferSizeTest(size, should) {
+ should(() => {
+ context.createScriptProcessor(size, 1, 1);
+ }, 'context.createScriptProcessor(' + size + ', 1, 1)').notThrow();
+ }
+
+ audit.define(
+ {label: 'test', description: 'Basic ScriptProcessorNode properties'},
+ (task, should) => {
+ // Create offline audio context.
+ context =
+ new OfflineAudioContext(2, renderLengthInFrames, sampleRate);
+
+ should(() => {
+ context.createScriptProcessor(512, 0, 0);
+ }, 'createScriptProcessor(512, 0, 0)').throw();
+
+ should(() => {
+ context.createScriptProcessor(512, 1, 0);
+ }, 'context.createScriptProcessor(512, 1, 0)').notThrow();
+
+ should(() => {
+ context.createScriptProcessor(512, 2, 0);
+ }, 'context.createScriptProcessor(512, 2, 0)').notThrow();
+
+ should(() => {
+ context.createScriptProcessor(512, 0, 1);
+ }, 'context.createScriptProcessor(512, 0, 1)').notThrow();
+
+ should(() => {
+ context.createScriptProcessor(512, 0, 2);
+ }, 'context.createScriptProcessor(512, 0, 2)').notThrow();
+ should(() => {
+ context.createScriptProcessor(511, 1, 1);
+ }, 'context.createScriptProcessor(511, 1, 1)').throw();
+
+ doBufferSizeTest(256, should);
+ doBufferSizeTest(512, should);
+ doBufferSizeTest(1024, should);
+ doBufferSizeTest(2048, should);
+ doBufferSizeTest(4096, should);
+ doBufferSizeTest(8192, should);
+ doBufferSizeTest(16384, should);
+
+ let sourceBuffer = createBuffer(context, renderLengthInFrames);
+
+ let bufferSource = context.createBufferSource();
+ bufferSource.buffer = sourceBuffer;
+
+ let jsnode =
+ context.createScriptProcessor(bufferSize, 2, outputChannels);
+
+ bufferSource.connect(jsnode);
+ jsnode.connect(context.destination);
+ jsnode.onaudioprocess = event => {
+ processAudioData(event, should);
+ };
+
+ bufferSource.start(0);
+
+ context.startRendering().then(() => task.done());
+ ;
+ });
+
+ audit.run();
+ </script>
+ </body>
</html>
« no previous file with comments | « third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessor-offlineaudiocontext.html ('k') | third_party/WebKit/LayoutTests/webaudio/ScriptProcessor/scriptprocessornode-0-output-channels.html » ('j') | third_party/WebKit/LayoutTests/webaudio/tools/layout-test-tidy.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698