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

Unified Diff: ui/file_manager/file_manager/foreground/js/ui/location_line.js

Issue 651403002: Fix trivial type-check errors in file_manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and correct a comment. Created 6 years, 2 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/ui/location_line.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/location_line.js b/ui/file_manager/file_manager/foreground/js/ui/location_line.js
index 97a87150479df01de510c87d2bb66b80ebbca295..9d50a364bd45654ed58f667cb37957491068a7ea 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/location_line.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/location_line.js
@@ -59,7 +59,7 @@ LocationLine.prototype.show = function(entry) {
this.volumeIcon_.setAttribute(
'volume-type-icon', location.volumeInfo.volumeType);
this.volumeIcon_.setAttribute(
- 'volume-subtype', location.volumeInfo.deviceType);
+ 'volume-subtype', location.volumeInfo.deviceType || '');
}
var queue = new AsyncUtil.Queue();
@@ -144,7 +144,7 @@ LocationLine.prototype.show = function(entry) {
/**
* Updates the breadcrumb display.
- * @param {Array.<Entry>} entries Entries on the target path.
+ * @param {Array.<!Entry>} entries Entries on the target path.
* @private
*/
LocationLine.prototype.updateInternal_ = function(entries) {

Powered by Google App Engine
This is Rietveld 408576698