Gallery: do not set zoom more than once in a frame.

Gallery: do not set zoom more than once in a frame.

BUG=591033
TEST=manually tested as described in the issue.

Committed: https://crrev.com/2afcf7037d39cd4cedf82d235123d5a3efa073c9
Cr-Commit-Position: refs/heads/master@{#379774}

[yawano, 2016-03-08 06:12:49]

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

[yawano, 2016-03-08 06:12:49]

PTAL. Thank you!

[fukino, 2016-03-08 06:28:54]

lgtm with a nit.

https://codereview.chromium.org/1770903003/diff/1/ui/file_manager/gallery/js/...
File ui/file_manager/gallery/js/slide_mode.js (right):

https://codereview.chromium.org/1770903003/diff/1/ui/file_manager/gallery/js/...
ui/file_manager/gallery/js/slide_mode.js:2090: // Request animation frame not to
set zoom more than once in a frame.
nit: please add a like for the bug this code is trying to fix.

[yawano, 2016-03-08 06:41:03]

Thank you!

https://codereview.chromium.org/1770903003/diff/1/ui/file_manager/gallery/js/...
File ui/file_manager/gallery/js/slide_mode.js (right):

https://codereview.chromium.org/1770903003/diff/1/ui/file_manager/gallery/js/...
ui/file_manager/gallery/js/slide_mode.js:2090: // Request animation frame not to
set zoom more than once in a frame.
On 2016/03/08 06:28:54, fukino wrote:
> nit: please add a like for the bug this code is trying to fix.

Done.

[yawano, 2016-03-08 06:49:28]

The CQ bit was checked by yawano@chromium.org

[yawano, 2016-03-08 06:49:28]

The patchset sent to the CQ was uploaded after l-g-t-m from fukino@chromium.org
Link to the patchset: https://codereview.chromium.org/1770903003/#ps20001
(title: "Add link to the issue.")

[commit-bot: I haz the power, 2016-03-08 06:49:41]

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

[commit-bot: I haz the power, 2016-03-08 07:29:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 07:30:34]

Description was changed from

==========
Gallery: do not set zoom more than once in a frame.

BUG=591033
TEST=manually tested as described in the issue.
==========

to

==========
Gallery: do not set zoom more than once in a frame.

BUG=591033
TEST=manually tested as described in the issue.

Committed: https://crrev.com/2afcf7037d39cd4cedf82d235123d5a3efa073c9
Cr-Commit-Position: refs/heads/master@{#379774}
==========

[commit-bot: I haz the power, 2016-03-08 07:30:36]

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