Support loading complete linearized PDF.

pdf/pdfium/pdfium_engine.cc

Index: pdf/pdfium/pdfium_engine.cc
diff --git a/pdf/pdfium/pdfium_engine.cc b/pdf/pdfium/pdfium_engine.cc
index 0528c94ecca007a1e58ec053c96689974027318d..4eb12d4c02091515e2bcfeead7a7c18130e77d96 100644
--- a/pdf/pdfium/pdfium_engine.cc
+++ b/pdf/pdfium/pdfium_engine.cc
@@ -1156,6 +1156,12 @@ void PDFiumEngine::OnDocumentComplete() {
     return;
   }
 
+  FinishLoadingDocument();
+}
+
+void PDFiumEngine::FinishLoadingDocument() {
+  DCHECK(doc_loader_.IsDocumentComplete() && doc_);
+
   bool need_update = false;
   for (size_t i = 0; i < pages_.size(); ++i) {
     if (pages_[i]->available())
@@ -1172,11 +1178,6 @@ void PDFiumEngine::OnDocumentComplete() {
   if (need_update)
     LoadPageInfo(true);
 
-  FinishLoadingDocument();
-}
-
-void PDFiumEngine::FinishLoadingDocument() {
-  DCHECK(doc_loader_.IsDocumentComplete() && doc_);
   if (called_do_document_action_)
     return;
   called_do_document_action_ = true;
@@ -2667,7 +2668,10 @@ void PDFiumEngine::LoadPageInfo(bool reload) {
     if (reload) {
       pages_[i]->set_rect(page_rect);
     } else {
-      pages_.push_back(new PDFiumPage(this, i, page_rect, doc_complete));
+      // The page is marked as not being available even if doc_complete is true

[Lei Zhang, 2015/12/28 23:55:02]

Please refer to variables are |var_name|.

[spelchat, 2015/12/29 00:04:03]

Done.

+      // because FPDFAvail_IsPageAvail still has to be called for this page,

[Lei Zhang, 2015/12/28 23:55:02]

If you refer to functions as FuncName(), then it's clear to readers that you are
referring to a function and not a class. Readers may not be familiar with the
functions you are referring to.

[spelchat, 2015/12/29 00:04:03]

Done.

+      // which will be done in FinishLoadingDocument.
+      pages_.push_back(new PDFiumPage(this, i, page_rect, false));
     }
   }