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

Unified Diff: LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js

Issue 17761003: Remove FileException (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix compile warning Created 7 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
Index: LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js
diff --git a/LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js b/LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js
index 3b30d22437dbaff9e890ca450e0728d17a4edea1..e2d7fa229e94d64b4e207100b198d9d7b64ccfe9 100644
--- a/LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js
+++ b/LayoutTests/http/tests/filesystem/workers/script-tests/resolve-url-sync.js
@@ -16,14 +16,16 @@ function createTestFile() {
return fileSystem.root.getFile(testFileName, {create:true});
}
-function assertEncodingErr(code) {
- errorCode = code;
- shouldBe("FileException.ENCODING_ERR", "errorCode");
+var error;
+
+function assertEncodingErr(e) {
+ error = e;
+ shouldBe("error.name", "'EncodingError'");
}
-function assertSecurityErr(code) {
- errorCode = code;
- shouldBe("FileException.SECURITY_ERR", "errorCode");
+function assertSecurityErr(e) {
+ error = e;
+ shouldBe("error.name", "'SecurityError'");
}
function assertPathsMatch(expected, actual) {
@@ -68,7 +70,7 @@ function runWrongDomain() {
webkitResolveLocalFileSystemSyncURL("filesystem:http://localhost:8000/temporary/foo");
testFailed();
} catch (e) {
- assertSecurityErr(e.code);
+ assertSecurityErr(e);
}
}
@@ -78,7 +80,7 @@ function runWrongPort() {
webkitResolveLocalFileSystemSyncURL("filesystem:http://127.0.0.1:8080/temporary/foo");
testFailed();
} catch (e) {
- assertSecurityErr(e.code);
+ assertSecurityErr(e);
}
}
@@ -88,7 +90,7 @@ function runWrongScheme() {
webkitResolveLocalFileSystemSyncURL("filesystem:https://127.0.0.1:8000/temporary/foo");
testFailed();
} catch (e) {
- assertSecurityErr(e.code);
+ assertSecurityErr(e);
}
}
@@ -98,7 +100,7 @@ function runBogusURL() {
webkitResolveLocalFileSystemSyncURL("foo");
testFailed();
} catch (e) {
- assertEncodingErr(e.code);
+ assertEncodingErr(e);
}
}
@@ -108,7 +110,7 @@ function runWrongProtocol() {
webkitResolveLocalFileSystemSyncURL("http://127.0.0.1:8000/foo/bar/baz");
testFailed();
} catch (e) {
- assertEncodingErr(e.code);
+ assertEncodingErr(e);
}
}
@@ -119,7 +121,7 @@ function runNotEnoughSlashes() {
webkitResolveLocalFileSystemSyncURL("filesystem:http://127.0.0.1:8000/temporarytestFile");
testFailed();
} catch (e) {
- assertEncodingErr(e.code);
+ assertEncodingErr(e);
}
}
@@ -129,7 +131,7 @@ function runNotEnoughSlashes2() {
try {
webkitResolveLocalFileSystemSyncURL("filesystem:http://127.0.0.1:8000temporary/testFile");
} catch (e) {
- assertSecurityErr(e.code);
+ assertSecurityErr(e);
}
}

Powered by Google App Engine
This is Rietveld 408576698