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

Unified Diff: chrome/test/data/extensions/api_test/file_browser/mount_test/test.js

Issue 160483004: Keep track of the list of currently mounted volumes in C++-side VolumeManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review fix Created 6 years, 10 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 | « chrome/browser/chromeos/file_manager/volume_manager_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/file_browser/mount_test/test.js
diff --git a/chrome/test/data/extensions/api_test/file_browser/mount_test/test.js b/chrome/test/data/extensions/api_test/file_browser/mount_test/test.js
index 620fb21b70f9535cd8b544bcfcca9006ee6862e2..b02f7e288d4cc159bba11f204b6b48043fdc05e0 100644
--- a/chrome/test/data/extensions/api_test/file_browser/mount_test/test.js
+++ b/chrome/test/data/extensions/api_test/file_browser/mount_test/test.js
@@ -70,11 +70,11 @@ var expectedArchiveVolume = {
// List of expected mount points.
// NOTE: this has to be synced with values in file_browser_private_apitest.cc
-// and values sorted by mountPath.
+// and values sorted by volumeId.
var expectedVolumeList = [
- expectedDriveVolume,
- expectedDownloadsVolume,
expectedArchiveVolume,
+ expectedDownloadsVolume,
+ expectedDriveVolume,
expectedVolume1,
expectedVolume2,
expectedVolume3,
@@ -126,7 +126,7 @@ chrome.test.runTests([
function getVolumeMetadataList() {
chrome.fileBrowserPrivate.getVolumeMetadataList(
chrome.test.callbackPass(function(result) {
- chrome.test.assertEq(result.length, expectedVolumeList.length,
+ chrome.test.assertEq(expectedVolumeList.length, result.length,
'getMountPoints returned wrong number of mount points.');
for (var i = 0; i < expectedVolumeList.length; i++) {
chrome.test.assertTrue(
« no previous file with comments | « chrome/browser/chromeos/file_manager/volume_manager_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698