diff options
| author | tsepez <tsepez@chromium.org> | 2016-05-19 15:55:11 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-19 15:55:11 -0700 |
| commit | 39faf4fde606fa3b71d2b6b232cff3e86cb1303f (patch) | |
| tree | b300d80918177d7f4dd5e4fd40de37f78f6bc574 /fpdfsdk/include | |
| parent | 9b1a0ee2a8f24411609a2f7554119597950dbd04 (diff) | |
| download | pdfium-39faf4fde606fa3b71d2b6b232cff3e86cb1303f.tar.xz | |
Remove Release() from CPDFDOC_Environment
It's never called. But looking at the comments in fpdf_formfill.h for
the Release callback, it seems like it should always be called as part
of the dtor.
BUG=
Review-Url: https://codereview.chromium.org/1995753004
Diffstat (limited to 'fpdfsdk/include')
| -rw-r--r-- | fpdfsdk/include/fsdk_mgr.h | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/fpdfsdk/include/fsdk_mgr.h b/fpdfsdk/include/fsdk_mgr.h index cd89a925ab..ca46a069f1 100644 --- a/fpdfsdk/include/fsdk_mgr.h +++ b/fpdfsdk/include/fsdk_mgr.h @@ -40,14 +40,6 @@ class CPDFDoc_Environment final { CPDFDoc_Environment(UnderlyingDocumentType* pDoc, FPDF_FORMFILLINFO* pFFinfo); ~CPDFDoc_Environment(); -#ifdef PDF_ENABLE_XFA - void Release() { - if (m_pInfo && m_pInfo->Release) - m_pInfo->Release(m_pInfo); - delete this; - } -#endif // PDF_ENABLE_XFA - void FFI_Invalidate(FPDF_PAGE page, double left, double top, |