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

Unified Diff: ui/file_manager/file_manager/foreground/elements/files_metadata_box.js

Issue 2313763003: Show a local folder's size in QuickView. (Closed)
Patch Set: Created 4 years, 3 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/elements/files_metadata_box.js
diff --git a/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js b/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js
index 9040b37a80b49d94af598ef1971330001db74c06..9bc95b9b2c44ae403cde27744859e803fc10dd3a 100644
--- a/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js
+++ b/ui/file_manager/file_manager/foreground/elements/files_metadata_box.js
@@ -18,6 +18,9 @@ var FilesMetadataBox = Polymer({
imageHeight: Number,
mediaTitle: String,
mediaArtist: String,
+
+ // Whether the value is the middle of loading.
+ loading: Boolean,
fukino 2016/09/06 09:12:44 'loading' sounds like all the properties are loadi
harukam 2016/09/06 11:25:24 Agree. I like 'isSizeLoading' better.
},
// Clears fields.
@@ -32,6 +35,8 @@ var FilesMetadataBox = Polymer({
this.imageHeight = 0;
this.mediaTitle = '';
this.mediaArtist = '';
+
+ this.loading = false;
},
/**
@@ -86,4 +91,10 @@ var FilesMetadataBox = Polymer({
return '';
},
+ /**
+ * @param {boolean} loading
+ */
+ setLoading: function(loading) {
fukino 2016/09/06 09:12:44 You can simply call, for example, 'metadataBox_.lo
harukam 2016/09/06 11:25:24 Acknowledged.
+ this.loading = loading;
+ },
});

Powered by Google App Engine
This is Rietveld 408576698