Remove repaint callbacks from Viewport.

ui/file_manager/gallery/js/image_editor/image_view.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';
 
 /**
  * The overlay displaying the image.
  *
  * @param {HTMLElement} container The container element.
  * @param {Viewport} viewport The viewport.
  * @constructor
+ * @extends {ImageBuffer.Overlay}
  */
 function ImageView(container, viewport) {
+  ImageBuffer.Overlay.call(this);
+
   this.container_ = container;
   this.viewport_ = viewport;
   this.document_ = container.ownerDocument;
   this.contentGeneration_ = 0;
   this.displayedContentGeneration_ = 0;
   this.displayedViewportGeneration_ = 0;
 
   this.imageLoader_ = new ImageUtil.ImageLoader(this.document_);
   // We have a separate image loader for prefetch which does not get cancelled
   // when the selection changes.
@@ skipping 57 matching lines @@
 ImageView.LOAD_TYPE_OFFLINE = 4;
 
 /**
  * The total number of load types.
  */
 ImageView.LOAD_TYPE_TOTAL = 5;
 
 ImageView.prototype = {__proto__: ImageBuffer.Overlay.prototype};
 
 /**
- * Draws below overlays with the default zIndex.
- * @return {number} Z-index.
+ * @override
  */
 ImageView.prototype.getZIndex = function() { return -1; };
 
 /**
- * Draws the image on screen.
+ * @override
  */
 ImageView.prototype.draw = function() {
+  console.log('draw');

[mtomasz, 2014/07/11 06:34:11]

nit: remove

[hirono, 2014/07/11 06:41:17]

Done.

   if (!this.contentCanvas_)  // Do nothing if the image content is not set.
     return;
 
   var forceRepaint = false;
 
   if (this.displayedViewportGeneration_ !==
       this.viewport_.getCacheGeneration()) {
     this.displayedViewportGeneration_ = this.viewport_.getCacheGeneration();
 
     this.setupDeviceBuffer(this.screenImage_);
 
     forceRepaint = true;
   }
 
   if (forceRepaint ||
       this.displayedContentGeneration_ !== this.contentGeneration_) {
     this.displayedContentGeneration_ = this.contentGeneration_;
 
     ImageUtil.trace.resetTimer('paint');
     this.paintDeviceRect(this.viewport_.getDeviceClipped(),
         this.contentCanvas_, this.viewport_.getImageClipped());
     ImageUtil.trace.reportTimer('paint');
   }
 };
 
 /**
- * @param {number} x X pointer position.
- * @param {number} y Y pointer position.
- * @param {boolean} mouseDown True if mouse is down.
- * @return {string} CSS cursor style.
+ * @override
  */
 ImageView.prototype.getCursorStyle = function(x, y, mouseDown) {
   // Indicate that the image is draggable.
   if (this.viewport_.isClipped() &&
       this.viewport_.getScreenClipped().inside(x, y))
     return 'move';
 
   return null;
 };
 
 /**
- * @param {number} x X pointer position.
- * @param {number} y Y pointer position.
- * @return {function} The closure to call on drag.
+ * @override
  */
 ImageView.prototype.getDragHandler = function(x, y) {
   var cursor = this.getCursorStyle(x, y);
   if (cursor === 'move') {
     // Return the handler that drags the entire image.
     return this.viewport_.createOffsetSetter(x, y);
   }
 
   return null;
 };
@@ skipping 824 matching lines @@
 };
 
 ImageView.Effect.Rotate.prototype = { __proto__: ImageView.Effect.prototype };
 
 /**
  * @override
  */
 ImageView.Effect.Rotate.prototype.transform = function(element, viewport) {
   return viewport.getInverseTransformForRotatedImage(this.orientation_);
 };