Move cloud print specific files out of local_discovery

Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631
Committed: https://crrev.com/33d7a06d03a2893108ddd74e5500ebf6630d1138
Cr-Commit-Position: refs/heads/master@{#367931}

[Vitaly Buka (NO REVIEWS), 2016-01-05 04:57:02]

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

[commit-bot: I haz the power, 2016-01-05 04:57:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/1

[commit-bot: I haz the power, 2016-01-05 05:00:05]

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

[commit-bot: I haz the power, 2016-01-05 05:00:06]

Dry run: Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:17:00]

Description was changed from

==========
Move files
==========

to

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on local_discovery but it's
not part of it. So better to move under printing/cloud_print/.

BUG=551631
==========

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:17:00]

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

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:17:13]

Description was changed from

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on local_discovery but it's
not part of it. So better to move under printing/cloud_print/.

BUG=551631
==========

to

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on local_discovery but it's
not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:17:28]

Description was changed from

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on local_discovery but it's
not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

to

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:17:39]

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

[commit-bot: I haz the power, 2016-01-05 23:19:06]

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

[commit-bot: I haz the power, 2016-01-05 23:25:58]

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

[commit-bot: I haz the power, 2016-01-05 23:25:58]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Vitaly Buka (NO REVIEWS), 2016-01-05 23:28:13]

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

[commit-bot: I haz the power, 2016-01-05 23:30:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/60001

[commit-bot: I haz the power, 2016-01-05 23:52:05]

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

[commit-bot: I haz the power, 2016-01-05 23:52:07]

Dry run: Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[Vitaly Buka (NO REVIEWS), 2016-01-06 00:21:54]

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

[Aleksey Shlyapnikov, 2016-01-06 02:50:43]

lgtm

[Vitaly Buka (NO REVIEWS), 2016-01-06 04:20:16]

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

[commit-bot: I haz the power, 2016-01-06 04:20:54]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/80001

[commit-bot: I haz the power, 2016-01-06 04:33:46]

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

[commit-bot: I haz the power, 2016-01-06 04:33:47]

Dry run: Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[Vitaly Buka (NO REVIEWS), 2016-01-06 05:22:47]

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

[commit-bot: I haz the power, 2016-01-06 05:23:09]

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

[commit-bot: I haz the power, 2016-01-06 06:27:48]

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

[commit-bot: I haz the power, 2016-01-06 06:27:49]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Vitaly Buka (NO REVIEWS), 2016-01-06 18:40:38]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2016-01-06 18:40:39]

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/1553333002/#ps120001
(title: " ")

[Vitaly Buka (NO REVIEWS), 2016-01-06 18:40:43]

The CQ bit was unchecked by vitalybuka@chromium.org

[commit-bot: I haz the power, 2016-01-06 18:40:49]

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

[Vitaly Buka (NO REVIEWS), 2016-01-06 18:40:50]

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

[commit-bot: I haz the power, 2016-01-06 18:40:50]

This CL has an open dependency (Issue 1554563002 Patch 80001). Please resolve
the dependency and try again.

[Vitaly Buka (NO REVIEWS), 2016-01-06 19:19:12]

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

[commit-bot: I haz the power, 2016-01-06 19:19:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/120001

[commit-bot: I haz the power, 2016-01-06 19:35:01]

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

[commit-bot: I haz the power, 2016-01-06 19:35:03]

Dry run: Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[Vitaly Buka (NO REVIEWS), 2016-01-06 20:13:49]

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

[commit-bot: I haz the power, 2016-01-06 20:14:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/120001

[Lei Zhang, 2016-01-06 20:16:21]

lgtm, just nits:

https://codereview.chromium.org/1553333002/diff/120001/chrome/browser/printin...
File chrome/browser/printing/pwg_raster_converter.h (right):

https://codereview.chromium.org/1553333002/diff/120001/chrome/browser/printin...
chrome/browser/printing/pwg_raster_converter.h:44: static
printing::PdfRenderSettings GetConversionSettings(
no need for printing:: inside the printing namespace.

https://codereview.chromium.org/1553333002/diff/120001/chrome/chrome_browser....
File chrome/chrome_browser.gypi (right):

https://codereview.chromium.org/1553333002/diff/120001/chrome/chrome_browser....
chrome/chrome_browser.gypi:2675: 'browser/local_discovery/endpoint_resolver.cc',
ordering

[commit-bot: I haz the power, 2016-01-06 20:21:33]

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

[commit-bot: I haz the power, 2016-01-06 20:21:40]

Dry run: This issue passed the CQ dry run.

[Vitaly Buka (NO REVIEWS), 2016-01-06 21:26:21]

https://codereview.chromium.org/1553333002/diff/120001/chrome/browser/printin...
File chrome/browser/printing/pwg_raster_converter.h (right):

https://codereview.chromium.org/1553333002/diff/120001/chrome/browser/printin...
chrome/browser/printing/pwg_raster_converter.h:44: static
printing::PdfRenderSettings GetConversionSettings(
On 2016/01/06 20:16:21, Lei Zhang wrote:
> no need for printing:: inside the printing namespace.

Done.

https://codereview.chromium.org/1553333002/diff/120001/chrome/chrome_browser....
File chrome/chrome_browser.gypi (right):

https://codereview.chromium.org/1553333002/diff/120001/chrome/chrome_browser....
chrome/chrome_browser.gypi:2675: 'browser/local_discovery/endpoint_resolver.cc',
On 2016/01/06 20:16:21, Lei Zhang wrote:
> ordering

Done.

[Vitaly Buka (NO REVIEWS), 2016-01-06 21:26:27]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2016-01-06 21:26:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, alekseys@chromium.org
Link to the patchset: https://codereview.chromium.org/1553333002/#ps160001
(title: " ")

[commit-bot: I haz the power, 2016-01-06 21:27:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1553333002/160001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1553333002/160001

[commit-bot: I haz the power, 2016-01-06 22:11:48]

Description was changed from

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

to

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

[commit-bot: I haz the power, 2016-01-06 22:11:51]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-01-06 22:12:50]

Description was changed from

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631
==========

to

==========
Move cloud print specific files out of local_discovery

Privet printing is part of GCP2.0 spec. It depends on
local_discovery but it's not part of it. So better to
move under printing/cloud_print/.

BUG=551631

Committed: https://crrev.com/33d7a06d03a2893108ddd74e5500ebf6630d1138
Cr-Commit-Position: refs/heads/master@{#367931}
==========

[commit-bot: I haz the power, 2016-01-06 22:12:52]

Patchset 9 (id:??) landed as
https://crrev.com/33d7a06d03a2893108ddd74e5500ebf6630d1138
Cr-Commit-Position: refs/heads/master@{#367931}