Fix various issues from r343263.

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

Index: chrome/browser/ui/webui/print_preview/print_preview_ui.h
diff --git a/chrome/browser/ui/webui/print_preview/print_preview_ui.h b/chrome/browser/ui/webui/print_preview/print_preview_ui.h
index 0ec43136922ecf3cc9045aa8ab62f94644924af7..9ae4576bade4501d5795977c13218896a797e372 100644
--- a/chrome/browser/ui/webui/print_preview/print_preview_ui.h
+++ b/chrome/browser/ui/webui/print_preview/print_preview_ui.h
@@ -10,7 +10,6 @@
 #include "base/callback_forward.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
-#include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 #include "chrome/browser/ui/webui/constrained_web_dialog_ui.h"
 
@@ -167,8 +166,6 @@ class PrintPreviewUI : public ConstrainedWebDialogUI {
   // Passes |closure| to PrintPreviewHandler::SetPdfSavedClosureForTesting().
   void SetPdfSavedClosureForTesting(const base::Closure& closure);
 
-  base::WeakPtr<PrintPreviewUI> GetWeakPtr();

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

You should not create a WeakPtrFactory and publicly hand out WeakPtrs. As seen
in the WeakPtrFactory comments. "This is helpful if you only need weak pointers
within the implementation of a class."

Instead, this class should have subclassed from 
SupportsWeakPtr, if you really needed WeakPtrs for PrintPreviewUI, but it turns
out you don't.

-
  private:
   friend class PrintPreviewHandlerTest;
   FRIEND_TEST_ALL_PREFIXES(PrintPreviewHandlerTest, StickyMarginsCustom);
@@ -210,8 +207,6 @@ class PrintPreviewUI : public ConstrainedWebDialogUI {
   // Keeps track of whether OnClosePrintPreviewDialog() has been called or not.
   bool dialog_closed_;
 
-  base::WeakPtrFactory<PrintPreviewUI> weak_ptr_factory_;
-
   DISALLOW_COPY_AND_ASSIGN(PrintPreviewUI);
 };