Remove bad CallJavascriptFunctionUnsafe call from PrintPreviewUI

chrome/browser/ui/webui/print_preview/print_preview_handler.h

Index: chrome/browser/ui/webui/print_preview/print_preview_handler.h
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_handler.h b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
index 34725e3ab04c4df43c1687b63a2bc422dd10ec5f..c79dd4b690db72ef36ba1af969b7cf8177015713 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_handler.h
+++ b/chrome/browser/ui/webui/print_preview/print_preview_handler.h
@@ -116,6 +116,9 @@ class PrintPreviewHandler
   // Sets |pdf_file_saved_closure_| to |closure|.
   void SetPdfSavedClosureForTesting(const base::Closure& closure);
 
+  // Handles the request to cancel the pending print request. |args| is unused.
+  void HandleCancelPendingPrintRequest(const base::ListValue* args);

[Lei Zhang, 2017/07/11 22:34:20]

Can we keep this methods protected, and add a new OnPrintRequestCancelled()
method. Ya, that would just call HandleCancelPendingPrintRequest(), which seems
silly, but it keeps up the pattern of only exposing OnFoo methods.

[rbpotter, 2017/07/12 00:26:35]

Done.

+
  protected:
   // If |prompt_user| is true, starts a task to create the default Save As PDF
   // directory if needed. OnDirectoryCreated() will be called when it
@@ -167,9 +170,6 @@ class PrintPreviewHandler
   // |args| is unused.
   void HandleHidePreview(const base::ListValue* args);
 
-  // Handles the request to cancel the pending print request. |args| is unused.
-  void HandleCancelPendingPrintRequest(const base::ListValue* args);
-
   // Handles a request to store data that the web ui wishes to persist.
   // First element of |args| is the data to persist.
   void HandleSaveAppState(const base::ListValue* args);