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

Side by Side Diff: ui/file_manager/gallery/js/image_editor/image_transform.js

Issue 354543002: Gallery.app: Fix removeEventListener call in Gallery.app. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/chromeos/file_manager/file_manager_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 'use strict'; 5 'use strict';
6 6
7 /** 7 /**
8 * Crop mode. 8 * Crop mode.
9 * @constructor 9 * @constructor
10 */ 10 */
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 }; 118 };
119 119
120 /** 120 /**
121 * TODO(JSDOC). 121 * TODO(JSDOC).
122 */ 122 */
123 ImageEditor.Mode.Crop.prototype.cleanUpUI = function() { 123 ImageEditor.Mode.Crop.prototype.cleanUpUI = function() {
124 ImageEditor.Mode.prototype.cleanUpUI.apply(this, arguments); 124 ImageEditor.Mode.prototype.cleanUpUI.apply(this, arguments);
125 this.domOverlay_.parentNode.removeChild(this.domOverlay_); 125 this.domOverlay_.parentNode.removeChild(this.domOverlay_);
126 this.domOverlay_ = null; 126 this.domOverlay_ = null;
127 this.editor_.hideOverlappingTools(); 127 this.editor_.hideOverlappingTools();
128 window.removeEventListener(this.onResizedBound_); 128 window.removeEventListener('resize', this.onResizedBound_);
129 this.onResizedBound_ = null; 129 this.onResizedBound_ = null;
130 }; 130 };
131 131
132 /** 132 /**
133 * @const 133 * @const
134 * @type {number} 134 * @type {number}
135 */ 135 */
136 ImageEditor.Mode.Crop.MOUSE_GRAB_RADIUS = 6; 136 ImageEditor.Mode.Crop.MOUSE_GRAB_RADIUS = 6;
137 /** 137 /**
138 * @const 138 * @const
(...skipping 345 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 DraggableRect.prototype.getDoubleTapAction = function(x, y, touch) { 484 DraggableRect.prototype.getDoubleTapAction = function(x, y, touch) {
485 x = this.viewport_.screenToImageX(x); 485 x = this.viewport_.screenToImageX(x);
486 y = this.viewport_.screenToImageY(y); 486 y = this.viewport_.screenToImageY(y);
487 487
488 var clipRect = this.viewport_.getImageClipped(); 488 var clipRect = this.viewport_.getImageClipped();
489 if (clipRect.inside(x, y)) 489 if (clipRect.inside(x, y))
490 return ImageBuffer.DoubleTapAction.COMMIT; 490 return ImageBuffer.DoubleTapAction.COMMIT;
491 else 491 else
492 return ImageBuffer.DoubleTapAction.NOTHING; 492 return ImageBuffer.DoubleTapAction.NOTHING;
493 }; 493 };
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/file_manager/file_manager_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698