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

Unified Diff: chrome/test/data/extensions/api_test/media_gallery/read_permission/test.js

Issue 10704258: Add extension permissions for Media Gallery API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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_gallery/read_permission/test.js
diff --git a/chrome/test/data/extensions/api_test/media_gallery/test.js b/chrome/test/data/extensions/api_test/media_gallery/read_permission/test.js
similarity index 53%
copy from chrome/test/data/extensions/api_test/media_gallery/test.js
copy to chrome/test/data/extensions/api_test/media_gallery/read_permission/test.js
index 6302065f5d8abbbdfced24b1fb80aa76da91fd99..d97c1a4fa5432b36df5d8ed4fffc78ce2fa12023 100644
--- a/chrome/test/data/extensions/api_test/media_gallery/test.js
+++ b/chrome/test/data/extensions/api_test/media_gallery/read_permission/test.js
@@ -11,16 +11,11 @@ var mediaFileSystemsListCallback = function(results) {
expectedFileSystems = 0;
}
chrome.test.assertEq(expectedFileSystems, results.length);
-};
-var nullCallback = function(result) {
- chrome.test.assertEq(null, result);
+ // TODO(vandebo) Test that we can read from the FS.
};
function runTests(tests) {
- chrome.test.getConfig(function(config) {
- operatingSystem = config.osName;
- chrome.test.runTests(tests);
- });
+ chrome.test.runTests(tests);
}
chrome.test.runTests([
@@ -28,21 +23,4 @@ chrome.test.runTests([
mediaGalleries.getMediaFileSystems(
chrome.test.callbackPass(mediaFileSystemsListCallback));
},
-
- function openMediaGalleryManager() {
- // Just confirm that the function exists.
- chrome.test.assertTrue(mediaGalleries.openMediaGalleryManager !== null);
- chrome.test.succeed()
- },
-
- function extractEmbeddedThumbnails() {
- var result = mediaGalleries.extractEmbeddedThumbnails({});
- chrome.test.assertEq(null, result);
- chrome.test.succeed()
- },
-
- function assembleMediaFile() {
- mediaGalleries.assembleMediaFile(
- {}, {}, chrome.test.callbackPass(nullCallback));
- },
]);

Powered by Google App Engine
This is Rietveld 408576698