Change ImageLoaderClient to return width and height of image.

Change ImageLoaderClient to return width and height of image.

ListThumbnailLoader(http://crrev.com/885323002) directly returns dataUrl, width and height of thumbnail image instead of attaching it to image element.
We want to directly obtain dataUrl, width and height of thumbnail from ImageLoaderClient.

BUG=438050
TEST=Manually tested.

Committed: https://crrev.com/d708495698cccaf3e1ced6bbbb7f90222e807f86
Cr-Commit-Position: refs/heads/master@{#314120}

[yawano, 2015-02-02 06:11:08]

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

[yawano, 2015-02-02 06:11:08]

PTAL. Thank you!

[hirono, 2015-02-02 06:56:01]

lgtm with a nit. Thanks!

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

https://codereview.chromium.org/890313002/diff/1/ui/file_manager/image_loader...
ui/file_manager/image_loader/image_loader_client.js:121: * @param
{function(Object)} callback Callback used to return response.
Could you describe about what are stored in Object passed to the callback?
Especially whether the width and the height are the original one or the resized
one.

[yawano, 2015-02-02 07:13:30]

Thank you for the review!

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

https://codereview.chromium.org/890313002/diff/1/ui/file_manager/image_loader...
ui/file_manager/image_loader/image_loader_client.js:121: * @param
{function(Object)} callback Callback used to return response.
On 2015/02/02 06:56:01, hirono wrote:
> Could you describe about what are stored in Object passed to the callback?
> Especially whether the width and the height are the original one or the
resized
> one.

Done.

[yawano, 2015-02-02 07:34:51]

The CQ bit was checked by yawano@chromium.org

[commit-bot: I haz the power, 2015-02-02 07:35:54]

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

[commit-bot: I haz the power, 2015-02-02 08:36:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-02 08:36:55]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yawano, 2015-02-02 08:48:57]

The previous patch set broke a test case. I fixed it with a new patch set. PTAL.
Thank you!

[hirono, 2015-02-02 08:50:47]

still lgtm!

[yawano, 2015-02-02 08:51:30]

On 2015/02/02 08:50:47, hirono wrote:
> still lgtm!

Thank you!

[yawano, 2015-02-02 08:51:37]

The CQ bit was checked by yawano@chromium.org

[commit-bot: I haz the power, 2015-02-02 08:52:08]

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

[commit-bot: I haz the power, 2015-02-02 09:31:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-02 09:32:28]

Patchset 3 (id:??) landed as
https://crrev.com/d708495698cccaf3e1ced6bbbb7f90222e807f86
Cr-Commit-Position: refs/heads/master@{#314120}