Gallery: Fix undo of cropping images.

Gallery: Fix undo of cropping images.

The CL also updates the browser test to handle the case.

BUG=401314
TEST=manually and run the test

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288300

[hirono, 2014-08-07 05:26:59]

PTAL the CL? Thank you!

[yoshiki, 2014-08-07 09:28:55]

lgtm

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
File
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js
(right):

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js:152:
* @param {function():Promise} callback Callback returns a promise.
unnecessary?

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js:156:
chrome.test.fail(error);
nit: please write out that this error is a window error.

[hirono, 2014-08-08 02:35:28]

Thank you!

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
File
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js
(right):

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js:152:
* @param {function():Promise} callback Callback returns a promise.
On 2014/08/07 09:28:55, yoshiki wrote:
> unnecessary?

Yes, I removed.

https://codereview.chromium.org/448973003/diff/1/chrome/test/data/extensions/...
chrome/test/data/extensions/api_test/file_manager_browsertest/gallery/test_util.js:156:
chrome.test.fail(error);
On 2014/08/07 09:28:55, yoshiki wrote:
> nit: please write out that this error is a window error.

Done.

[hirono, 2014-08-08 02:35:51]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-08-08 02:37:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hirono@chromium.org/448973003/20001

[commit-bot: I haz the power, 2014-08-08 06:43:16]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-08 10:11:37]

Change committed as 288300