Fix infinite loading for small images in Files.app.

Fix infinite loading for small images in Files.app.

When images are small and the same as thumbnails, then the onload callback is not called. This patch fixes this by explicitely clearing the src attribute before any load.

TEST=Follow the bug's repro steps on Drive. The image should load fast.
BUG=248235
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205782

[mtomasz, 2013-06-11 06:20:28]

@yoshiki: PTAL. Thanks.

[yoshiki, 2013-06-11 06:45:39]

lgtm with nit

https://codereview.chromium.org/16124013/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/image_editor/image_util.js
(right):

https://codereview.chromium.org/16124013/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/image_editor/image_util.js:507: var
startLoad = function() {
nit:
var startLoad = resetImage.bind(null, loadImage);

[mtomasz, 2013-06-11 08:14:13]

https://codereview.chromium.org/16124013/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/image_editor/image_util.js
(right):

https://codereview.chromium.org/16124013/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/image_editor/image_util.js:507: var
startLoad = function() {
On 2013/06/11 06:45:39, yoshiki wrote:
> nit:
> var startLoad = resetImage.bind(null, loadImage);

Done.

[commit-bot: I haz the power, 2013-06-11 08:14:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/16124013/5001

[commit-bot: I haz the power, 2013-06-11 09:53:19]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-06-12 03:41:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/16124013/5001

[commit-bot: I haz the power, 2013-06-12 05:14:56]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-06-12 05:20:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/16124013/5001

[commit-bot: I haz the power, 2013-06-12 06:33:34]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-12 14:07:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/16124013/5001

[commit-bot: I haz the power, 2013-06-12 15:11:22]

Change committed as 205782