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

Unified Diff: LayoutTests/webaudio/audiocontext-suspend-resume.html

Issue 1140723003: Implement suspend() and resume() for OfflineAudioContext (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Restructured the resolution of suspend promise Created 5 years, 6 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 | « no previous file | LayoutTests/webaudio/audiocontext-suspend-resume-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/webaudio/audiocontext-suspend-resume.html
diff --git a/LayoutTests/webaudio/audiocontext-suspend-resume.html b/LayoutTests/webaudio/audiocontext-suspend-resume.html
index 8c16380b9a6cc867460b063fafd397231b770ddd..c0b0fc21d6c66010297a67a7aac186437b7717a1 100644
--- a/LayoutTests/webaudio/audiocontext-suspend-resume.html
+++ b/LayoutTests/webaudio/audiocontext-suspend-resume.html
@@ -57,12 +57,12 @@
p1.then(
handlePromise(testFailed, "offlineContext.suspend() should have been rejected for an offline context"),
function (e) {
- if (e.name === "InvalidAccessError") {
+ if (e.name === "TypeError") {
testPassed(
"offlineContext.suspend() was correctly rejected: " + e);
} else {
testFailed(
- "offlineContext.suspend() was correctly rejected but expected InvalidAccessError, not: " + e);
+ "offlineContext.suspend() was correctly rejected but expected TypeError, not: " + e);
}
}
).then(done);
@@ -83,12 +83,12 @@
p2.then(
handlePromise(testFailed, "offlineContext.resume() should have been rejected for an offline context"),
function (e) {
- if (e.name === "InvalidAccessError") {
+ if (e.name === "InvalidStateError") {
testPassed(
"offlineContext.resume() was correctly rejected: " + e);
} else {
testFailed(
- "offlineContext.resume() was correctly rejected but expected InvalidAccessError, not: " + e);
+ "offlineContext.resume() was correctly rejected but expected InvalidStateError, not: " + e);
}
}
).then(done);
@@ -107,13 +107,13 @@
shouldBeEqualToString("offlineContext.state", "closed");
// suspend() should be rejected on a closed context.
- offlineContext.suspend().then(
+ offlineContext.suspend(1.0).then(
handlePromise(testFailed, "offlineContext.suspend() on a closed context not rejected"),
function (e) {
- if (e.name === "InvalidAccessError") {
+ if (e.name === "InvalidStateError") {
testPassed("offlineContext.suspend() on a closed context rejected: " + e);
} else {
- testFailed("offlineContext.suspend() on a closed context rejected but expected InvalidAccessError, not: " + e);
+ testFailed("offlineContext.suspend() on a closed context rejected but expected InvalidStateError, not: " + e);
}
}
).then(function () {
@@ -121,10 +121,10 @@
offlineContext.resume().then(
handlePromise(testFailed, "offlineContext.resume() on a closed context not rejected"),
function (e) {
- if (e.name === "InvalidAccessError") {
+ if (e.name === "InvalidStateError") {
testPassed("offlineContext.resume() on a closed context rejected: " + e);
} else {
- testFailed("offlineContext.resume() on a closed context rejected but expected InvalidAccessError, not: " + e);
+ testFailed("offlineContext.resume() on a closed context rejected but expected InvalidStateError, not: " + e);
}
}
).then(done);
« no previous file with comments | « no previous file | LayoutTests/webaudio/audiocontext-suspend-resume-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698