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 --- .../cxfa_traversestrategy_contentlayoutitem.h | 30 ---------------------- xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h | 5 ++++ 2 files changed, 5 insertions(+), 30 deletions(-) delete mode 100644 xfa/fxfa/parser/cxfa_traversestrategy_contentlayoutitem.h (limited to 'xfa/fxfa/parser') diff --git a/xfa/fxfa/parser/cxfa_traversestrategy_contentlayoutitem.h b/xfa/fxfa/parser/cxfa_traversestrategy_contentlayoutitem.h deleted file mode 100644 index de0d52ab10..0000000000 --- a/xfa/fxfa/parser/cxfa_traversestrategy_contentlayoutitem.h +++ /dev/null @@ -1,30 +0,0 @@ -// Copyright 2016 PDFium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com - -#ifndef XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_CONTENTLAYOUTITEM_H_ -#define XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_CONTENTLAYOUTITEM_H_ - -#include "xfa/fxfa/parser/cxfa_contentlayoutitem.h" - -class CXFA_TraverseStrategy_ContentLayoutItem { - public: - static CXFA_ContentLayoutItem* GetFirstChild( - CXFA_ContentLayoutItem* pLayoutItem) { - return static_cast(pLayoutItem->m_pFirstChild); - } - - static CXFA_ContentLayoutItem* GetNextSibling( - CXFA_ContentLayoutItem* pLayoutItem) { - return static_cast(pLayoutItem->m_pNextSibling); - } - - static CXFA_ContentLayoutItem* GetParent( - CXFA_ContentLayoutItem* pLayoutItem) { - return static_cast(pLayoutItem->m_pParent); - } -}; - -#endif // XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_CONTENTLAYOUTITEM_H_ diff --git a/xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h b/xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h index 7b39826130..0371eef449 100644 --- a/xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h +++ b/xfa/fxfa/parser/cxfa_traversestrategy_layoutitem.h @@ -8,6 +8,7 @@ #define XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_LAYOUTITEM_H_ #include "xfa/fxfa/parser/cxfa_layoutitem.h" +#include "xfa/fxfa/parser/cxfa_nodeiteratortemplate.h" class CXFA_TraverseStrategy_LayoutItem { public: @@ -22,4 +23,8 @@ class CXFA_TraverseStrategy_LayoutItem { } }; +using CXFA_LayoutItemIterator = + CXFA_NodeIteratorTemplate; + #endif // XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_LAYOUTITEM_H_ -- cgit v1.2.3