Include user email into document title before spooling

Include user email into document title before spooling

Could be used to monitor print jobs on OS side.
Windows spooler "owner" cannot be changed so we should modify title.
Max title length increased from 50 to 80 better readability.

BUG=504826
Committed: https://crrev.com/27e401c8a69340bbc6cfbae460eeae7646e2dbbf
Cr-Commit-Position: refs/heads/master@{#337647}

[Vitaly Buka (NO REVIEWS), 2015-07-02 22:04:08]

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

[Vitaly Buka (NO REVIEWS), 2015-07-02 22:07:38]

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

[Vitaly Buka (NO REVIEWS), 2015-07-03 01:27:24]

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

[commit-bot: I haz the power, 2015-07-03 01:27:32]

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

[commit-bot: I haz the power, 2015-07-03 01:53:23]

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

[commit-bot: I haz the power, 2015-07-03 01:53:24]

Dry run: Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

(exceeded global retry quota)

[Vitaly Buka (NO REVIEWS), 2015-07-06 02:14:11]

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

[commit-bot: I haz the power, 2015-07-06 02:14:16]

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

[commit-bot: I haz the power, 2015-07-06 02:43:42]

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

[commit-bot: I haz the power, 2015-07-06 02:43:43]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

(exceeded global retry quota)

[Vitaly Buka (NO REVIEWS), 2015-07-06 17:14:27]

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

[commit-bot: I haz the power, 2015-07-06 17:14:53]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1212883003/40001

[commit-bot: I haz the power, 2015-07-06 19:00:34]

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

[commit-bot: I haz the power, 2015-07-06 19:00:35]

Dry run: This issue passed the CQ dry run.

[Aleksey Shlyapnikov, 2015-07-06 23:06:25]

https://codereview.chromium.org/1212883003/diff/40001/chrome/service/cloud_pr...
File chrome/service/cloud_print/printer_job_handler.cc (right):

https://codereview.chromium.org/1212883003/diff/40001/chrome/service/cloud_pr...
chrome/service/cloud_print/printer_job_handler.cc:796: if
(!job_details.job_owner_.empty()) {
What if it is empty? We still need to call printing::SimplifyDocumentTitle,
don't we?

https://codereview.chromium.org/1212883003/diff/40001/printing/printing_utils.h
File printing/printing_utils.h (right):

https://codereview.chromium.org/1212883003/diff/40001/printing/printing_utils...
printing/printing_utils.h:21: size_t desi);
desi -> length

[Vitaly Buka (NO REVIEWS), 2015-07-07 01:22:50]

https://codereview.chromium.org/1212883003/diff/40001/chrome/service/cloud_pr...
File chrome/service/cloud_print/printer_job_handler.cc (right):

https://codereview.chromium.org/1212883003/diff/40001/chrome/service/cloud_pr...
chrome/service/cloud_print/printer_job_handler.cc:796: if
(!job_details.job_owner_.empty()) {
On 2015/07/06 23:06:25, Aleksey Shlyapnikov wrote:
> What if it is empty? We still need to call printing::SimplifyDocumentTitle,
> don't we?

Re-done.

[Vitaly Buka (NO REVIEWS), 2015-07-07 01:22:56]

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

[commit-bot: I haz the power, 2015-07-07 01:23:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1212883003/100001

[Aleksey Shlyapnikov, 2015-07-07 01:29:22]

lgtm

[Vitaly Buka (NO REVIEWS), 2015-07-07 01:29:45]

The CQ bit was checked by vitalybuka@chromium.org

[commit-bot: I haz the power, 2015-07-07 01:30:08]

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

[commit-bot: I haz the power, 2015-07-07 01:40:15]

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

[commit-bot: I haz the power, 2015-07-07 01:40:16]

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

(exceeded global retry quota)

[Vitaly Buka (NO REVIEWS), 2015-07-07 17:19:34]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-07-07 17:19:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
alekseys@chromium.org
Link to the patchset: https://codereview.chromium.org/1212883003/#ps140001
(title: "Tue Jul  7 10:19:02 PDT 2015")

[commit-bot: I haz the power, 2015-07-07 17:20:54]

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

[commit-bot: I haz the power, 2015-07-07 18:21:26]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-07-07 18:22:14]

Patchset 8 (id:??) landed as
https://crrev.com/27e401c8a69340bbc6cfbae460eeae7646e2dbbf
Cr-Commit-Position: refs/heads/master@{#337647}