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

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

Issue 985533004: Implement chrome.fileSystem.requestFileSystem(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed kiosk mode. Created 5 years, 9 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_browser/content_checksum_test/test.js
diff --git a/chrome/test/data/extensions/api_test/file_browser/content_checksum_test/test.js b/chrome/test/data/extensions/api_test/file_browser/content_checksum_test/test.js
index 3e3c34e025aaf9f3c56c6f5cd4f0ded71d7be343..de11f55989447b9e988c70b0b07398c047784fa6 100644
--- a/chrome/test/data/extensions/api_test/file_browser/content_checksum_test/test.js
+++ b/chrome/test/data/extensions/api_test/file_browser/content_checksum_test/test.js
@@ -20,8 +20,8 @@ function getTestFilesystem() {
return volume.volumeType === 'testing';
})[0];
- chrome.fileManagerPrivate.requestFileSystem(
- testVolume.volumeId,
+ chrome.fileSystem.requestFileSystem(
+ {volumeId: testVolume.volumeId},
function(fileSystem) {
if (!fileSystem) {
reject(new Error('Failed to acquire the testing volume.'));
@@ -38,17 +38,39 @@ getTestFilesystem().then(
chrome.test.runTests([
// Checks the checksum code using a golden file.
function testGoldenChecksum() {
- fileSystem.root.getFile(
- 'test_dir/test_file.txt',
- {create: false},
- function(entry) {
- chrome.fileManagerPrivate.computeChecksum(
- entry.toURL(),
- chrome.test.callbackPass(function(result) {
- chrome.test.assertEq(kExpectedHash, result);
- }));
- },
- chrome.test.fail);
+ new Promise(
+ function(fulfill, reject) {
+ fileSystem.root.getFile(
+ 'test_dir/test_file.txt',
+ {create: false},
+ function(entry) {
+ chrome.test.assertTrue(!!entry);
+ fulfill(entry);
+ });
+ })
+ // TODO(mtomasz): Remove this step once computeChecksum works
+ // on isolated entries.
+ .then(
+ function(entry) {
+ return new Promise(function(fulfill, reject) {
+ chrome.fileManagerPrivate.resolveIsolatedEntries(
+ [entry],
+ function(externalEntries) {
+ chrome.test.assertTrue(!!externalEntries);
+ chrome.test.assertTrue(!!externalEntries[0]);
+ fulfill(externalEntries[0]);
+ });
+ });
+ })
+ .then(
+ function(externalEntry) {
+ chrome.fileManagerPrivate.computeChecksum(
+ externalEntry.toURL(),
+ chrome.test.callbackPass(function(result) {
+ chrome.test.assertEq(kExpectedHash, result);
+ }));
+ })
+ .catch(chrome.test.fail);
}
]);
});

Powered by Google App Engine
This is Rietveld 408576698