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

Unified Diff: LayoutTests/fast/filesystem/entry-points-missing-arguments.html

Issue 24203002: Improve generated "Not enough arguments." TypeError exceptions. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: NeedsRebaseline Created 7 years, 3 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/fast/filesystem/entry-points-missing-arguments.html
diff --git a/LayoutTests/fast/filesystem/entry-points-missing-arguments.html b/LayoutTests/fast/filesystem/entry-points-missing-arguments.html
index 79c531fa6f3a52b1976ec764bf4dbcb1bfb94c7e..09e6341499e125730a5de99ec73c87a0a6729724 100644
--- a/LayoutTests/fast/filesystem/entry-points-missing-arguments.html
+++ b/LayoutTests/fast/filesystem/entry-points-missing-arguments.html
@@ -4,12 +4,11 @@
description('Ensures that we get exceptions when we do not include required arguments for webkitRequestFileSystem and webkitResolveLocalFileSystemURL methods.');
if (window.webkitResolveLocalFileSystemURL) {
- var notEnoughArguments = '"TypeError: Not enough arguments"';
- shouldThrow('webkitResolveLocalFileSystemURL()', notEnoughArguments);
- shouldThrow('webkitResolveLocalFileSystemURL("")', notEnoughArguments);
- shouldThrow('webkitRequestFileSystem()', notEnoughArguments);
- shouldThrow('webkitRequestFileSystem(TEMPORARY)', notEnoughArguments);
- shouldThrow('webkitRequestFileSystem(TEMPORARY, 1)', notEnoughArguments);
+ shouldThrow('webkitResolveLocalFileSystemURL()', '"TypeError: Failed to execute \'webkitResolveLocalFileSystemURL\' on \'DOMWindow\': 2 arguments required, but only 0 present."');
+ shouldThrow('webkitResolveLocalFileSystemURL("")', '"TypeError: Failed to execute \'webkitResolveLocalFileSystemURL\' on \'DOMWindow\': 2 arguments required, but only 1 present."');
+ shouldThrow('webkitRequestFileSystem()', '"TypeError: Failed to execute \'webkitRequestFileSystem\' on \'DOMWindow\': 3 arguments required, but only 0 present."');
+ shouldThrow('webkitRequestFileSystem(TEMPORARY)', '"TypeError: Failed to execute \'webkitRequestFileSystem\' on \'DOMWindow\': 3 arguments required, but only 1 present."');
+ shouldThrow('webkitRequestFileSystem(TEMPORARY, 1)', '"TypeError: Failed to execute \'webkitRequestFileSystem\' on \'DOMWindow\': 3 arguments required, but only 2 present."');
} else
debug("This test requires window.webkitResolveLocalFileSystemURL.");
</script>

Powered by Google App Engine
This is Rietveld 408576698