Files.app: Let MetadataCacheModel use cached URLs.

ui/file_manager/file_manager/common/js/util.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * Namespace for utility functions.
  */
 var util = {};
 
 /**
@@ skipping 749 matching lines @@
   return str('UI_LOCALE') || 'en';
 };
 
 /**
  * Converts array of entries to an array of corresponding URLs.
  * @param {Array<Entry>} entries Input array of entries.
  * @return {!Array<string>} Output array of URLs.
  */
 util.entriesToURLs = function(entries) {
   return entries.map(function(entry) {
-    return entry.toURL();
+    return entry['cachedUrl'] || entry.toURL();

[fukino, 2015/03/05 03:46:02]

Coudl you use entry.cachedUrl for consistency and for static analysis?

[hirono, 2015/03/05 03:47:12]

Closure compiler generates error for entry.cachedUrl, because normal Entry does
not have such property.

   });
 };
 
 /**
  * Converts array of URLs to an array of corresponding Entries.
  *
  * @param {Array.<string>} urls Input array of URLs.
  * @param {function(!Array.<!Entry>, !Array.<!URL>)=} opt_callback Completion
  *     callback with array of success Entries and failure URLs.
  * @return {Promise} Promise fulfilled with the object that has entries property
@@ skipping 249 matching lines @@
  * @param {!cr.EventTarget} target
  * @param {string} type
  * @param {Function} handler
  */
 util.addEventListenerToBackgroundComponent = function(target, type, handler) {
   target.addEventListener(type, handler);
   window.addEventListener('pagehide', function() {
     target.removeEventListener(type, handler);
   });
 };