Remove repaint callbacks from Viewport.

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.
 
 'use strict';
 
 /**
  * Viewport class controls the way the image is displayed (scale, offset etc).
  * @constructor
  */
@@ skipping 17 matching lines @@
    * not zoom operated by users.
    * @type {number}
    * @private
    */
   this.scale_ = 1;
   this.offsetX_ = 0;
   this.offsetY_ = 0;
 
   this.generation_ = 0;
 
-  this.repaintCallbacks_ = [];
   this.update();
 }
 
 /**
  * @param {number} width Image width.
  * @param {number} height Image height.
  */
 Viewport.prototype.setImageSize = function(width, height) {
   this.imageBounds_ = new Rect(width, height);
   this.invalidateCaches();
@@ skipping 99 matching lines @@
     x = this.clampOffsetX_(x);
     y = this.clampOffsetY_(y);
   }
   if (this.offsetX_ == x && this.offsetY_ == y) return;
   this.offsetX_ = x;
   this.offsetY_ = y;
   this.invalidateCaches();
 };
 
 /**
- * Return a closure that can be called to pan the image.
- * Useful for implementing non-trivial variants of panning (overview etc).
- * @param {number} originalX The x coordinate on the screen canvas that
- *                 corresponds to zero change to offsetX.
- * @param {number} originalY The y coordinate on the screen canvas that
- *                 corresponds to zero change to offsetY.
- * @param {function():number} scaleFunc returns the image to screen scale.
- * @param {function(number,number):boolean} hitFunc returns true if (x,y) is
- *                                                  in the valid region.
- * @return {function} The closure to pan the image.
- */
-Viewport.prototype.createOffsetSetter = function(
-    originalX, originalY, scaleFunc, hitFunc) {
-  var originalOffsetX = this.offsetX_;
-  var originalOffsetY = this.offsetY_;
-  if (!hitFunc) hitFunc = function() { return true; };
-  if (!scaleFunc) scaleFunc = this.getScale.bind(this);
-
-  var self = this;
-  return function(x, y) {
-    if (hitFunc(x, y)) {
-      var scale = scaleFunc();
-      self.setOffset(
-          originalOffsetX + (x - originalX) / scale,
-          originalOffsetY + (y - originalY) / scale);
-      self.repaint();
-    }
-  };
-};
-
-/*
- * Access to the current viewport state.
- */
-
-/**
  * @return {Rect} The image bounds in image coordinates.
  */
 Viewport.prototype.getImageBounds = function() { return this.imageBounds_; };
 
 /**
 * @return {Rect} The screen bounds in screen coordinates.
 */
 Viewport.prototype.getScreenBounds = function() { return this.screenBounds_; };
 
 /**
@@ skipping 10 matching lines @@
 
 /**
  * A counter that is incremented with each viewport state change.
  * Clients that cache anything that depends on the viewport state should keep
  * track of this counter.
  * @return {number} counter.
  */
 Viewport.prototype.getCacheGeneration = function() { return this.generation_; };
 
 /**
- * Called on event view port state change (even if repaint has not been called).
+ * Called on event view port state change.
  */
 Viewport.prototype.invalidateCaches = function() { this.generation_++; };
 
 /**
  * @return {Rect} The image bounds in screen coordinates.
  */
 Viewport.prototype.getImageBoundsOnScreen = function() {
   return this.imageOnScreen_;
 };
 
-/*
- * Conversion between the screen and image coordinate spaces.
- */
-
 /**
  * @param {number} size Size in screen coordinates.
  * @return {number} Size in image coordinates.
  */
 Viewport.prototype.screenToImageSize = function(size) {
   return size / this.getScale();
 };
 
 /**
  * @param {number} x X in screen coordinates.
@@ skipping 169 matching lines @@
         Math.round(this.clampOffsetY_(this.offsetY_) * scale);
     this.imageClipped_.top = Math.round(-this.imageOnScreen_.top / scale);
     this.imageClipped_.height = Math.round(this.screenBounds_.height / scale);
   } else {
     this.screenClipped_.top = this.imageOnScreen_.top;
     this.screenClipped_.height = this.imageOnScreen_.height;
   }
 };
 
 /**
- * @param {function} callback Repaint callback.
- */
-Viewport.prototype.addRepaintCallback = function(callback) {
-  this.repaintCallbacks_.push(callback);
-};
-
-/**
- * Repaint all clients.
- */
-Viewport.prototype.repaint = function() {
-  this.update();
-  for (var i = 0; i != this.repaintCallbacks_.length; i++)
-    this.repaintCallbacks_[i]();
-};
-
-/**
  * Obtains CSS transformation for the screen image.
  * @return {string} Transformation description.
  */
 Viewport.prototype.getTransformation = function() {
   return 'scale(' + (1 / window.devicePixelRatio) + ')';
 };
 
 /**
  * Obtains shift CSS transformation for the screen image.
  * @param {number} dx Amount of shift.
@@ skipping 70 matching lines @@
   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(' ');
 };