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

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

Issue 12262017: Revert 182285 (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 182297)
+++ chrome/browser/resources/file_manager/js/image_editor/image_buffer.js (working copy)
@@ -13,7 +13,10 @@
this.overlays_ = [];
}
-//TODO(JSDOC)
+/**
+ * TODO(JSDOC).
+ * @param {ImageBuffer.Overlay} overlay //TODO(JSDOC).
+ */
ImageBuffer.prototype.addOverlay = function(overlay) {
var zIndex = overlay.getZIndex();
// Store the overlays in the ascending Z-order.
@@ -24,7 +27,10 @@
this.overlays_.splice(i, 0, overlay);
};
-//TODO(JSDOC)
+/**
+ * TODO(JSDOC).
+ * @param {ImageBuffer.Overlay} overlay //TODO(JSDOC).
+ */
ImageBuffer.prototype.removeOverlay = function(overlay) {
for (var i = 0; i != this.overlays_.length; i++) {
if (this.overlays_[i] == overlay) {
@@ -119,25 +125,58 @@
* and the behavior of the ImageBuffer instance.
* @class
*/
-//TODO(JSDOC)
ImageBuffer.Overlay = function() {};
-//TODO(JSDOC)
+/**
+ * TODO(JSDOC).
+ * @return {number} //TODO(JSDOC).
+ */
ImageBuffer.Overlay.prototype.getZIndex = function() { return 0 };
-//TODO(JSDOC)
+/**
+ * TODO(JSDOC).
+ */
ImageBuffer.Overlay.prototype.draw = function() {};
-//TODO(JSDOC)
-ImageBuffer.Overlay.prototype.getCursorStyle = function() { return null };
+/**
+ * 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.onClick = function() { return false };
+/**
+ * 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.getDragHandler = function() { return null };
+/**
+ * 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)
+/**
+ * TODO(JSDOC).
+ * @param {number} x //TODO(JSDOC).
+ * @param {number} y //TODO(JSDOC).
+ * @return {ImageBuffer.DoubleTapAction} //TODO(JSDOC).
+ */
ImageBuffer.Overlay.prototype.getDoubleTapAction = function(x, y) {
return ImageBuffer.DoubleTapAction.NOTHING;
};

Powered by Google App Engine
This is Rietveld 408576698