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

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

Issue 904003002: Files.app: Start to use new metadata models in DirectoryContents. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/new_metadata_provider.js
diff --git a/ui/file_manager/file_manager/foreground/js/metadata/new_metadata_provider.js b/ui/file_manager/file_manager/foreground/js/metadata/new_metadata_provider.js
index ccfd2274f49dc6732aa0f50c3554a23d8c9d6345..1ece1292bd0d6758818b73d1d09bae5bc2b4a7cb 100644
--- a/ui/file_manager/file_manager/foreground/js/metadata/new_metadata_provider.js
+++ b/ui/file_manager/file_manager/foreground/js/metadata/new_metadata_provider.js
@@ -121,6 +121,10 @@ NewMetadataProvider.prototype.get = function(entries, names) {
* @return {!Array<!T>}
*/
NewMetadataProvider.prototype.getCache = function(entries, names) {
+ // Check if the property name is correct or not.
+ for (var i = 0; i < names.length; i++) {
+ assert(this.validPropertyNames_[names[i]]);
+ }
return this.cache_.get(entries, names);
};

Powered by Google App Engine
This is Rietveld 408576698