Revert of clean up printing::Image and printing::Metafile

Revert of clean up printing::Image and printing::Metafile (patchset #7 id:140001 of https://codereview.chromium.org/2812263002/ )

Reason for revert:
Reverting so https://crrev.com/463828 can be cleanly reverted.

r463828 is likely causing https://crbug.com/712309



Original issue's description:
> clean up printing::Image and printing::Metafile
> 
> printing::Image now takes raw data, rather than a metafile. this removes
> the need for a virtual on printing::Metafile, Playback().
> printing::Image gets move constructor
> printing::Image loses dead code.
> 
> delete dead code: chrome/browser/printing/printing_layout_browsertest.cc
> 
> ServiceUtilityProcessHost takes metafile data, not a MetafilePlayer.  This will
> help remove printing::Metafile::SafePlayback().
> 
> printing::PdfMetafileCg::OnRenderPage is added to copy data fewer times.
> 
> Review-Url: https://codereview.chromium.org/2812263002
> Cr-Commit-Position: refs/heads/master@{#464714}
> Committed: https://chromium.googlesource.com/chromium/src/+/574b10750dfea3d1103946615a1c8a4a9bff3b5c

TBR=halcanary@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.

Review-Url: https://codereview.chromium.org/2835193007
Cr-Commit-Position: refs/heads/master@{#467473}
Committed: https://chromium.googlesource.com/chromium/src/+/3109df1dc485aa29d6dba54f57a5762fd8957830

[Lei Zhang, 2017-04-26 20:36:19]

Created Revert of clean up printing::Image and printing::Metafile

[Lei Zhang, 2017-04-26 20:36:32]

Description was changed from

==========
Revert of clean up printing::Image and printing::Metafile (patchset #7 id:140001
of https://codereview.chromium.org/2812263002/ )

Reason for revert:
Reverting so https://crrev.com/463828 can be cleanly reverted.

r463828 is likely causing https://crbug.com/712309



Original issue's description:
> clean up printing::Image and printing::Metafile
> 
> printing::Image now takes raw data, rather than a metafile. this removes
> the need for a virtual on printing::Metafile, Playback().
> printing::Image gets move constructor
> printing::Image loses dead code.
> 
> delete dead code: chrome/browser/printing/printing_layout_browsertest.cc
> 
> ServiceUtilityProcessHost takes metafile data, not a MetafilePlayer.  This
will
> help remove printing::Metafile::SafePlayback().
> 
> printing::PdfMetafileCg::OnRenderPage is added to copy data fewer times.
> 
> Review-Url: https://codereview.chromium.org/2812263002
> Cr-Commit-Position: refs/heads/master@{#464714}
> Committed:
https://chromium.googlesource.com/chromium/src/+/574b10750dfea3d1103946615a1c...

TBR=halcanary@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of clean up printing::Image and printing::Metafile (patchset #7 id:140001
of https://codereview.chromium.org/2812263002/ )

Reason for revert:
Reverting so https://crrev.com/463828 can be cleanly reverted.

r463828 is likely causing https://crbug.com/712309



Original issue's description:
> clean up printing::Image and printing::Metafile
> 
> printing::Image now takes raw data, rather than a metafile. this removes
> the need for a virtual on printing::Metafile, Playback().
> printing::Image gets move constructor
> printing::Image loses dead code.
> 
> delete dead code: chrome/browser/printing/printing_layout_browsertest.cc
> 
> ServiceUtilityProcessHost takes metafile data, not a MetafilePlayer.  This
will
> help remove printing::Metafile::SafePlayback().
> 
> printing::PdfMetafileCg::OnRenderPage is added to copy data fewer times.
> 
> Review-Url: https://codereview.chromium.org/2812263002
> Cr-Commit-Position: refs/heads/master@{#464714}
> Committed:
https://chromium.googlesource.com/chromium/src/+/574b10750dfea3d1103946615a1c...

TBR=halcanary@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[Lei Zhang, 2017-04-26 20:36:41]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-04-26 20:37:14]

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-04-26 21:57:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493239001020470, "parent_rev":
"f264f1a41214087a86a32e5d5497650fcf40cb4b", "commit_rev":
"3109df1dc485aa29d6dba54f57a5762fd8957830"}

[commit-bot: I haz the power, 2017-04-26 21:57:39]

Description was changed from

==========
Revert of clean up printing::Image and printing::Metafile (patchset #7 id:140001
of https://codereview.chromium.org/2812263002/ )

Reason for revert:
Reverting so https://crrev.com/463828 can be cleanly reverted.

r463828 is likely causing https://crbug.com/712309



Original issue's description:
> clean up printing::Image and printing::Metafile
> 
> printing::Image now takes raw data, rather than a metafile. this removes
> the need for a virtual on printing::Metafile, Playback().
> printing::Image gets move constructor
> printing::Image loses dead code.
> 
> delete dead code: chrome/browser/printing/printing_layout_browsertest.cc
> 
> ServiceUtilityProcessHost takes metafile data, not a MetafilePlayer.  This
will
> help remove printing::Metafile::SafePlayback().
> 
> printing::PdfMetafileCg::OnRenderPage is added to copy data fewer times.
> 
> Review-Url: https://codereview.chromium.org/2812263002
> Cr-Commit-Position: refs/heads/master@{#464714}
> Committed:
https://chromium.googlesource.com/chromium/src/+/574b10750dfea3d1103946615a1c...

TBR=halcanary@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of clean up printing::Image and printing::Metafile (patchset #7 id:140001
of https://codereview.chromium.org/2812263002/ )

Reason for revert:
Reverting so https://crrev.com/463828 can be cleanly reverted.

r463828 is likely causing https://crbug.com/712309



Original issue's description:
> clean up printing::Image and printing::Metafile
> 
> printing::Image now takes raw data, rather than a metafile. this removes
> the need for a virtual on printing::Metafile, Playback().
> printing::Image gets move constructor
> printing::Image loses dead code.
> 
> delete dead code: chrome/browser/printing/printing_layout_browsertest.cc
> 
> ServiceUtilityProcessHost takes metafile data, not a MetafilePlayer.  This
will
> help remove printing::Metafile::SafePlayback().
> 
> printing::PdfMetafileCg::OnRenderPage is added to copy data fewer times.
> 
> Review-Url: https://codereview.chromium.org/2812263002
> Cr-Commit-Position: refs/heads/master@{#464714}
> Committed:
https://chromium.googlesource.com/chromium/src/+/574b10750dfea3d1103946615a1c...

TBR=halcanary@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.

Review-Url: https://codereview.chromium.org/2835193007
Cr-Commit-Position: refs/heads/master@{#467473}
Committed:
https://chromium.googlesource.com/chromium/src/+/3109df1dc485aa29d6dba54f57a5...
==========

[commit-bot: I haz the power, 2017-04-26 21:57:41]

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