Chromium Code Reviews| Index: ui/file_manager/gallery/js/image_editor/image_transform.js |
| diff --git a/ui/file_manager/gallery/js/image_editor/image_transform.js b/ui/file_manager/gallery/js/image_editor/image_transform.js |
| index 2f06b831d595668c6ebd09db40ff3bae992474cd..aa5ae345a9dc043e9c78c77233794768068206b0 100644 |
| --- a/ui/file_manager/gallery/js/image_editor/image_transform.js |
| +++ b/ui/file_manager/gallery/js/image_editor/image_transform.js |
| @@ -143,6 +143,7 @@ ImageEditor.Mode.Crop.prototype.createTools = function(toolbar) { |
| GALLERY_ASPECT_RATIO_16_9: 16 / 9 |
| }; |
| + // TODO(fukino): The loop order is not guaranteed. Fix it! |
|
oka
2016/10/06 09:11:55
suggestion: Consider using Map.
A Map object itera
fukino
2016/10/06 14:34:30
Map can be used if tooling support works well. I'l
|
| for (var name in aspects) { |
| var button = toolbar.addButton( |
| name, |