summaryrefslogtreecommitdiff
path: root/xfa/fxfa/parser/cxfa_layoutprocessor.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2018-01-31 20:13:44 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-01-31 20:13:44 +0000
commit405f17ae3f297d28917370d013d95adc31e7c4a1 (patch)
tree0afaaaa4f8652903dfd49a381dd871c1da8a017b /xfa/fxfa/parser/cxfa_layoutprocessor.cpp
parent990c09373b733346a06b03167ca43252cb93e986 (diff)
downloadpdfium-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.cpp12
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 {