Gallery: Store image caches in Gallery items.

ui/file_manager/gallery/js/gallery.js

 // Copyright 2014 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.
 
 'use strict';
 
 /**
  * Called from the main frame when unloading.
  * @param {boolean=} opt_exiting True if the app is exiting.
  */
@@ skipping 116 matching lines @@
             // correctly.
             this.splice(this.indexOf(item) + 1, 0, anotherItem);
           }
 
           fulfill();
         }.bind(this));
   }.bind(this));
 };
 
 /**
+ * Evicts image caches in the items.
+ * @param {Gallery.Item} currentSelectedItem Current selected item.
+ */
+GalleryDataModel.prototype.evictCache = function(currentSelectedItem) {
+  // Updates the last accessed date.
+  if (currentSelectedItem)
+    currentSelectedItem.lastAccessed = Date.now();
+
+  // Sort the item by the last accessed date.
+  var sorted = this.slice().sort(function(a, b) {
+    return b.lastAccessed - a.lastAccessed;
+  });
+
+  // Evict caches.
+  var contentCacheCount = 2;

[mtomasz, 2014/07/28 05:13:01]

I think the code is too complicated. contentCacheCount becomes a negative number
in #155. What does it mean if count is negative? Can we simplify?

[hirono, 2014/07/28 06:50:32]

Done.

+  var screenCacheCount = 5;
+  for (var i = 0; i < sorted.length; i++) {
+    if (sorted[i].contentImage) {
+      if (contentCacheCount-- <= 0) {
+        if (sorted[i].contentImage.parentNode) {
+          console.error('The content image has a parent node.');
+        } else {
+          sorted[i].contentImage.width = 0;

[mtomasz, 2014/07/28 05:13:01]

Please add a comment, that this is needed to force GC.

[hirono, 2014/07/28 06:50:32]

Done.

+          sorted[i].contentImage.height = 0;
+          sorted[i].contentImage = null;
+        }
+      }
+    }
+    if (sorted[i].screenImage) {
+      if (screenCacheCount-- <= 0) {
+        if (sorted[i].screenImage.parentNode) {
+          console.error('The screen image has a parent node.');
+        } else {
+          sorted[i].screenImage.width = 0;
+          sorted[i].screenImage.height = 0;
+          sorted[i].screenImage = null;
+        }
+      }
+    }
+  }
+};
+
+/**
  * Gallery for viewing and editing image files.
  *
  * @param {!VolumeManager} volumeManager The VolumeManager instance of the
  *     system.
  * @constructor
  */
 function Gallery(volumeManager) {
   this.context_ = {
     appWindow: chrome.app.window.current(),
     onClose: function() { close(); },
@@ skipping 492 matching lines @@
   return items[0];
 };
 
 /**
   * Selection change event handler.
   * @private
   */
 Gallery.prototype.onSelection_ = function() {
   this.updateSelectionAndState_();
   this.updateShareMenu_();
+  this.dataModel_.evictCache(this.getSelectedItems()[0]);
 };
 
 /**
   * Data model splice event handler.
   * @private
   */
 Gallery.prototype.onSplice_ = function() {
   this.selectionModel_.adjustLength(this.dataModel_.length);
 };
 
@@ skipping 310 matching lines @@
   window.loadTimeData.data = backgroundComponents.stringData;
   gallery = new Gallery(backgroundComponents.volumeManager);
 };
 
 /**
  * Loads entries.
  */
 window.loadEntries = function(entries, selectedEntries) {
   gallery.load(entries, selectedEntries);
 };