diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-29 21:17:57 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-29 21:17:57 +0000 |
| commit | c30eb662f8a8ea20564e887182e422fd5c4e2099 (patch) | |
| tree | 96b89195e21153082f13652c3310097579fa5031 /xfa/fxfa/parser/cxfa_node.cpp | |
| parent | 4dd1b07b00842f33be3ebccdc8e4fd3219191c63 (diff) | |
| download | pdfium-c30eb662f8a8ea20564e887182e422fd5c4e2099.tar.xz | |
Cleanup CXFA_Node visibility
Make methods private or remove if not needed.
Change-Id: I2f9f718ba08a4b49332c896d2385ec60cf9cf01b
Reviewed-on: https://pdfium-review.googlesource.com/24490
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_node.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index c97a0f3bfd..1b53b6b7b0 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -813,7 +813,7 @@ CXFA_Node* CXFA_Node::GetBindData() { } std::vector<UnownedPtr<CXFA_Node>>* CXFA_Node::GetBindItems() { - return GetBindingNodes(); + return &binding_nodes_; } int32_t CXFA_Node::AddBindItem(CXFA_Node* pFormNode) { @@ -843,7 +843,7 @@ int32_t CXFA_Node::AddBindItem(CXFA_Node* pFormNode) { std::vector<UnownedPtr<CXFA_Node>> items; items.emplace_back(pOldFormItem); items.emplace_back(pFormNode); - SetBindingNodes(std::move(items)); + binding_nodes_ = std::move(items); m_uNodeFlags |= XFA_NodeFlag_BindFormItems; return 2; @@ -3024,11 +3024,6 @@ void CXFA_Node::GetImageEditDpi(int32_t& iImageXDpi, int32_t& iImageYDpi) { iImageYDpi = pData->m_iImageYDpi; } -void CXFA_Node::LoadText(CXFA_FFDoc* doc) { - InitLayoutData(); - static_cast<CXFA_TextLayoutData*>(m_pLayoutData.get())->LoadText(doc, this); -} - float CXFA_Node::CalculateWidgetAutoWidth(float fWidthCalc) { CXFA_Margin* margin = GetMarginIfExists(); if (margin) @@ -3426,7 +3421,8 @@ void CXFA_Node::InitLayoutData() { void CXFA_Node::StartTextLayout(CXFA_FFDoc* doc, float& fCalcWidth, float& fCalcHeight) { - LoadText(doc); + InitLayoutData(); + static_cast<CXFA_TextLayoutData*>(m_pLayoutData.get())->LoadText(doc, this); CXFA_TextLayout* pTextLayout = static_cast<CXFA_TextLayoutData*>(m_pLayoutData.get())->GetTextLayout(); |