Tidy fpdfsave.cpp

core/include/fpdfapi/fpdf_parser.h

Index: core/include/fpdfapi/fpdf_parser.h
diff --git a/core/include/fpdfapi/fpdf_parser.h b/core/include/fpdfapi/fpdf_parser.h
index 6620a71e2e6b371387fa14e01db10107b5b5b8fd..18a942e4e882b31dbd08a0b4266b451ad6680f5b 100644
--- a/core/include/fpdfapi/fpdf_parser.h
+++ b/core/include/fpdfapi/fpdf_parser.h
@@ -12,7 +12,7 @@
 #include <set>
 
 #include "core/include/fpdfapi/fpdf_objects.h"
-#include "core/include/fxcrt/fx_system.h"
+#include "core/include/fxcrt/fx_basic.h"
 
 class CFX_Font;
 class CFX_Matrix;
@@ -28,7 +28,6 @@ class CPDF_Object;
 class CPDF_Parser;
 class CPDF_Pattern;
 class CPDF_StandardSecurityHandler;
-class IFX_FileRead;
 class IPDF_SecurityHandler;
 
 #define FPDFPERM_PRINT 0x0004
@@ -41,6 +40,11 @@ class IPDF_SecurityHandler;
 #define FPDFPERM_PRINT_HIGH 0x0800
 #define FPDF_PAGE_MAX_NUM 0xFFFFF
 
+// TODO(thestig) Using unique_ptr with ReleaseDeleter is still not ideal.

[Lei Zhang, 2016/02/05 23:22:43]

Shall we move this to fx_stream.h where IFX_FileStream lives?

+// Come up or wait for something better.
+using ScopedFileStream =
+    std::unique_ptr<IFX_FileStream, ReleaseDeleter<IFX_FileStream>>;
+
 // Use the accessors below instead of directly accessing PDF_CharType.
 extern const char PDF_CharType[256];