diff options
author | dsinclair <dsinclair@chromium.org> | 2016-10-05 12:00:34 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-10-05 12:00:34 -0700 |
commit | 8afe15a019c78a24168ebc42306342c25358ccdf (patch) | |
tree | 09012697e63b0a1fa4654f6e6f93b69a04421171 /fpdfsdk/cba_annotiterator.h | |
parent | 7c1b07fde27dfbbf1d979d9dfd616b1a42591e5f (diff) | |
download | pdfium-8afe15a019c78a24168ebc42306342c25358ccdf.tar.xz |
Cleanup some CPDFSDK_PageView annotation code.
This Cl cleans up the code regarding CPDFSDK_Annots in CPDFSDK_PageView.
This includes:
* Makes DeleteAnnot XFA only and wraps at the call site.
* Removes unused methods
* Replaces use of CountAnnots and GetAnnot with vector iteration
* Removes {Set|Kill}FocusAnnot from CPDFSDK_PageView
* Renames m_fxAnnotArray to m_SDKAnnotArray
Review-Url: https://codereview.chromium.org/2384323005
Diffstat (limited to 'fpdfsdk/cba_annotiterator.h')
-rw-r--r-- | fpdfsdk/cba_annotiterator.h | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/fpdfsdk/cba_annotiterator.h b/fpdfsdk/cba_annotiterator.h index a896c862c7..5cbe8e3d57 100644 --- a/fpdfsdk/cba_annotiterator.h +++ b/fpdfsdk/cba_annotiterator.h @@ -31,13 +31,10 @@ class CBA_AnnotIterator { private: void GenerateResults(); - static CFX_FloatRect GetAnnotRect(const CPDFSDK_Annot* pAnnot); - - // Function signature compatible with std::sort(). - static bool CompareByLeftAscending(const CPDFSDK_Annot* p1, - const CPDFSDK_Annot* p2); - static bool CompareByTopDescending(const CPDFSDK_Annot* p1, - const CPDFSDK_Annot* p2); + void CollectAnnots(std::vector<CPDFSDK_Annot*>* pArray); + CFX_FloatRect AddToAnnotsList(std::vector<CPDFSDK_Annot*>* sa, size_t idx); + void AddSelectedToAnnots(std::vector<CPDFSDK_Annot*>* sa, + std::vector<size_t>* aSelect); TabOrder m_eTabOrder; CPDFSDK_PageView* m_pPageView; |