OOP PDF: Reset FittingType when manually setting the zoom.

OOP PDF: Reset FittingType when manually setting the zoom.

Currently, when the zoom level is set to a particular value, the
viewport's fitting type is unchanged. This can cause the zoom to be
manually set to one value, but be overridden if the viewport changes
size as it will automatically re-apply the zoom for the fitting type.
This CL fixes this by setting the fitting type to none when the zoom is
set manually.

BUG=470427
Committed: https://crrev.com/5050705ee16d1b3ca4acf279c0d174a5336629fb
Cr-Commit-Position: refs/heads/master@{#322502}

[Sam McNally, 2015-03-25 06:58:45]

sammc@chromium.org changed reviewers:
+ raymes@chromium.org

[raymes, 2015-03-26 05:52:24]

https://codereview.chromium.org/1038573003/diff/1/chrome/test/data/pdf/viewpo...
File chrome/test/data/pdf/viewport_test.js (right):

https://codereview.chromium.org/1038573003/diff/1/chrome/test/data/pdf/viewpo...
chrome/test/data/pdf/viewport_test.js:119:
chrome.test.assertEq(Viewport.FittingType.NONE, viewport.fittingType);
can we test the fitting type after fittopage and fittowidth (perhaps in their
respective tests below?)

[Sam McNally, 2015-03-26 23:05:02]

https://codereview.chromium.org/1038573003/diff/1/chrome/test/data/pdf/viewpo...
File chrome/test/data/pdf/viewport_test.js (right):

https://codereview.chromium.org/1038573003/diff/1/chrome/test/data/pdf/viewpo...
chrome/test/data/pdf/viewport_test.js:119:
chrome.test.assertEq(Viewport.FittingType.NONE, viewport.fittingType);
On 2015/03/26 05:52:24, raymes wrote:
> can we test the fitting type after fittopage and fittowidth (perhaps in their
> respective tests below?)

Done.

[raymes, 2015-03-26 23:22:51]

lgtm, thanks!

[Sam McNally, 2015-03-26 23:31:36]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2015-03-26 23:32:12]

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

[commit-bot: I haz the power, 2015-03-27 00:01:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-27 00:03:04]

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