diff options
| author | Tom Sepez <tsepez@chromium.org> | 2016-01-20 14:42:27 -0800 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2016-01-20 14:42:27 -0800 |
| commit | f409da4b8a3e6a3885605a47bed4f8798c4703b9 (patch) | |
| tree | 1fc8ed34508234cd2cf7fb4ff15dc8ff96d21d9f /fpdfsdk/include/fsdk_actionhandler.h | |
| parent | a505d9d3e13485e090e50d5d83f882fb707d3f48 (diff) | |
| download | pdfium-f409da4b8a3e6a3885605a47bed4f8798c4703b9.tar.xz | |
Use std::set to track visited dictionaries in fsdk_actionhandler.
Find() in a list isn't the best idea.
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1610303002 .
Diffstat (limited to 'fpdfsdk/include/fsdk_actionhandler.h')
| -rw-r--r-- | fpdfsdk/include/fsdk_actionhandler.h | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/fpdfsdk/include/fsdk_actionhandler.h b/fpdfsdk/include/fsdk_actionhandler.h index 33f127d8c9..b5672d178d 100644 --- a/fpdfsdk/include/fsdk_actionhandler.h +++ b/fpdfsdk/include/fsdk_actionhandler.h @@ -8,12 +8,12 @@ #define FPDFSDK_INCLUDE_FSDK_ACTIONHANDLER_H_ #include <memory> +#include <set> #include "core/include/fpdfdoc/fpdf_doc.h" #include "core/include/fxcrt/fx_string.h" #include "fsdk_baseform.h" -class CFX_PtrList; class CPDFSDK_Annot; class CPDFSDK_Document; class CPDF_Bookmark; @@ -68,29 +68,29 @@ class CPDFSDK_ActionHandler { private: FX_BOOL ExecuteDocumentOpenAction(const CPDF_Action& action, CPDFSDK_Document* pDocument, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); FX_BOOL ExecuteDocumentPageAction(const CPDF_Action& action, CPDF_AAction::AActionType type, CPDFSDK_Document* pDocument, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); FX_BOOL ExecuteFieldAction(const CPDF_Action& action, CPDF_AAction::AActionType type, CPDFSDK_Document* pDocument, CPDF_FormField* pFormField, PDFSDK_FieldAction& data, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); FX_BOOL ExecuteScreenAction(const CPDF_Action& action, CPDF_AAction::AActionType type, CPDFSDK_Document* pDocument, CPDFSDK_Annot* pScreen, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); FX_BOOL ExecuteBookMark(const CPDF_Action& action, CPDFSDK_Document* pDocument, CPDF_Bookmark* pBookmark, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); FX_BOOL ExecuteLinkAction(const CPDF_Action& action, CPDFSDK_Document* pDocument, - CFX_PtrList& list); + std::set<CPDF_Dictionary*>* visited); void DoAction_NoJs(const CPDF_Action& action, CPDFSDK_Document* pDocument); void RunDocumentPageJavaScript(CPDFSDK_Document* pDocument, |