Crop image at image_loader.

Crop image at image_loader.

To avoid wide image becomes blurry thumbnail, crop image at image_loader.
Currently it is used only for thumbnails, we only support to crop an image into square.

BUG=480679
TEST=ImageLoaderJsTest.ImageLoaderTest

Committed: https://crrev.com/1fc7409719fa3598a2861aaa9bfcd8d3a00be696
Cr-Commit-Position: refs/heads/master@{#329609}

[yawano, 2015-05-13 04:09:46]

yawano@chromium.org changed reviewers:
+ hirono@chromium.org

[yawano, 2015-05-13 04:09:47]

PTAL. Thank you!

[hirono, 2015-05-13 09:02:25]

lgtm!

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
File ui/file_manager/image_loader/image_loader.js (right):

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
ui/file_manager/image_loader/image_loader.js:268: source.width < source.height ?
source.width : source.height;
nit: You can use Math.min.

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
ui/file_manager/image_loader/image_loader.js:296:
console.log(JSON.stringify(targetCanvasDimensions));
Remove the debug log.

[yawano, 2015-05-13 09:11:11]

Thank you!

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
File ui/file_manager/image_loader/image_loader.js (right):

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
ui/file_manager/image_loader/image_loader.js:268: source.width < source.height ?
source.width : source.height;
On 2015/05/13 09:02:24, hirono wrote:
> nit: You can use Math.min.

Done.

https://codereview.chromium.org/1137993007/diff/1/ui/file_manager/image_loade...
ui/file_manager/image_loader/image_loader.js:296:
console.log(JSON.stringify(targetCanvasDimensions));
On 2015/05/13 09:02:24, hirono wrote:
> Remove the debug log.

Done. Thank you!

[yawano, 2015-05-13 09:11:16]

The CQ bit was checked by yawano@chromium.org

[yawano, 2015-05-13 09:11:17]

The patchset sent to the CQ was uploaded after l-g-t-m from hirono@chromium.org
Link to the patchset: https://codereview.chromium.org/1137993007/#ps20001
(title: "Address comments.")

[commit-bot: I haz the power, 2015-05-13 09:11:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1137993007/20001

[commit-bot: I haz the power, 2015-05-13 09:58:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-13 09:59:11]

Patchset 2 (id:??) landed as
https://crrev.com/1fc7409719fa3598a2861aaa9bfcd8d3a00be696
Cr-Commit-Position: refs/heads/master@{#329609}