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

Unified Diff: chrome/test/data/extensions/api_test/media_galleries/read_access/test.js

Issue 224963010: Media Galleries: Add a dropPermissionForMediaFileSystem() API. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: renamed, fixed JS metadata 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/media_galleries/read_access/test.js
===================================================================
--- chrome/test/data/extensions/api_test/media_galleries/read_access/test.js (revision 261942)
+++ chrome/test/data/extensions/api_test/media_galleries/read_access/test.js (working copy)
@@ -64,7 +64,7 @@
TestFirstFilesystem(verifyFilesystem);
}
-function GetMediaFileSystemMetadata() {
+function GetMediaFileSystemMetadataTest() {
function verifyFilesystem(filesystem) {
var metadata = mediaGalleries.getMediaFileSystemMetadata(filesystem);
checkMetadata(metadata);
@@ -74,7 +74,7 @@
TestFirstFilesystem(verifyFilesystem);
}
-function GetAllMediaFileSystemMetadata() {
+function GetAllMediaFileSystemMetadataTest() {
function verifyMetadataList(metadataList) {
chrome.test.assertEq(1, metadataList.length)
checkMetadata(metadataList[0]);
@@ -84,6 +84,46 @@
mediaGalleries.getAllMediaFileSystemMetadata(verifyMetadataList);
}
+function DropPermissionForMediaFileSystemTest() {
+ var droppedFilesystem;
+ var droppedGalleryId;
+
+ function callDropPermission(filesystem) {
+ var metadata = mediaGalleries.getMediaFileSystemMetadata(filesystem);
+ droppedFilesystem = filesystem;
+ droppedGalleryId = metadata.galleryId;
+ mediaGalleries.dropPermissionForMediaFileSystem(
+ droppedGalleryId, verifyDropPermissionSucceeded);
+ }
+
+ function verifyDropPermissionSucceeded(dropped) {
+ chrome.test.assertTrue(dropped);
+ var metadata = mediaGalleries.getMediaFileSystemMetadata(droppedFilesystem);
+ var notFoundMetadata = {
+ "name": "",
+ "galleryId": "",
+ "isRemovable": false,
+ "isMediaDevice": false,
+ "isAvailable": false,
+ }
+ chrome.test.assertEq(notFoundMetadata, metadata);
+ mediaGalleries.getMediaFileSystems(verifyNoFileSystemAccess);
+ }
+
+ function verifyNoFileSystemAccess(results) {
+ chrome.test.assertEq(0, results.length);
+ mediaGalleries.dropPermissionForMediaFileSystem(
+ droppedGalleryId, verifyDropPermissionFailed);
+ }
+
+ function verifyDropPermissionFailed(dropped) {
+ chrome.test.assertFalse(dropped);
+ chrome.test.succeed();
+ }
+
+ TestFirstFilesystem(callDropPermission);
+}
+
CreateDummyWindowToPreventSleep();
chrome.test.getConfig(function(config) {
@@ -94,7 +134,8 @@
ReadDirectoryTest,
ReadFileToBytesTest,
GetMetadataTest,
- GetMediaFileSystemMetadata,
- GetAllMediaFileSystemMetadata,
+ GetMediaFileSystemMetadataTest,
+ GetAllMediaFileSystemMetadataTest,
+ DropPermissionForMediaFileSystemTest,
]);
})

Powered by Google App Engine
This is Rietveld 408576698