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

Unified Diff: LayoutTests/webaudio/decode-audio-data-basic.html

Issue 1006963003: AudioContext.decodeAudioData returns a Promise (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix idl; small cleanups Created 5 years, 9 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: LayoutTests/webaudio/decode-audio-data-basic.html
diff --git a/LayoutTests/webaudio/decode-audio-data-basic.html b/LayoutTests/webaudio/decode-audio-data-basic.html
index d97e4c74bed44782c0fc096fc0bc643155358ac0..69764230d6d0a322bea352dab781a1d6d02816e0 100644
--- a/LayoutTests/webaudio/decode-audio-data-basic.html
+++ b/LayoutTests/webaudio/decode-audio-data-basic.html
@@ -18,12 +18,13 @@ window.jsTestIsAsync = true;
var context = new AudioContext();
-try {
- context.decodeAudioData(null, function(){}, function(){});
- testFailed("decodeAudioData should raise exception when arraybuffer parameter is null.");
-} catch(e) {
- testPassed("decodeAudioData raises exception correctly when arraybuffer parameter is null.");
-}
+context.decodeAudioData(null,
hongchan 2015/03/23 16:18:49 I think we should wrap this with try & catch to av
+ function () {
+ testFailed("decodeAudioData(null) did not invoke errorCallback.");
+ },
+ function () {
+ testPassed("decodeAudioData(null) invoked errorCallback correctly.");
+ });
var decodeCaseArray = [{url: "resources/media/24bit-44khz.wav", result: true},
{url: "resources/media/invalid-audio-file.txt", result: false}];

Powered by Google App Engine
This is Rietveld 408576698