diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-07-30 19:31:19 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-07-30 19:31:19 +0000 |
| commit | d718d634b7aa3311a5b6298574d06f091365259f (patch) | |
| tree | 39560860d111a143e6d3b27e42245ca79710daab /xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h | |
| parent | 5822da79e645ad896c4aeec623d38a92ee7231a8 (diff) | |
| download | pdfium-d718d634b7aa3311a5b6298574d06f091365259f.tar.xz | |
Type-safe iteration over CXFA_ContentLayoutItems.
There's no reason that every CXFA_LayoutItem must be a
CXFA_ContentLayout item, so use checked conversions. The
code may already be arranging things so the constraint is
already satisfied, but this adds an additional layer of
safety.
Tidy some long expressions with pre-existing typedef for node
iterator while we're at it.
Change-Id: If1448131ca73bfedb2266b070f1a913688598e64
Reviewed-on: https://pdfium-review.googlesource.com/39051
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h b/xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h index fc98d18623..90f39a8ba0 100644 --- a/xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h +++ b/xfa/fxfa/parser/cxfa_traversestrategy_xfanode.h @@ -22,7 +22,7 @@ class CXFA_TraverseStrategy_XFANode { } }; -typedef CXFA_NodeIteratorTemplate<CXFA_Node, CXFA_TraverseStrategy_XFANode> - CXFA_NodeIterator; +using CXFA_NodeIterator = + CXFA_NodeIteratorTemplate<CXFA_Node, CXFA_TraverseStrategy_XFANode>; #endif // XFA_FXFA_PARSER_CXFA_TRAVERSESTRATEGY_XFANODE_H_ |