[fsp] Add requestUnmount() method together with the request manager.

chrome/test/data/extensions/api_test/file_system_provider/unmount/test.js

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+var firstFileSystemId;
+var secondFileSystemId;
+
+function setUp(callback) {
+  chrome.fileSystemProvider.mount('chocolate.zip', function(id) {
+    firstFileSystemId = id;
+    if (firstFileSystemId && secondFileSystemId)
+      callback();
+  }, function() {
+    chrome.test.fail();
+  });
+
+  chrome.fileSystemProvider.mount('banana.zip', function(id) {
+    secondFileSystemId = id;
+    if (firstFileSystemId && secondFileSystemId)
+      callback();
+  }, function() {
+    chrome.test.fail();
+  });
+}
+
+function runTests() {
+  chrome.test.runTests([
+    // Tests the fileBrowserPrivate.unmount(). Verifies if the unmount event

[satorux1, 2014/03/27 04:43:24]

Did you mean: fileSystemProvider.unmount

[mtomasz, 2014/03/27 04:55:30]

Done.

+    // is emitted by VolumeManager.
+    function unmount() {
+      var firstVolumeId =
+          'provided:' + chrome.runtime.id + '-' + firstFileSystemId + '-user';
+
+      var onMountCompleted = function(event) {
+        chrome.test.assertEq('unmount', event.eventType);
+        chrome.test.assertEq('success', event.status);
+        chrome.test.assertEq(firstVolumeId, event.volumeMetadata.volumeId);
+        chrome.fileBrowserPrivate.onMountCompleted.removeListener(
+            onMountCompleted);
+        chrome.test.succeed();
+      };
+
+      chrome.fileBrowserPrivate.onMountCompleted.addListener(
+          onMountCompleted);
+      chrome.fileSystemProvider.unmount(firstFileSystemId, function() {
+        // Wait for the unmount event.
+      }, function(error) {
+        chrome.test.fail();
+      });
+    },
+
+    // Tests the fileBrowserPrivate.unmount() with a wrong id. Verifies that

[satorux1, 2014/03/27 04:43:24]

ditto

[mtomasz, 2014/03/27 04:55:30]

Done.

+    // it fails with a correct error code.
+    function unmountWrongId() {
+      chrome.fileSystemProvider.unmount(1337, function() {
+        chrome.test.fail();
+      }, function(error) {
+        chrome.test.assertEq('SecurityError', error.name);
+        chrome.test.succeed();
+      });
+    },
+
+    // Tests if fileBrowserPrivate.removeMount() for provided file systems emits
+    // the onMountRequested() event with correct arguments.
+    function requestUnmountSuccess() {
+      var secondVolumeId =
+          'provided:' + chrome.runtime.id + '-' + secondFileSystemId + '-user';
+
+      var onUnmountRequested = function(fileSystemId, onSuccess, onError) {
+        chrome.test.assertEq(secondFileSystemId, fileSystemId);
+        onSuccess();
+        // Not calling fileSystemProvider.unmount(), so the onMountCompleted
+        // event will not be raised.
+        chrome.fileSystemProvider.onUnmountRequested.removeListener(
+            onUnmountRequested);
+        chrome.test.succeed();
+      };
+
+      chrome.fileSystemProvider.onUnmountRequested.addListener(
+          onUnmountRequested);
+      chrome.fileBrowserPrivate.removeMount(secondVolumeId);
+    },
+
+    // End to end test with a failure. Invokes fileSystemProvider.removeMount()
+    // on a provided file system, and verifies (1) if the onMountRequested()
+    // event is called with correct aguments, and (2) if calling onError(),
+    // results in an unmount event fired from the VolumeManager instance.

[satorux1, 2014/03/27 04:43:24]

I found these comments very useful. Could you add these to mount/test.js as
well?

[mtomasz, 2014/03/27 04:55:30]

Done.

+    function requestUnmountError() {
+      var secondVolumeId =
+          'provided:' + chrome.runtime.id + '-' + secondFileSystemId + '-user';
+      var unmountRequested = false;
+
+      var onUnmountRequested = function(fileSystemId, onSuccess, onError) {
+        chrome.test.assertEq(false, unmountRequested);
+        chrome.test.assertEq(secondFileSystemId, fileSystemId);
+        onError('IN_USE');  // enum ProviderError.
+        unmountRequested = true;
+        chrome.fileSystemProvider.onUnmountRequested.removeListener(
+            onUnmountRequested);
+      };
+
+      var onMountCompleted = function(event) {
+        chrome.test.assertEq('unmount', event.eventType);
+        chrome.test.assertEq('error_unknown', event.status);
+        chrome.test.assertEq(secondVolumeId, event.volumeMetadata.volumeId);
+        chrome.test.assertTrue(unmountRequested);
+
+        // Remove the handlers and mark the test as succeeded.
+        chrome.fileBrowserPrivate.removeMount(secondVolumeId);
+        chrome.fileBrowserPrivate.onMountCompleted.removeListener(
+            onMountCompleted);
+        chrome.test.succeed();
+      };
+
+      chrome.fileSystemProvider.onUnmountRequested.addListener(
+          onUnmountRequested);
+      chrome.fileBrowserPrivate.onMountCompleted.addListener(onMountCompleted);
+      chrome.fileBrowserPrivate.removeMount(secondVolumeId);
+    }
+  ]);
+};
+
+setUp(runTests);