Enable basic printing on ChromeOS

Enable basic printing on ChromeOS

Rudimentary print support is being implemented for Chrome OS using CUPS.
This enables native printing on Chrome OS.  If CUPS is not enabled, no
differences are noticable.

BUG=607668
R=briannorris@chromium.org

Committed: https://crrev.com/687d7248d07c24b9c6a5da4e9ee5c4dd939f16e9
Cr-Commit-Position: refs/heads/master@{#404913}

[skau, 2016-07-08 00:56:12]

skau@chromium.org changed reviewers:
+ dpranke@chromium.org

[skau, 2016-07-08 00:56:13]

I would like to enable basic printing for Chrome OS as I have a functioning
print system using CUPS which is under review.

[skau, 2016-07-12 18:39:38]

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

[skau, 2016-07-12 18:39:38]

Could you look at this when you get a chance?

[Dirk Pranke, 2016-07-12 19:06:36]

lgtm

[Lei Zhang, 2016-07-12 19:22:03]

Never got an email for this CL until today. Did you forget to add reviewers
earlier?

Basic printing on other platforms imply there's a native system print dialog.
Does one exist on ChromeOS yet? Can you remind me what the difference is with
this CL when CUPS is enabled?

[skau, 2016-07-12 19:52:25]

On 2016/07/12 19:22:03, Lei Zhang wrote:
> Never got an email for this CL until today. Did you forget to add reviewers
> earlier?
> 
> Basic printing on other platforms imply there's a native system print dialog.
> Does one exist on ChromeOS yet? Can you remind me what the difference is with
> this CL when CUPS is enabled?

The native print dialog is still disabled, we won't have one.  But we will be
able to send jobs to printers.  The UI looks the same with the CUPS features
enabled except that configured printers will be listed.  It looks almost
identical to a Cloud Print 2.0 printer.

With this CL, jobs will actually be sent to the printing context for printing. 
Without this, the dialog just closes when you press the print button.

[Lei Zhang, 2016-07-12 20:10:17]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-07-12 20:10:20]

The CQ bit was unchecked by thestig@chromium.org

[Lei Zhang, 2016-07-12 20:10:25]

lgtm

[skau, 2016-07-12 20:10:56]

The CQ bit was checked by skau@chromium.org

[commit-bot: I haz the power, 2016-07-12 20:12:41]

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-07-12 21:07:15]

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

[commit-bot: I haz the power, 2016-07-12 21:07:16]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[skau, 2016-07-12 21:59:27]

The CQ bit was checked by skau@chromium.org

[commit-bot: I haz the power, 2016-07-12 22:01:20]

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-07-13 02:28:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-13 02:29:24]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 02:29:57]

Description was changed from

==========
Enable basic printing on ChromeOS

Rudimentary print support is being implemented for Chrome OS using CUPS.
This enables native printing on Chrome OS.  If CUPS is not enabled, no
differences are noticable.

BUG=607668
R=briannorris@chromium.org
==========

to

==========
Enable basic printing on ChromeOS

Rudimentary print support is being implemented for Chrome OS using CUPS.
This enables native printing on Chrome OS.  If CUPS is not enabled, no
differences are noticable.

BUG=607668
R=briannorris@chromium.org

Committed: https://crrev.com/687d7248d07c24b9c6a5da4e9ee5c4dd939f16e9
Cr-Commit-Position: refs/heads/master@{#404913}
==========

[commit-bot: I haz the power, 2016-07-13 02:29:58]

Patchset 3 (id:??) landed as
https://crrev.com/687d7248d07c24b9c6a5da4e9ee5c4dd939f16e9
Cr-Commit-Position: refs/heads/master@{#404913}