diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-31 20:13:44 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-31 20:13:44 +0000 |
commit | 405f17ae3f297d28917370d013d95adc31e7c4a1 (patch) | |
tree | 0afaaaa4f8652903dfd49a381dd871c1da8a017b /xfa/fxfa/parser/cxfa_layoutprocessor.cpp | |
parent | 990c09373b733346a06b03167ca43252cb93e986 (diff) | |
download | pdfium-405f17ae3f297d28917370d013d95adc31e7c4a1.tar.xz |
Cleanup visibility in CXFA_ItemLayoutProcessor
This CL moves anonymous namespace methods in CXFA_ItemLayoutProcessor to
be private methods and then fixes the visibility on most of the
methods/data to be private.
Change-Id: Idfccbc53b94628b18bc3576e4466391b0f704b44
Reviewed-on: https://pdfium-review.googlesource.com/24851
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_layoutprocessor.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_layoutprocessor.cpp | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/xfa/fxfa/parser/cxfa_layoutprocessor.cpp b/xfa/fxfa/parser/cxfa_layoutprocessor.cpp index 540e1c9156..c31766f6f6 100644 --- a/xfa/fxfa/parser/cxfa_layoutprocessor.cpp +++ b/xfa/fxfa/parser/cxfa_layoutprocessor.cpp @@ -101,17 +101,7 @@ bool CXFA_LayoutProcessor::IncrementLayout() { StartLayout(true); return DoLayout() == 100; } - for (CXFA_Node* pNode : m_rgChangedContainers) { - CXFA_Node* pParentNode = pNode->GetContainerParent(); - if (!pParentNode) - return false; - if (!CXFA_ItemLayoutProcessor::IncrementRelayoutNode(this, pNode, - pParentNode)) { - return false; - } - } - m_rgChangedContainers.clear(); - return true; + return m_rgChangedContainers.empty(); } int32_t CXFA_LayoutProcessor::CountPages() const { |