Fix various issues from r343263.

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

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_
 
 #include <string>
 
 #include "base/files/file_path.h"
@@ skipping 319 matching lines @@
 
   // Called when an extension print job is completed.
   // |success|: Whether the job succeeded.
   // |status|: The returned print job status. Useful for reporting a specific
   //     error.
   void OnExtensionPrintResult(bool success, const std::string& status);
 
   // Called when the DOM Distiller determines whether or not this page can
   // be distilled.
   // |distillable|: Whether or not this page can be distilled.
-  void HandleIsPageDistillableResult(bool distillable);

[Lei Zhang, 2015/08/14 05:44:35]

nit: name it consistently with everything else.

+  void OnGotPageDistillableResult(bool distillable);
+
+  // Called if the DOM Distiller failed.
+  void OnDistillerFailed();
 
   // Register/unregister from notifications of changes done to the GAIA
   // cookie.
   void RegisterForGaiaCookieChanges();
   void UnregisterForGaiaCookieChanges();
 
   // The underlying dialog object.
   scoped_refptr<ui::SelectFileDialog> select_file_dialog_;
 
   // A count of how many requests received to regenerate preview data.
@@ skipping 37 matching lines @@
 #endif
 
   // Handles requests for extension printers. Created lazily by calling
   // |EnsureExtensionPrinterHandlerSet|.
   scoped_ptr<PrinterHandler> extension_printer_handler_;
 
   // Notifies tests that want to know if the PDF has been saved. This doesn't
   // notify the test if it was a successful save, only that it was attempted.
   base::Closure pdf_file_saved_closure_;
 
-  // A print preview that is responsible for rendering the page after

[Lei Zhang, 2015/08/14 05:44:36]

A PrintPreviewDistiller != "a print preview"

-  // being processed by the DOM Distiller.
+  // The class that is responsible for rendering the page after being processed
+  // by the DOM Distiller.
   scoped_ptr<PrintPreviewDistiller> print_preview_distiller_;
 
   base::WeakPtrFactory<PrintPreviewHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintPreviewHandler);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_HANDLER_H_