Fix loading page using hint tables.

core/fpdfapi/parser/cpdf_document.cpp

Index: core/fpdfapi/parser/cpdf_document.cpp
diff --git a/core/fpdfapi/parser/cpdf_document.cpp b/core/fpdfapi/parser/cpdf_document.cpp
index c5f64a790ca1d31f6cb4357f5a01b18d6b0585e1..5f64c14d3113de5de19ae353b0adb5649be290a9 100644
--- a/core/fpdfapi/parser/cpdf_document.cpp
+++ b/core/fpdfapi/parser/cpdf_document.cpp
@@ -425,7 +425,9 @@ CPDF_Document::CPDF_Document(std::unique_ptr<CPDF_Parser> pParser)
 
 CPDF_Document::~CPDF_Document() {
   delete m_pDocPage;
-  CPDF_ModuleMgr::Get()->GetPageModule()->ClearStockFont(this);
+  if (CPDF_ModuleMgr::Get()->GetPageModule()) {

[Lei Zhang, 2016/10/20 21:18:43]

Is this to work around the crash in the unit test? Given that all the
GetPageModule() calls assume it won't return a nullptr, we could have changed
the unit test to act more like the real code, rather than changing the real code
to accommodate the unit test.

[Lei Zhang, 2016/10/20 21:43:36]

See fpdfsdk/fpdfdoc_unittest.cpp for an example of how to do this the other way.

+    CPDF_ModuleMgr::Get()->GetPageModule()->ClearStockFont(this);
+  }
   m_pByteStringPool.DeleteObject();  // Make weak.
 }