Cleanup: Slight simplification in the PDFiumEngine dtor.

The functions being called already checks for NULL input. Review URL: https://codereview.chromium.org/1131093004 Cr-Commit-Position: refs/heads/master@{#330214}
Denger-Network · May 15, 2015 · 1626f00 · 1626f00
1 parent 1cf60af
commit 1626f00
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/pdf/pdfium/pdfium_engine.cc b/pdf/pdfium/pdfium_engine.cc
@@ -661,17 +661,11 @@ PDFiumEngine::~PDFiumEngine() {
     pages_[i]->Unload();
 
   if (doc_) {
-    if (form_) {
-      FORM_DoDocumentAAction(form_, FPDFDOC_AACTION_WC);
-    }
+    FORM_DoDocumentAAction(form_, FPDFDOC_AACTION_WC);
     FPDF_CloseDocument(doc_);
-    if (form_) {
-      FPDFDOC_ExitFormFillEnvironment(form_);
-    }
+    FPDFDOC_ExitFormFillEnvironment(form_);
   }
-
-  if (fpdf_availability_)
-    FPDFAvail_Destroy(fpdf_availability_);
+  FPDFAvail_Destroy(fpdf_availability_);
 
   STLDeleteElements(&pages_);
 }