Delegates for the printing component

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 79bc401e1934d29c26f97a5724ef645b89341508..319828bdd00ed7ea4ff90b4df992fb11de54fba6 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -49,6 +49,7 @@
 #include "chrome/renderer/prerender/prerender_media_load_deferrer.h"
 #include "chrome/renderer/prerender/prerenderer_client.h"
 #include "chrome/renderer/principals_extension_bindings.h"
+#include "chrome/renderer/printing/chrome_print_web_view_helper_delegate.h"
 #include "chrome/renderer/printing/print_web_view_helper.h"
 #include "chrome/renderer/safe_browsing/malware_dom_details.h"
 #include "chrome/renderer/safe_browsing/phishing_classifier_delegate.h"
@@ -506,7 +507,9 @@ void ChromeContentRendererClient::RenderViewCreated(
 #endif
   new PageLoadHistograms(render_view);
 #if defined(ENABLE_PRINTING)
-  new printing::PrintWebViewHelper(render_view);
+  new printing::PrintWebViewHelper(render_view,
+      scoped_ptr<printing::ChromePrintWebViewHelperDelegate>(

[dgn, 2014/12/30 17:15:48]

I was wondering here why the delegates are all owned by
ChromeContentRendererClient. Isn't it more reliable to have them owned by the
delegate? That way at least we know the object is always there instead of just
assuming it?

+          new printing::ChromePrintWebViewHelperDelegate()));
 #endif
 #if defined(ENABLE_SPELLCHECK)
   new SpellCheckProvider(render_view, spellcheck_.get());