Refactor webview to use //components/printing

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 356f7af0751fd310b6ae8ef434f386299cbe1b46..bd053e79107057debb1af25674d5320d56257919 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -509,6 +509,7 @@ void ChromeContentRendererClient::RenderViewCreated(
       render_view,
       switches::OutOfProcessPdfEnabled(),
       command_line->HasSwitch(switches::kDisablePrintPreview),
+      true /* should_ask_print_settings */,
       scoped_ptr<printing::PrintWebViewHelper::Delegate>(
           new ChromePrintWebViewHelperDelegate()));

[sgurun-gerrit only, 2015/01/16 21:22:04]

don't you need to use Pass here?

[Bernhard Bauer, 2015/01/19 10:05:25]

Not if the scoped_ptr is directly constructed. You only need .Pass() if someone
already has ownership, like an existing scoped_ptr variable.

[sgurun-gerrit only, 2015/01/19 18:21:35]

You are right, my bad.

 #endif