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

Unified Diff: ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js

Issue 971653003: Files.app: Extract cache part from NewMetadataProvider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix audio player. Created 5 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
Index: ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js
diff --git a/ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js b/ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js
index 01c8519e871a196d65773d45c2a71ac60814e7ed..41b40f6f87177a16d732f4ba5668b193d06e1f99 100644
--- a/ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js
+++ b/ui/file_manager/file_manager/foreground/js/metadata/file_system_metadata_provider.js
@@ -5,14 +5,12 @@
/**
* Metadata provider for FileEntry#getMetadata.
*
- * @param {!MetadataProviderCache} cache
* @constructor
* @extends {NewMetadataProvider}
* @struct
*/
-function FileSystemMetadataProvider(cache) {
- NewMetadataProvider.call(
- this, cache, FileSystemMetadataProvider.PROPERTY_NAMES);
+function FileSystemMetadataProvider() {
+ NewMetadataProvider.call(this, FileSystemMetadataProvider.PROPERTY_NAMES);
}
/**
@@ -27,7 +25,9 @@ FileSystemMetadataProvider.prototype.__proto__ = NewMetadataProvider.prototype;
/**
* @override
*/
-FileSystemMetadataProvider.prototype.getImpl = function(requests) {
+FileSystemMetadataProvider.prototype.get = function(requests) {
+ if (!requests.length)
+ return Promise.resolve([]);
return Promise.all(requests.map(function(request) {
return Promise.all([
new Promise(function(fulfill, reject) {

Powered by Google App Engine
This is Rietveld 408576698