update expectation tests for Chrome v59

chrisguttandin · Jun 9, 2017 · cce7371 · cce7371
1 parent 24255b7
commit cce7371
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 30 deletions.
diff --git a/src/providers/audio-context-constructor.ts b/src/providers/audio-context-constructor.ts
@@ -487,7 +487,7 @@ export const AUDIO_CONTEXT_CONSTRUCTOR_PROVIDER = {
             public decodeAudioData (
                 audioData: ArrayBuffer, successCallback?: TDecodeSuccessCallback, errorCallback?: TDecodeErrorCallback
             ): Promise<AudioBuffer> {
-                // Bug #43: Only Chrome Canary does yet throw a DataCloneError.
+                // Bug #43: Only Chrome does yet throw a DataCloneError.
                 if (detachedAudioBuffers.has(audioData)) {
                     const err = dataCloneErrorFactory.create();
 

diff --git a/src/providers/offline-audio-context-constructor.ts b/src/providers/offline-audio-context-constructor.ts
@@ -184,7 +184,8 @@ export const OFFLINE_AUDIO_CONTEXT_CONSTRUCTOR_PROVIDER = {
 
             public decodeAudioData (
                 audioData: ArrayBuffer, successCallback?: TDecodeSuccessCallback, errorCallback?: TDecodeErrorCallback
-            ): Promise<AudioBuffer> {                // Bug #43: Only Chrome Canary does yet throw a DataCloneError.
+            ): Promise<AudioBuffer> {
+                // Bug #43: Only Chrome does yet throw a DataCloneError.
                 if (detachedAudioBuffers.has(audioData)) {
                     const err = dataCloneErrorFactory.create();
 

diff --git a/test/expectation/chrome/current/audio-context-constructor.js b/test/expectation/chrome/current/audio-context-constructor.js
@@ -2,7 +2,6 @@ import 'core-js/es7/reflect';
 import { UNPATCHED_AUDIO_CONTEXT_CONSTRUCTOR_PROVIDER, unpatchedAudioContextConstructor } from '../../../../src/providers/unpatched-audio-context-constructor';
 import { ReflectiveInjector } from '@angular/core';
 import { WINDOW_PROVIDER } from '../../../../src/providers/window';
-import { loadFixture } from '../../../helper/load-fixture';
 import { spy } from 'sinon';
 
 describe('audioContextConstructor', () => {
@@ -119,19 +118,6 @@ describe('audioContextConstructor', () => {
             }, 1000);
         });
 
-        // bug #43
-
-        it('should not throw a DataCloneError', (done) => {
-            loadFixture('1000-frames-of-noise.wav', (err, arrayBuffer) => {
-                expect(err).to.be.null;
-
-                audioContext
-                    .decodeAudioData(arrayBuffer)
-                    .then(() => audioContext.decodeAudioData(arrayBuffer))
-                    .then(() => done());
-            });
-        });
-
     });
 
 });
diff --git a/test/expectation/chrome/current/offline-audio-context-constructor.js b/test/expectation/chrome/current/offline-audio-context-constructor.js
@@ -2,7 +2,6 @@ import 'core-js/es7/reflect';
 import { UNPATCHED_OFFLINE_AUDIO_CONTEXT_CONSTRUCTOR_PROVIDER, unpatchedOfflineAudioContextConstructor } from '../../../../src/providers/unpatched-offline-audio-context-constructor';
 import { ReflectiveInjector } from '@angular/core';
 import { WINDOW_PROVIDER } from '../../../../src/providers/window';
-import { loadFixture } from '../../../helper/load-fixture';
 import { spy } from 'sinon';
 
 describe('offlineAudioContextConstructor', () => {
@@ -122,19 +121,6 @@ describe('offlineAudioContextConstructor', () => {
             }, 1000);
         });
 
-        // bug #43
-
-        it('should not throw a DataCloneError', (done) => {
-            loadFixture('1000-frames-of-noise.wav', (err, arrayBuffer) => {
-                expect(err).to.be.null;
-
-                offlineAudioContext
-                    .decodeAudioData(arrayBuffer)
-                    .then(() => offlineAudioContext.decodeAudioData(arrayBuffer))
-                    .then(() => done());
-            });
-        });
-
     });
 
 });