Refactor webview to use //components/printing

components/printing/renderer/print_web_view_helper.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/printing/renderer/print_web_view_helper.h"
 
 #include <string>
 
 #include "base/auto_reset.h"
 #include "base/json/json_writer.h"
@@ skipping 749 matching lines @@
     }
   }
   frame_.Reset(NULL);
   on_ready_.Reset();
 }
 
 PrintWebViewHelper::PrintWebViewHelper(
     content::RenderView* render_view,
     bool out_of_process_pdf_enabled,
     bool print_preview_disabled,
+    bool should_ask_print_settings,
     scoped_ptr<Delegate> delegate)
     : content::RenderViewObserver(render_view),
       content::RenderViewObserverTracker<PrintWebViewHelper>(render_view),
       reset_prep_frame_view_(false),
       is_print_ready_metafile_sent_(false),
       ignore_css_margins_(false),
       is_scripted_printing_blocked_(false),
       notify_browser_of_print_failure_(true),
       print_for_preview_(false),
       out_of_process_pdf_enabled_(out_of_process_pdf_enabled),
+      should_ask_print_settings_(should_ask_print_settings),
       delegate_(delegate.Pass()),
       print_node_in_progress_(false),
       is_loading_(false),
       is_scripted_preview_delayed_(false),
       weak_ptr_factory_(this) {
   if (print_preview_disabled)
     DisablePreview();
 }
 
 PrintWebViewHelper::~PrintWebViewHelper() {}
@@ skipping 468 matching lines @@
     return;  // Failed to init print page settings.
   }
 
   // Some full screen plugins can say they don't want to print.
   if (!expected_page_count) {
     DidFinishPrinting(FAIL_PRINT);
     return;
   }
 
   // Ask the browser to show UI to retrieve the final print settings.
-  if (!GetPrintSettingsFromUser(frame_ref.GetFrame(), node,
+  if (should_ask_print_settings_ &&
+      !GetPrintSettingsFromUser(frame_ref.GetFrame(), node,

[dgn, 2015/01/15 19:28:48]

This block is disabled with a #if 0 in webview. I did't see a special flag used
in the code to check for webview builds. Would finding/adding something to ifdef
this out be better that such a boolean?

[sgurun-gerrit only, 2015/01/15 19:39:32]

I think this logic is chrome only. Webview cannot get print settings from user
since it does not have a UI.  we will need to use a delegate for this, which
webview will not implement.

                                 expected_page_count,
                                 is_scripted)) {
     DidFinishPrinting(OK);  // Release resources and fail silently.
     return;
   }
 
   // Render Pages for printing.
   if (!RenderPagesForPrint(frame_ref.GetFrame(), node)) {
     LOG(ERROR) << "RenderPagesForPrint failed";
     DidFinishPrinting(FAIL_PRINT);
@@ skipping 749 matching lines @@
       blink::WebConsoleMessage::LevelWarning, message));
   return false;
 }
 
 void PrintWebViewHelper::ScriptingThrottler::Reset() {
   // Reset counter on successful print.
   count_ = 0;
 }
 
 }  // namespace printing