Add thumbnailMetadataItem to GalleryItem.

ui/file_manager/gallery/js/gallery_item.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.
 
 /**
  * Object representing an image item (a photo).
  *
  * @param {!FileEntry} entry Image entry.
  * @param {!EntryLocation} locationInfo Entry location information.
- * @param {!Object} metadata Metadata for the entry.
  * @param {!MetadataItem} metadataItem
- * @param {!MetadataCache} metadataCache Metadata cache instance.
- * @param {!MetadataModel} metadataModel File system metadata.
+ * @param {!ThumbnailMetadataItem} thumbnailMetadataItem
  * @param {boolean} original Whether the entry is original or edited.
  * @constructor
  * @struct
  */
 Gallery.Item = function(
-    entry, locationInfo, metadata, metadataItem, metadataCache,
-    metadataModel, original) {
+    entry, locationInfo, metadataItem, thumbnailMetadataItem, original) {
   /**
-   * @type {!FileEntry}
-   * @private
+   * @private {!FileEntry}
    */
   this.entry_ = entry;
 
   /**
-   * @type {!EntryLocation}
-   * @private
+   * @private {!EntryLocation}
    */
   this.locationInfo_ = locationInfo;
 
   /**
-   * @type {!Object}
-   * @private
-   */
-  this.metadata_ = Object.preventExtensions(metadata);
-
-  /**
-   * @type {!MetadataItem}
+   * @private {!MetadataItem}
    */
   this.metadataItem_ = metadataItem;
 
   /**
-   * @type {!MetadataCache}
-   * @private
-   * @const
+   * @private {!ThumbnailMetadataItem}
    */
-  this.metadataCache_ = metadataCache;
-
-  /**
-   * @type {!MetadataModel}
-   * @private
-   * @const
-   */
-  this.metadataModel_ = metadataModel;
+  this.thumbnailMetadataItem_ = metadataItem;
 
   // TODO(yawano): Change this.contentImage and this.screenImage to private
   // fields and provide utility methods for them (e.g. revokeFullImageCache).
   /**
    * The content cache is used for prefetching the next image when going through
    * the images sequentially. The real life photos can be large (18Mpix = 72Mb
    * pixel array) so we want only the minimum amount of caching.
    * @type {(HTMLCanvasElement|HTMLImageElement)}
    */
   this.contentImage = null;
@@ skipping 27 matching lines @@
 Gallery.Item.prototype.getEntry = function() { return this.entry_; };
 
 /**
  * @return {!EntryLocation} Entry location information.
  */
 Gallery.Item.prototype.getLocationInfo = function() {
   return this.locationInfo_;
 };
 
 /**
- * @return {!Object} Metadata.
+ * @return {!MetadataItem} Metadata.
  */
-Gallery.Item.prototype.getMetadata = function() { return this.metadata_; };
+Gallery.Item.prototype.getMetadataItem = function() {
+  return this.metadataItem_;
+};
 
 /**
  * @return {!MetadataItem} Metadata.
  */
 Gallery.Item.prototype.getMetadataItem = function() {
   return this.metadataItem_;
 };
 
 /**
+ * @return {!ThumbnailMetadataItem} Thumbnail metadata item.
+ */
+Gallery.Item.prototype.getThumbnailMetadataItem = function() {
+  return this.thumbnailMetadataItem_;
+};
+
+/**
  * @return {string} File name.
  */
 Gallery.Item.prototype.getFileName = function() {
   return this.entry_.name;
 };
 
 /**
  * @return {boolean} True if this image has not been created in this session.
  */
 Gallery.Item.prototype.isOriginal = function() { return this.original_; };
@@ skipping 91 matching lines @@
         callback.bind(null, name + ext));
   }
 
   tryNext(10);
 };
 
 /**
  * Writes the new item content to either the existing or a new file.
  *
  * @param {!VolumeManager} volumeManager Volume manager instance.
+ * @param {!MetadataModel} metadataModel
  * @param {DirectoryEntry} fallbackDir Fallback directory in case the current
  *     directory is read only.
  * @param {boolean} overwrite Whether to overwrite the image to the item or not.
  * @param {!HTMLCanvasElement} canvas Source canvas.
  * @param {function(boolean)} callback Callback accepting true for success.
  */
 Gallery.Item.prototype.saveToFile = function(
-    volumeManager, fallbackDir, overwrite, canvas, callback) {
+    volumeManager, metadataModel, fallbackDir, overwrite, canvas, callback) {
   ImageUtil.metrics.startInterval(ImageUtil.getMetricName('SaveTime'));
 
   var name = this.getFileName();
 
   var onSuccess = function(entry) {
     var locationInfo = volumeManager.getLocationInfo(entry);
     if (!locationInfo) {
       // Reuse old location info if it fails to obtain location info.
       locationInfo = this.locationInfo_;
     }
     ImageUtil.metrics.recordEnum(ImageUtil.getMetricName('SaveResult'), 1, 2);
     ImageUtil.metrics.recordInterval(ImageUtil.getMetricName('SaveTime'));
 
     this.entry_ = entry;
     this.locationInfo_ = locationInfo;
 
     // Updates the metadata.
-    this.metadataCache_.clear([this.entry_], '*');
-    var oldMetadataPromise = new Promise(function(fulfill, reject) {
-      this.metadataCache_.getLatest(
-          [this.entry_],
-          Gallery.METADATA_TYPE,
-          function(metadataList) {
-            if (metadataList.length === 1) {
-              this.metadata_ = metadataList[0];
-              fulfill();
-            } else {
-              reject();
-            }
-          }.bind(this));
-    }.bind(this));
-    this.metadataModel_.notifyEntriesChanged([this.entry_]);
-    var newMetadataPromise = this.metadataModel_.get(
-        [entry], Gallery.PREFETCH_PROPERTY_NAMES).then(
-        function(metadataItems) {
-          this.metadataItem_ = metadataItems[0];
-        }.bind(this));
-    if (callback) {
-      Promise.all([oldMetadataPromise, newMetadataPromise]).then(
-          callback.bind(null, true),
-          callback.bind(null, false));
-    }
+    metadataModel.notifyEntriesChanged([this.entry_]);
+    Promise.all([
+      metadataModel.get([entry], Gallery.PREFETCH_PROPERTY_NAMES),
+      new ThumbnailModel(metadataModel).get([entry])
+    ]).then(function(metadataLists) {
+      this.metadataItem_ = metadataLists[0][0];
+      this.thumbnailMetadataItem_ = metadataLists[1][0];
+      callback(true);
+    }.bind(this), function() {
+      callback(false);
+    });
   }.bind(this);
 
   var onError = function(error) {
     console.error('Error saving from gallery', name, error);
     ImageUtil.metrics.recordEnum(ImageUtil.getMetricName('SaveResult'), 0, 2);
     if (callback)
       callback(false);
   };
 
   var doSave = function(newFile, fileEntry) {
-    var metadataPromise = this.metadataModel_.get(
+    var metadataPromise = metadataModel.get(
         [fileEntry],
         ['mediaMimeType', 'contentMimeType', 'ifd', 'exifLittleEndian']);
     metadataPromise.then(function(metadataItems) {
       fileEntry.createWriter(function(fileWriter) {
         var writeContent = function() {
           fileWriter.onwriteend = onSuccess.bind(null, fileEntry);
           var metadataItem = metadataItems[0];
           metadataItem.modificationTime = new Date();
           var metadataEncoder = ImageEncoder.encodeMetadata(
               metadataItem, canvas, /* quality for thumbnail*/ 0.8);
@@ skipping 81 matching lines @@
         return Promise.reject(str('GALLERY_FILE_EXISTS'));
       }, function() {
         return new Promise(
             this.entry_.moveTo.bind(this.entry_, parentDirectory, newFileName));
       }.bind(this));
     }.bind(this));
   }.bind(this)).then(function(entry) {
     this.entry_ = entry;
   }.bind(this));
 };