Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(821)

Unified Diff: chrome/browser/resources/file_manager/js/image_editor/image_buffer.js

Issue 12254015: Revert 182242 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/file_manager/js/image_editor/image_buffer.js
===================================================================
--- chrome/browser/resources/file_manager/js/image_editor/image_buffer.js (revision 182284)
+++ chrome/browser/resources/file_manager/js/image_editor/image_buffer.js (working copy)
@@ -13,10 +13,7 @@
this.overlays_ = [];
}
-/**
- * TODO(JSDOC).
- * @param {ImageBuffer.Overlay} overlay //TODO(JSDOC).
- */
+//TODO(JSDOC)
ImageBuffer.prototype.addOverlay = function(overlay) {
var zIndex = overlay.getZIndex();
// Store the overlays in the ascending Z-order.
@@ -27,10 +24,7 @@
this.overlays_.splice(i, 0, overlay);
};
-/**
- * TODO(JSDOC).
- * @param {ImageBuffer.Overlay} overlay //TODO(JSDOC).
- */
+//TODO(JSDOC)
ImageBuffer.prototype.removeOverlay = function(overlay) {
for (var i = 0; i != this.overlays_.length; i++) {
if (this.overlays_[i] == overlay) {
@@ -125,58 +119,25 @@
* and the behavior of the ImageBuffer instance.
* @class
*/
+//TODO(JSDOC)
ImageBuffer.Overlay = function() {};
-/**
- * TODO(JSDOC).
- * @return {number} //TODO(JSDOC).
- */
+//TODO(JSDOC)
ImageBuffer.Overlay.prototype.getZIndex = function() { return 0 };
-/**
- * TODO(JSDOC).
- */
+//TODO(JSDOC)
ImageBuffer.Overlay.prototype.draw = function() {};
-/**
- * TODO(JSDOC).
- * @param {number} x X coordinate for cursor.
- * @param {number} y Y coordinate for cursor.
- * @param {boolean} mouseDown If mouse button is down.
- * @return {?string} A value for style.cursor CSS property or null for
- * default.
- */
-ImageBuffer.Overlay.prototype.getCursorStyle = function(x, y, mouseDown) {
- return null;
-};
+//TODO(JSDOC)
+ImageBuffer.Overlay.prototype.getCursorStyle = function() { return null };
-/**
- * TODO(JSDOC).
- * @param {number} x //TODO(JSDOC).
- * @param {number} y //TODO(JSDOC).
- * @return {boolean} //TODO(JSDOC).
- */
-ImageBuffer.Overlay.prototype.onClick = function(x, y) {
- return false;
-};
+//TODO(JSDOC)
+ImageBuffer.Overlay.prototype.onClick = function() { return false };
-/**
- * TODO(JSDOC).
- * @param {number} x Event X coordinate.
- * @param {number} y Event Y coordinate.
- * @param {boolean} touch True if it's a touch event, false if mouse.
- * @return {function(number,number)} A function to be called on mouse drag.
- */
-ImageBuffer.Overlay.prototype.getDragHandler = function(x, y, touch) {
- return null;
-};
+//TODO(JSDOC)
+ImageBuffer.Overlay.prototype.getDragHandler = function() { return null };
-/**
- * TODO(JSDOC).
- * @param {number} x //TODO(JSDOC).
- * @param {number} y //TODO(JSDOC).
- * @return {ImageBuffer.DoubleTapAction} //TODO(JSDOC).
- */
+//TODO(JSDOC)
ImageBuffer.Overlay.prototype.getDoubleTapAction = function(x, y) {
return ImageBuffer.DoubleTapAction.NOTHING;
};

Powered by Google App Engine
This is Rietveld 408576698