Support linearized loading

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 04c7d90f6fc067cae49787f340f27f41ecd0840b..17429b431fc558d62f11b35e427486f103bdf19c 100644
--- a/core/include/fpdfapi/fpdf_parser.h
+++ b/core/include/fpdfapi/fpdf_parser.h
@@ -7,33 +7,36 @@
 #ifndef CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_
 #define CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_
 
+#include "../../../public/fpdf_macro.h"

[Tom Sepez, 2015/10/14 16:57:45]

this should just include public/fpdf_datavail.h, no need for the new file.

[jun_fang, 2015/10/15 10:22:28]

Pdfium has two parts, sdk and core. sdk has the dependency on core so sdk
includes header files from core. In contrast, core doesn't 
include any sdk header files in order not to break the rule.
For marcos used by sdk and core, currently they are defined twice.
One is defined in public for applications to use. The other is
defined in core. 

public/fpdf_datavail.h declares some APIs related to linearization. 
Usually, it's included by customer's applications to implement
linearization functionality. The purpose of including public/fpdf_datavail.h in
core/include/fpdfapi/fpdf_parser.h is only to use macros. It's not worthy to
include whole header file. I want to define all marcos used by applications, sdk
and core in this new head file to make design clear. In this way, we don't need
to define
macros twice. We don't include the whole header file if only macros
will be used.

[Tom Sepez, 2015/10/15 16:20:04]

Nah, it's much better to keep the declarations near where they are returned to
make thing more understandable for the embedders, even at the expense of
processing some additional lines when including the whole header.

As to the the core/fpdfsdk split, if you want to be really strict about
layering, then you'd have to have two definitions, with static_asserts() to
ensure they line up, along with a comment about layering.  Maybe its better to
go back to that model.

On the other hand, we already violate this, I'd presumed that public
can be included at any level, which is part of the reason I moved it from
fpdfsdk/include.

$ git grep public/ core
core/include/fpdfapi/fpdf_parser.h:#include "../../../public/fpdfview.h"
core/include/fpdfapi/fpdf_render.h:#include "../../../public/fpdf_progressive.h"
core/src/fpdfapi/fpdf_parser/fpdf_parser_decode_embeddertest.cpp:#include
"../../../../public/fpdfview.h"

[jun_fang, 2015/10/19 13:20:30]

Acknowledged.

 #include "../fxcrt/fx_system.h"
 #include "fpdf_objects.h"
 
-class CPDF_Document;
-class CPDF_Parser;
-class CPDF_SecurityHandler;
-class CPDF_StandardSecurityHandler;
-class CPDF_CryptoHandler;
-class CPDF_Object;
-class IFX_FileRead;
 class CFDF_Document;
 class CFDF_Parser;
-class CFX_Font;
 class CFX_AffineMatrix;
+class CFX_DIBSource;
 class CFX_FloatRect;
-class CPDF_Point;
+class CFX_Font;
+class CFX_PrivateData;
+class CPDF_ColorSpace;
+class CPDF_CryptoHandler;
 class CPDF_DocPageData;
 class CPDF_DocRenderData;
-class CPDF_ModuleMgr;
-class CFX_DIBSource;
+class CPDF_Document;
 class CPDF_Font;
-class CPDF_Image;
-class CPDF_ColorSpace;
-class CPDF_Pattern;
 class CPDF_FontEncoding;
+class CPDF_HintTables;
 class CPDF_IccProfile;
-class CFX_PrivateData;
+class CPDF_Image;
+class CPDF_ModuleMgr;
+class CPDF_Object;
+class CPDF_Parser;
+class CPDF_Pattern;
+class CPDF_Point;
+class CPDF_SecurityHandler;
+class CPDF_StandardSecurityHandler;
+class IFX_FileRead;
+
 #define FPDFPERM_PRINT 0x0004
 #define FPDFPERM_MODIFY 0x0008
 #define FPDFPERM_EXTRACT 0x0010
@@ -845,12 +848,7 @@ class IFX_DownloadHints {
   virtual ~IFX_DownloadHints() {}
   virtual void AddSegment(FX_FILESIZE offset, FX_DWORD size) = 0;
 };
-#define PDF_IS_LINEARIZED 1
-#define PDF_NOT_LINEARIZED 0
-#define PDF_UNKNOW_LINEARIZED -1
-#define PDFFORM_NOTAVAIL 0
-#define PDFFORM_AVAIL 1
-#define PDFFORM_NOTEXIST 2
+
 class IPDF_DataAvail {
  public:
   static IPDF_DataAvail* Create(IFX_FileAvail* pFileAvail,
@@ -907,6 +905,7 @@ enum PDF_DATAAVAIL_STATUS {
   PDF_DATAAVAIL_HEADER = 0,
   PDF_DATAAVAIL_FIRSTPAGE,
   PDF_DATAAVAIL_FIRSTPAGE_PREPARE,
+  PDF_DATAAVAIL_HINTTABLE,
   PDF_DATAAVAIL_END,
   PDF_DATAAVAIL_CROSSREF,
   PDF_DATAAVAIL_CROSSREF_ITEM,