Set orientation to 'AUTO' if we hide orientation option from user.

Set orientation to 'AUTO' if we hide orientation option from user.

BUG=383877
Committed: https://crrev.com/317e48ff59b1e42c730c1edbc494bd3f67079ead
Cr-Commit-Position: refs/heads/master@{#295545}

[Vitaly Buka (NO REVIEWS), 2014-09-18 18:35:48]

vitalybuka@chromium.org changed reviewers:
+ alekseys@chromium.org

[Aleksey Shlyapnikov, 2014-09-18 19:03:01]

lgtm

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
File chrome/browser/resources/print_preview/data/ticket_items/landscape.js
(right):

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:94: //
TODO(vitalybuka): For landscape PDF dimmensions should be
dimensions

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:96: return
(!doc.isModifiable || doc.hasCssMediaStyles) ?
Let's not add this check.

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:122: }
Remove brackets.

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:124:
cap.option.forEach(function(option) {
return capability.option.some(...

[Vitaly Buka (NO REVIEWS), 2014-09-18 19:08:47]

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
File chrome/browser/resources/print_preview/data/ticket_items/landscape.js
(right):

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:94: //
TODO(vitalybuka): For landscape PDF dimmensions should be
On 2014/09/18 19:03:01, Aleksey Shlyapnikov wrote:
> dimensions

Done.

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:96: return
(!doc.isModifiable || doc.hasCssMediaStyles) ?
On 2014/09/18 19:03:01, Aleksey Shlyapnikov wrote:
> Let's not add this check.

Done.

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:122: }
On 2014/09/18 19:03:01, Aleksey Shlyapnikov wrote:
> Remove brackets.

Done.

https://codereview.chromium.org/580203002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/data/ticket_items/landscape.js:124:
cap.option.forEach(function(option) {
On 2014/09/18 19:03:01, Aleksey Shlyapnikov wrote:
> return capability.option.some(...

Done.

[Vitaly Buka (NO REVIEWS), 2014-09-18 19:11:08]

vitalybuka@chromium.org changed reviewers:
+ thestig@chromium.org

[Lei Zhang, 2014-09-18 19:20:12]

lgtm stamp

[Vitaly Buka (NO REVIEWS), 2014-09-18 19:20:28]

The CQ bit was checked by vitalybuka@chromium.org

[commit-bot: I haz the power, 2014-09-18 19:21:45]

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

[commit-bot: I haz the power, 2014-09-18 20:22:14]

Committed patchset #2 (id:20001) as be279069837e9baf61a9c2156a8ab7929fc86ec7

[commit-bot: I haz the power, 2014-09-18 20:22:51]

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