Gallery: Takes into account target canvas size when calculating view port.

ui/file_manager/gallery/js/image_editor/viewport.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.
 
 /**
  * Viewport class controls the way the image is displayed (scale, offset etc).
  * @constructor
  * @struct
  */
 function Viewport() {
@@ skipping 175 matching lines @@
 /**
  * Obtains the rotation value.
  * @return {number} Current rotation value.
  */
 Viewport.prototype.getRotation = function() {
   return this.rotation_;
 };
 
 /**
  * Obtains the scale for the specified image size.
  *

[mtomasz, 2015/05/14 00:32:32]

ni: Please update jsdoc.

[hirono, 2015/05/14 01:57:21]

Done.

  * @param {number} width Width of the full resolution image.
  * @param {number} height Height of the full resolution image.
  * @return {number} The ratio of the full resotion image size and the calculated
  * displayed image size.
  * @private
  */
-Viewport.prototype.getFittingScaleForImageSize_ = function(width, height) {
-  var scaleX = this.screenBounds_.width / width;
-  var scaleY = this.screenBounds_.height / height;
-  return Math.min(scaleX, scaleY, 1);
+Viewport.prototype.getFittingScaleForImageSize_ = function(
+    width, height, maxWidth, maxHeight) {
+  return Math.min(
+      maxWidth / width,
+      maxHeight / height,
+      this.screenBounds_.width / width,
+      this.screenBounds_.height / height);
 };
 
 /**
  * Returns offset X.
  * @return {number} X-offset of the viewport.
  */
 Viewport.prototype.getOffsetX = function() { return this.offsetX_; };
 
 /**
  * Returns offset Y.
@@ skipping 53 matching lines @@
   assert(this.imageBoundsOnScreen_);
   return this.imageBoundsOnScreen_;
 };
 
 /**
  * The image bounds in screen coordinates.
  * This returns the bounds of element before applying zoom and offset.
  * @return {!ImageRect}
  */
 Viewport.prototype.getImageElementBoundsOnScreen = function() {
   assert(this.imageElementBoundsOnScreen_);

[mtomasz, 2015/05/13 10:27:10]

nit: This seems to be used only in this.getDeviceBounds. Shall we make private,
or even remove?

[hirono, 2015/05/14 01:57:21]

Done.

   return this.imageElementBoundsOnScreen_;
 };
 
 /**
  * The image bounds on screen, which is clipped with the screen size.
  * @return {!ImageRect}
  */
 Viewport.prototype.getImageBoundsOnScreenClipped = function() {
   assert(this.imageBoundsOnScreenClipped_);
   return this.imageBoundsOnScreenClipped_;
@@ skipping 108 matching lines @@
   this.update_();
 };
 
 /**
  * Recalculate the viewport parameters.
  * @private
  */
 Viewport.prototype.update_ = function() {
   // Update scale.
   this.scale_ = this.getFittingScaleForImageSize_(
+      this.imageBounds_.width, this.imageBounds_.height,
       this.imageBounds_.width, this.imageBounds_.height);
 
   // Limit offset values.
   var zoomedWidht;
   var zoomedHeight;
   if (this.rotation_ % 2 == 0) {
     zoomedWidht = ~~(this.imageBounds_.width * this.scale_ * this.zoom_);
     zoomedHeight = ~~(this.imageBounds_.height * this.scale_ * this.zoom_);
   } else {
     var scale = this.getFittingScaleForImageSize_(
+        this.imageBounds_.height, this.imageBounds_.width,
         this.imageBounds_.height, this.imageBounds_.width);
     zoomedWidht = ~~(this.imageBounds_.height * scale * this.zoom_);
     zoomedHeight = ~~(this.imageBounds_.width * scale * this.zoom_);
   }
   var dx = Math.max(zoomedWidht - this.screenBounds_.width, 0) / 2;
   var dy = Math.max(zoomedHeight - this.screenBounds_.height, 0) / 2;
   this.offsetX_ = ImageUtil.clamp(-dx, this.offsetX_, dx);
   this.offsetY_ = ImageUtil.clamp(-dy, this.offsetY_, dy);
 
   // Image bounds on screen.
@@ skipping 35 matching lines @@
   viewport.scale_ = this.scale_;
   viewport.zoom_ = this.zoom_;
   viewport.offsetX_ = this.offsetX_;
   viewport.offsetY_ = this.offsetY_;
   viewport.rotation_ = this.rotation_;
   viewport.generation_ = this.generation_;
   viewport.update_();
   return viewport;
 };
 
+Viewport.prototype.getScreenRectTransformation = function(

[mtomasz, 2015/05/14 00:32:32]

nit: jsdoc missing

[hirono, 2015/05/14 01:57:21]

Done.

+    width, height, screenRect) {
+  var dx = screenRect.left + (screenRect.width - width) / 2;
+  var dy = screenRect.top + (screenRect.height - height) / 2;
+
+  return [

[mtomasz, 2015/05/14 00:32:32]

optional: I think simply concatenating is easier to read, but it's up to you.

[hirono, 2015/05/14 01:57:21]

I tries to create stringFormat helper function. Done.

+    'translate(', dx, 'px,', dy, 'px) ',
+    'scale(', screenRect.width / width, ',', screenRect.height / height, ') '
+  ].join('');
+};
+
+Viewport.prototype.getCroppingTransformation = function(

[mtomasz, 2015/05/14 00:32:32]

nit: jsdoc missing

[hirono, 2015/05/14 01:57:21]

Done.

+    width,
+    height,
+    wholeWidthMax,
+    wholeHeightMax,
+    cropRect) {
+  var fittingScale = this.getFittingScaleForImageSize_(
+      wholeWidthMax, wholeHeightMax, wholeWidthMax, wholeHeightMax);
+  var wholeWidth = wholeWidthMax * fittingScale;
+  var wholeHeight = wholeHeightMax * fittingScale;
+  var wholeLeft = (this.screenBounds_.width - wholeWidth) / 2;
+  var wholeTop = (this.screenBounds_.height - wholeHeight) / 2;
+  return this.getScreenRectTransformation(
+      width,
+      height,
+      new ImageRect(
+          wholeLeft + cropRect.left * fittingScale,
+          wholeTop + cropRect.top * fittingScale,
+          cropRect.width * fittingScale,
+          cropRect.height * fittingScale));
+};
+
 /**
  * Obtains CSS transformation for the screen image.
+ * @param {number} width Width of image.
+ * @param {number} height Height of image.
+ * @param {number=} opt_dx Amount of horizontal shift.
  * @return {string} Transformation description.
  */
-Viewport.prototype.getTransformation = function() {
-  var rotationScaleAdjustment;
-  if (this.rotation_ % 2) {
-    rotationScaleAdjustment = this.getFittingScaleForImageSize_(
-        this.imageBounds_.height, this.imageBounds_.width) / this.scale_;
-  } else {
-    rotationScaleAdjustment = 1;
-  }
-  return [
-    'translate(' + this.offsetX_ + 'px, ' + this.offsetY_ + 'px) ',
-    'rotate(' + (this.rotation_ * 90) + 'deg)',
-    'scale(' + (this.zoom_ * rotationScaleAdjustment) + ')'
-  ].join(' ');
+Viewport.prototype.getTransformation = function(width, height, opt_dx) {
+  return this.getTransformationInternal_(
+      width,
+      height,
+      this.rotation_,
+      this.zoom_,
+      this.offsetX_ + (opt_dx || 0),
+      this.offsetY_,
+      null);
 };
 
 /**
- * Obtains shift CSS transformation for the screen image.
- * @param {number} dx Amount of shift.
- * @return {string} Transformation description.
- */
-Viewport.prototype.getShiftTransformation = function(dx) {
-  return 'translateX(' + dx + 'px) ' + this.getTransformation();
-};
-
-/**
  * Obtains CSS transformation that makes the rotated image fit the original
  * image. The new rotated image that the transformation is applied to looks the
  * same with original image.
  *
- * @param {boolean} orientation Orientation of the rotation from the original
+ * @param {number} rotation90 Orientation of the rotation from the original

[mtomasz, 2015/05/14 00:32:32]

nit: Why the name is rotation90? It's hard for me to understand the meaning.

[hirono, 2015/05/14 01:57:21]

It was number of times for 90 degrees. e.g. 180 degree = 2 (x 90 degree). I
renamed it with rotation.

  *     image to the rotated image. True is for clockwise and false is for

[mtomasz, 2015/05/14 00:32:32]

nit: Please update the comment, it's not bool anymore.

[hirono, 2015/05/14 01:57:21]

Done.

  *     counterclockwise.
  * @return {string} Transformation description.
  */
-Viewport.prototype.getInverseTransformForRotatedImage = function(orientation) {
-  var previousImageWidth = this.imageBounds_.height;
-  var previousImageHeight = this.imageBounds_.width;
-  var oldScale = this.getFittingScaleForImageSize_(
-      previousImageWidth, previousImageHeight);
-  var scaleRatio = oldScale / this.scale_;
-  var degree = orientation ? '-90deg' : '90deg';
-  return [
-    'scale(' + scaleRatio + ')',
-    'rotate(' + degree + ')',
-    this.getTransformation()
-  ].join(' ');
+Viewport.prototype.getRotatingTransformation = function(
+    width, height, rotation90) {
+  return this.getTransformationInternal_(
+      width, height, rotation90, 1, 0, 0, null);
 };
 
 /**
- * Obtains CSS transformation that makes the cropped image fit the original
- * image. The new cropped image that the transformation is applied to fits to
- * the cropped rectangle in the original image.
- *
- * @param {number} imageWidth Width of the original image.
- * @param {number} imageHeight Height of the original image.
- * @param {!ImageRect} imageCropRect Crop rectangle in the image's coordinate
- *     system.
- * @return {string} Transformation description.
+ * @private

[mtomasz, 2015/05/14 00:32:32]

nit: @jsdoc missing

[hirono, 2015/05/14 01:57:21]

Done.

  */
-Viewport.prototype.getInverseTransformForCroppedImage =
-    function(imageWidth, imageHeight, imageCropRect) {
-  var wholeScale = this.getFittingScaleForImageSize_(
-      imageWidth, imageHeight);
-  var croppedScale = this.getFittingScaleForImageSize_(
-      imageCropRect.width, imageCropRect.height);
-  var dx =
-      (imageCropRect.left + imageCropRect.width / 2 - imageWidth / 2) *
-      wholeScale;
-  var dy =
-      (imageCropRect.top + imageCropRect.height / 2 - imageHeight / 2) *
-      wholeScale;
+Viewport.prototype.getTransformationInternal_ = function(
+    width,
+    height,
+    rotation90,
+    zoom,
+    offsetX,
+    offsetY,
+    cropRect) {
+  var rotatedWidth = rotation90 % 2 ? height : width;
+  var rotatedHeight = rotation90 % 2 ? width : height;
+  var rotatedMaxWidth = rotation90 % 2 ?
+      this.imageBounds_.height : this.imageBounds_.width;
+  var rotatedMaxHeight = rotation90 % 2 ?
+      this.imageBounds_.width : this.imageBounds_.height;
+
+  // Scale.
+  var fittingScale = this.getFittingScaleForImageSize_(
+      rotatedWidth, rotatedHeight, rotatedMaxWidth, rotatedMaxHeight);
+
+  // Offset for centering.
+  var dx = (this.screenBounds_.width - width) / 2;
+  var dy = (this.screenBounds_.height - height) / 2;
+
   return [
-    'translate(' + dx + 'px,' + dy + 'px)',
-    'scale(' + wholeScale / croppedScale + ')',
-    this.getTransformation()
-  ].join(' ');
+    'translate(', dx + offsetX, 'px,', dy + offsetY, 'px) ',
+    'scale(', fittingScale * zoom, ') ',
+    'rotate(', rotation90 * 90, 'deg) '
+  ].join('');
 };
-
-/**
- * Obtains CSS transformation that makes the image fit to the screen rectangle.
- *
- * @param {!ImageRect} screenRect Screen rectangle.
- * @return {string} Transformation description.
- */
-Viewport.prototype.getScreenRectTransformForImage = function(screenRect) {
-  var imageBounds = this.getImageElementBoundsOnScreen();
-  var scaleX = screenRect.width / imageBounds.width;
-  var scaleY = screenRect.height / imageBounds.height;
-  var screenWidth = this.screenBounds_.width;
-  var screenHeight = this.screenBounds_.height;
-  var dx = screenRect.left + screenRect.width / 2 - screenWidth / 2;
-  var dy = screenRect.top + screenRect.height / 2 - screenHeight / 2;
-  return [
-    'translate(' + dx + 'px,' + dy + 'px)',
-    'scale(' + scaleX + ',' + scaleY + ')',
-    this.getTransformation()
-  ].join(' ');
-};