From 8584902d81ffabc06bb4895f558e375e5429fe72 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Sat, 28 Jul 2018 00:00:33 +0000 Subject: Remove CXFA_TraverseStrategy_ContentLayoutItem. It makes illegal casts and requires its callers to double-check the type validity of results before use. Instead, use the parent class iterator and perform checked casts. No functional difference, since it looks like the requisite checks were being made in all places. Make one "using" visible to other files to save some verbosity. Change-Id: I894ca15b4bdddd4723b787663950a58bc58b7f06 Reviewed-on: https://pdfium-review.googlesource.com/39030 Reviewed-by: Lei Zhang Commit-Queue: Lei Zhang --- xfa/fxfa/cxfa_ffpageview.h | 5 ----- 1 file changed, 5 deletions(-) (limited to 'xfa/fxfa/cxfa_ffpageview.h') diff --git a/xfa/fxfa/cxfa_ffpageview.h b/xfa/fxfa/cxfa_ffpageview.h index f3ae069638..daf2b1933e 100644 --- a/xfa/fxfa/cxfa_ffpageview.h +++ b/xfa/fxfa/cxfa_ffpageview.h @@ -12,7 +12,6 @@ #include "xfa/fxfa/parser/cxfa_containerlayoutitem.h" #include "xfa/fxfa/parser/cxfa_contentlayoutitem.h" -#include "xfa/fxfa/parser/cxfa_nodeiteratortemplate.h" #include "xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h" class CXFA_FFWidget; @@ -34,10 +33,6 @@ class CXFA_FFPageView : public CXFA_ContainerLayoutItem { UnownedPtr const m_pDocView; }; -using CXFA_LayoutItemIterator = - CXFA_NodeIteratorTemplate; - class CXFA_FFPageWidgetIterator : public IXFA_WidgetIterator { public: CXFA_FFPageWidgetIterator(CXFA_FFPageView* pPageView, uint32_t dwFilter); -- cgit v1.2.3