Fix memory leak of PreviewModeClient in PDF

pdf/out_of_process_instance.h

Index: pdf/out_of_process_instance.h
diff --git a/pdf/out_of_process_instance.h b/pdf/out_of_process_instance.h
index 1215f192bcf34c8c94220a8bcf9944676bcd8842..7e8a7c634a1102906fc5d826ee22c2cc6df8dbeb 100644
--- a/pdf/out_of_process_instance.h
+++ b/pdf/out_of_process_instance.h
@@ -261,6 +261,7 @@ class OutOfProcessInstance : public pp::Instance,
   // used only in print preview mode. This will use |PreviewModeClient|
   // interface which has very limited access to the pp::Instance.
   scoped_ptr<PDFEngine> preview_engine_;
+  scoped_ptr<PreviewModeClient> preview_client_;

[Lei Zhang, 2015/09/07 19:31:46]

Doesn't member destruction go bottom up? i.e. as is, |preview_client_| will be
freed before |preview_engine_|, though |preview_engine_| still has a pointer to
|preview_client_|.

[raymes, 2015/09/07 23:59:00]

Done.

 
   std::string url_;