Remove print preview scaling flag

Remove print preview scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2834453004
Cr-Commit-Position: refs/heads/master@{#467346}
Committed: https://chromium.googlesource.com/chromium/src/+/f0597dfb66bbd6824a3c9e3ae14070fc882b422e

[rbpotter, 2017-04-20 00:38:55]

Description was changed from

==========
Remove scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
==========

to

==========
Remove scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[rbpotter, 2017-04-20 00:39:14]

The CQ bit was checked by rbpotter@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-20 00:40:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 01:47:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-20 01:47:11]

Dry run: This issue passed the CQ dry run.

[rbpotter, 2017-04-20 16:39:38]

Description was changed from

==========
Remove scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[rbpotter, 2017-04-20 16:39:38]

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

[Lei Zhang, 2017-04-20 18:53:21]

lgtm

https://codereview.chromium.org/2834453004/diff/1/chrome/common/chrome_featur...
File chrome/common/chrome_features.cc (right):

https://codereview.chromium.org/2834453004/diff/1/chrome/common/chrome_featur...
chrome/common/chrome_features.cc:261: // Enables the Print as Image feature in
print preview.
Move the comment inside the #if while we are here? Like line 255 above.

[Lei Zhang, 2017-04-20 18:53:42]

Description was changed from

==========
Remove scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove print preview scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[rbpotter, 2017-04-22 00:46:57]

rbpotter@chromium.org changed reviewers:
+ dpapad@chromium.org

[rbpotter, 2017-04-22 00:47:27]

https://codereview.chromium.org/2834453004/diff/1/chrome/common/chrome_featur...
File chrome/common/chrome_features.cc (right):

https://codereview.chromium.org/2834453004/diff/1/chrome/common/chrome_featur...
chrome/common/chrome_features.cc:261: // Enables the Print as Image feature in
print preview.
On 2017/04/20 18:53:21, Lei Zhang wrote:
> Move the comment inside the #if while we are here? Like line 255 above.

Done.

[dpapad, 2017-04-25 21:29:10]

LGTM

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
File chrome/test/data/webui/print_preview.js (right):

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
chrome/test/data/webui/print_preview.js:64: // Enable print as image for tests
on non Windows/Mac
Period missing.

[rbpotter, 2017-04-26 00:32:55]

The CQ bit was checked by rbpotter@chromium.org to run a CQ dry run

[rbpotter, 2017-04-26 00:33:01]

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
File chrome/test/data/webui/print_preview.js (right):

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
chrome/test/data/webui/print_preview.js:64: // Enable print as image for tests
on non Windows/Mac
On 2017/04/25 21:29:10, dpapad wrote:
> Period missing.

Done.

[rbpotter, 2017-04-26 00:33:01]

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
File chrome/test/data/webui/print_preview.js (right):

https://codereview.chromium.org/2834453004/diff/20001/chrome/test/data/webui/...
chrome/test/data/webui/print_preview.js:64: // Enable print as image for tests
on non Windows/Mac
On 2017/04/25 21:29:10, dpapad wrote:
> Period missing.

Done.

[commit-bot: I haz the power, 2017-04-26 00:33:55]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-26 02:26:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-26 02:26:08]

Dry run: This issue passed the CQ dry run.

[rbpotter, 2017-04-26 16:37:15]

The CQ bit was checked by rbpotter@chromium.org

[rbpotter, 2017-04-26 16:37:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, dpapad@chromium.org
Link to the patchset: https://codereview.chromium.org/2834453004/#ps40001
(title: "address comment")

[commit-bot: I haz the power, 2017-04-26 16:37:51]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-26 16:42:23]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493224635495780,
"parent_rev": "9fbda07cfc941305ace888e3d3b742cdb0295bbf", "commit_rev":
"f0597dfb66bbd6824a3c9e3ae14070fc882b422e"}

[commit-bot: I haz the power, 2017-04-26 16:42:32]

Description was changed from

==========
Remove print preview scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Remove print preview scaling flag

Remove the print scaling flag since it is now fully launched to Stable.

BUG=659430
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2834453004
Cr-Commit-Position: refs/heads/master@{#467346}
Committed:
https://chromium.googlesource.com/chromium/src/+/f0597dfb66bbd6824a3c9e3ae140...
==========

[commit-bot: I haz the power, 2017-04-26 16:42:33]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/f0597dfb66bbd6824a3c9e3ae140...