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

Unified Diff: chrome/test/data/extensions/api_test/file_system_provider/mount/test.js

Issue 194693002: [fsp] Add requestUnmount() method together with the request manager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed some too strict thread checks. Created 6 years, 8 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: chrome/test/data/extensions/api_test/file_system_provider/mount/test.js
diff --git a/chrome/test/data/extensions/api_test/file_system_provider/mount/test.js b/chrome/test/data/extensions/api_test/file_system_provider/mount/test.js
index b5566d5c1c35a85563aae8cbbdf1f095ebbc2e55..00addb80c0e04245cafe8ef59e0fbf9274f76ef5 100644
--- a/chrome/test/data/extensions/api_test/file_system_provider/mount/test.js
+++ b/chrome/test/data/extensions/api_test/file_system_provider/mount/test.js
@@ -3,6 +3,7 @@
// found in the LICENSE file.
chrome.test.runTests([
+ // Tests whether mounting succeeds, when a non-empty name is provided.
function goodDisplayName() {
chrome.fileSystemProvider.mount(
'test file system',
@@ -17,6 +18,7 @@ chrome.test.runTests([
);
},
+ // Verifies that mounting fails, when an empty string is provided as a name.
function emptyDisplayName() {
chrome.fileSystemProvider.mount(
'',
@@ -30,6 +32,9 @@ chrome.test.runTests([
);
},
+ // End to end test. Mounts a volume using fileSystemProvider.mount(), then
+ // checks if the mounted volume is added to VolumeManager, by querying
+ // fileBrowserPrivate.getVolumeMetadataList().
function successfulMount() {
chrome.fileSystemProvider.mount(
'caramel-candy.zip',
@@ -49,10 +54,11 @@ chrome.test.runTests([
});
},
+ // Checks is limit for mounted file systems per profile works correctly.
+ // Tries to create more than allowed number of file systems. All of the mount
+ // requests should succeed, except the last one which should fail with a
+ // security error.
function stressMountTest() {
- // Try to create more than allowed number of file systems. All of the mount
- // requests should succeed, except the last one which should fail with a
- // security error.
var ALREADY_MOUNTED_FILE_SYSTEMS = 2; // By previous tests.
var MAX_FILE_SYSTEMS = 16;
var index = 0;

Powered by Google App Engine
This is Rietveld 408576698