Defer request to print a PDF when the user initiates the entire frame and the PDF hasn't loaded.

chrome/renderer/pepper/ppb_pdf_impl.cc

Index: chrome/renderer/pepper/ppb_pdf_impl.cc
diff --git a/chrome/renderer/pepper/ppb_pdf_impl.cc b/chrome/renderer/pepper/ppb_pdf_impl.cc
index 16317adf2e728d154e580f2e3bfacaf11ca4c252..b27718545e733376d17338234924610ad3cca859 100644
--- a/chrome/renderer/pepper/ppb_pdf_impl.cc
+++ b/chrome/renderer/pepper/ppb_pdf_impl.cc
@@ -460,5 +460,10 @@ void PPB_PDF_Impl::InvokePrintingForInstance(PP_Instance instance_id) {
   printing::PrintWebViewHelper* helper = GetPrintWebViewHelper(element);
   if (helper)
     helper->PrintNode(element);
+
+  // DidStopLoading() is not call anywhere. Calling it here fixes a problem
+  // where PrintWebViewHelper() is waiting for DidStopLoading to be called to
+  // progress the preview process.
+  DidStopLoading(instance_id);

[ivandavid, 2014/08/14 03:38:13]

Is this an appropriate place to place DidStopLoading()? It fixes the sheets
problem, however it might not be a good idea to put it here. I tried looking for
references to DidStartLoading, but I couldn't really find one.

[Vitaly Buka (NO REVIEWS), 2014/08/14 05:19:43]

It does not look like a right place
let's remove this and _NODE related change and fix that in a separate CL

On 2014/08/14 03:38:13, ivandavid wrote:

 #endif  // ENABLE_FULL_PRINTING
 }