Printing: Add command line switch to rendering text with GDI.

Printing: Add command line switch to rendering text with GDI.

Add --enable-gdi-text-printing for those who want to experiment with it.
Rendering text with GDI is still disabled by default and
--disable-gdi-text-printing overrules the enable flag.

BUG=409472, 658606
Committed: https://crrev.com/653aaed16ec270dcb3c4da44a5adabb501228bc3
Cr-Commit-Position: refs/heads/master@{#432052}

[Lei Zhang, 2016-11-14 22:25:37]

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

[commit-bot: I haz the power, 2016-11-14 22:26:45]

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

[Lei Zhang, 2016-11-14 23:15:06]

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

[Lei Zhang, 2016-11-14 23:15:06]

We haven't given up on this yet, so adding back the code from r429177 as
disabled by default, with a flag to enable it if desired. Thus we can take
future builds and experiment with printer + driver combinations to figure out
what went wrong.

[scottmg, 2016-11-14 23:29:03]

lgtm

https://codereview.chromium.org/2503663002/diff/1/chrome/common/chrome_switch...
File chrome/common/chrome_switches.h (right):

https://codereview.chromium.org/2503663002/diff/1/chrome/common/chrome_switch...
chrome/common/chrome_switches.h:357: extern const char kEnableGDITextPrinting[];
These were sorted-ish before.

[Lei Zhang, 2016-11-14 23:36:43]

https://codereview.chromium.org/2503663002/diff/1/chrome/common/chrome_switch...
File chrome/common/chrome_switches.h (right):

https://codereview.chromium.org/2503663002/diff/1/chrome/common/chrome_switch...
chrome/common/chrome_switches.h:357: extern const char kEnableGDITextPrinting[];
On 2016/11/14 23:29:03, scottmg wrote:
> These were sorted-ish before.

Indeed. Sorted.

[Lei Zhang, 2016-11-14 23:36:54]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-11-14 23:36:54]

The patchset sent to the CQ was uploaded after l-g-t-m from scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/2503663002/#ps20001
(title: "Sort")

[commit-bot: I haz the power, 2016-11-14 23:37:22]

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, 2016-11-15 01:51:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-15 01:56:52]

Description was changed from

==========
Printing: Add command line switch to rendering text with GDI.

Add --enable-gdi-text-printing for those who want to experiment with it.
Rendering text with GDI is still disabled by default and
--disable-gdi-text-printing overrules the enable flag.

BUG=409472,658606
==========

to

==========
Printing: Add command line switch to rendering text with GDI.

Add --enable-gdi-text-printing for those who want to experiment with it.
Rendering text with GDI is still disabled by default and
--disable-gdi-text-printing overrules the enable flag.

BUG=409472,658606

Committed: https://crrev.com/653aaed16ec270dcb3c4da44a5adabb501228bc3
Cr-Commit-Position: refs/heads/master@{#432052}
==========

[commit-bot: I haz the power, 2016-11-15 01:56:53]

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