diff options
author | Lei Zhang <thestig@chromium.org> | 2015-06-22 15:10:28 -0700 |
---|---|---|
committer | Lei Zhang <thestig@chromium.org> | 2015-06-22 15:10:28 -0700 |
commit | f55771af6429ecf14beed1cae5ec2340ef7f37ad (patch) | |
tree | 96f83002cb0e831e5091920925998bcc9fcea55d /fpdfsdk/src/fsdk_mgr.cpp | |
parent | f2d3911ce1a07812d75e7671e038d0922a823528 (diff) | |
download | pdfium-f55771af6429ecf14beed1cae5ec2340ef7f37ad.tar.xz |
Delete dead code for deleting pages and icons.
1) Document::deletePages()
2) CPDFSDK_Document::DeletePages()
3) Document::removeIcon()
4) IconTree::DeleteIconElement()
Originally (1) called (2), but only when FOXIT_CHROME_BUILD was not
defined. Since it was always defined for PDFium, this was effectively
dead code the whole time. Ditto for (3) and (4). The functions were
deemed unsafe in https://crbug.com/67100
R=jam@chromium.org
Review URL: https://codereview.chromium.org/1193323002.
Diffstat (limited to 'fpdfsdk/src/fsdk_mgr.cpp')
-rw-r--r-- | fpdfsdk/src/fsdk_mgr.cpp | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/fpdfsdk/src/fsdk_mgr.cpp b/fpdfsdk/src/fsdk_mgr.cpp index 003753223f..ad27aeb92f 100644 --- a/fpdfsdk/src/fsdk_mgr.cpp +++ b/fpdfsdk/src/fsdk_mgr.cpp @@ -517,25 +517,6 @@ FX_BOOL CPDFSDK_Document::KillFocusAnnot(FX_UINT nFlag) return FALSE; } -FX_BOOL CPDFSDK_Document::DeletePages(int nStart, int nCount) -{ - if ( nStart < 0 || nStart >= GetPageCount() || nCount <= 0 ) - { - return FALSE; - } - - CPDF_Page * pTempPage = NULL; - for ( int i = nCount-1; i >= 0; i-- ) - { - pTempPage = GetPage(nStart+i); - if ( pTempPage != NULL ) - { - ReMovePageView(pTempPage); - } - } - return TRUE; -} - void CPDFSDK_Document::OnCloseDocument() { KillFocusAnnot(); |