Avoid UI unresponsiveness when switching between large images.

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 {MetadataItem} metadataItem
@@ skipping 422 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));
 };
+
+/**
+ * The threshold size of an image in pixels, which we always use thumbnail
+ * image for slide-in animation above this. This is a hack to avoid an UI
+ * unresponsiveness when switching between images.
+ *
+ * @type{number}
+ */
+GalleryItem.HEAVY_RENDERING_THRESHOLD_PIXELS = 4e3 * 3e3;

[fukino, 2017/01/13 02:50:33]

nit: prefer 4000 * 3000 for easier read.

[yamaguchi, 2017/01/13 12:25:39]

Done.

+
+/**
+ * Whether the image requires long rendering time.
+ *
+ * @return {boolean}
+ */
+GalleryItem.prototype.requireLongRenderingTime = function() {
+  var numPixels = this.metadataItem_.imageHeight *

[oka, 2017/01/13 04:23:19]

I think imageHeight can be undefined. Maybe it's better to early-return if
imageHeight/width is undefined for readability.

[yamaguchi, 2017/01/13 12:25:39]

Done.

+      this.metadataItem_.imageWidth;
+  return numPixels > GalleryItem.HEAVY_RENDERING_THRESHOLD_PIXELS;
+};