Improve postscript language level detection

Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=614988
Review-Url: https://codereview.chromium.org/2691653002
Cr-Commit-Position: refs/heads/master@{#449749}
Committed: https://chromium.googlesource.com/chromium/src/+/54f7fa845c0e48f14955b46d83ccff44f90281da

[rbpotter, 2017-02-10 21:03:07]

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

[rbpotter, 2017-02-10 21:03:17]

Description was changed from

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=61988
==========

to

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=61988
==========

[rbpotter, 2017-02-10 21:03:17]

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

[commit-bot: I haz the power, 2017-02-10 21:03:47]

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

[Vitaly Buka (NO REVIEWS), 2017-02-10 21:15:39]

lgtm

[commit-bot: I haz the power, 2017-02-10 21:57:27]

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

[commit-bot: I haz the power, 2017-02-10 21:57:27]

Dry run: This issue passed the CQ dry run.

[rbpotter, 2017-02-10 21:57:56]

The CQ bit was checked by rbpotter@chromium.org

[commit-bot: I haz the power, 2017-02-10 21:58: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-02-10 22:05:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486763876347820, "parent_rev":
"7d1ee017dbdb0d8e3890d40891dbb52ac53ed84f", "commit_rev":
"54f7fa845c0e48f14955b46d83ccff44f90281da"}

[commit-bot: I haz the power, 2017-02-10 22:06:28]

Description was changed from

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=61988
==========

to

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=61988

Review-Url: https://codereview.chromium.org/2691653002
Cr-Commit-Position: refs/heads/master@{#449749}
Committed:
https://chromium.googlesource.com/chromium/src/+/54f7fa845c0e48f14955b46d83cc...
==========

[commit-bot: I haz the power, 2017-02-10 22:06:29]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/54f7fa845c0e48f14955b46d83cc...

[rbpotter, 2017-02-10 22:07:52]

Description was changed from

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=61988

Review-Url: https://codereview.chromium.org/2691653002
Cr-Commit-Position: refs/heads/master@{#449749}
Committed:
https://chromium.googlesource.com/chromium/src/+/54f7fa845c0e48f14955b46d83cc...
==========

to

==========
Improve postscript language level detection

Improve language level detection by setting printer mode to Postscript
if it is identified as postscript before performing any other ExtEscape
commands. Change PASSTHROUGH to POSTSCRIPT_PASSTHROUGH since normal
PASSTHROUGH is not supported in PS mode.

BUG=614988

Review-Url: https://codereview.chromium.org/2691653002
Cr-Commit-Position: refs/heads/master@{#449749}
Committed:
https://chromium.googlesource.com/chromium/src/+/54f7fa845c0e48f14955b46d83cc...
==========

[Lei Zhang, 2017-02-25 02:49:49]

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

[Lei Zhang, 2017-02-25 02:49:50]

https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
File printing/print_settings_initializer_win.cc (left):

https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
printing/print_settings_initializer_win.cc:40: // Query the PS Level if
possible. Many PS printers do not implement this.
Are there printers that only support this in PS mode or something?

https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
File printing/print_settings_initializer_win.cc (right):

https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
printing/print_settings_initializer_win.cc:27: if (strcmp(technology,
"PostScript") == 0 &&
Doing this within IsTechnology() seems weird, but it looks like we are going to
fix it in https://codereview.chromium.org/2714073002/.

[rbpotter, 2017-02-27 16:58:48]

On 2017/02/25 02:49:50, Lei Zhang (super slow) wrote:
>
https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
> File printing/print_settings_initializer_win.cc (left):
> 
>
https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
> printing/print_settings_initializer_win.cc:40: // Query the PS Level if
> possible. Many PS printers do not implement this.
> Are there printers that only support this in PS mode or something?
> 
>
https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
> File printing/print_settings_initializer_win.cc (right):
> 
>
https://codereview.chromium.org/2691653002/diff/1/printing/print_settings_ini...
> printing/print_settings_initializer_win.cc:27: if (strcmp(technology,
> "PostScript") == 0 &&
> Doing this within IsTechnology() seems weird, but it looks like we are going
to
> fix it in https://codereview.chromium.org/2714073002/.

Yes, most printers only support the postscript level function in PS (or GDI)
mode.

New patch fixes the IsTechnology() issue.