diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-20 22:23:50 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-20 22:23:50 +0000 |
| commit | 120c19b6a19377c341658c342c4351eccbb59741 (patch) | |
| tree | 6799a4d504efd2eb1c7a036bf7959a9352c199e4 /xfa/fxfa/cxfa_fffield.cpp | |
| parent | 2501ae2474c1f097a06634d647de2eb3fb730864 (diff) | |
| download | pdfium-chromium/3274.tar.xz | |
Rename CXFA_MarginData methodschromium/3274
This CL renames the CXFA_MarginData::Get*Inset methods to
CXFA_MarginData::Try*Inset to match the fact that they may fail. The
default value has been removed and all the callers correctly set the
default values.
Change-Id: Ib0dc8dc35807b60a0ddc0e5724604cb39845fcea
Reviewed-on: https://pdfium-review.googlesource.com/18771
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_fffield.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_fffield.cpp | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/xfa/fxfa/cxfa_fffield.cpp b/xfa/fxfa/cxfa_fffield.cpp index 7e36fea2cf..b4142ba701 100644 --- a/xfa/fxfa/cxfa_fffield.cpp +++ b/xfa/fxfa/cxfa_fffield.cpp @@ -172,11 +172,14 @@ void CXFA_FFField::CapPlacement() { CXFA_MarginData marginData = m_pDataAcc->GetMarginData(); if (marginData.HasValidNode()) { CXFA_LayoutItem* pItem = this; - float fLeftInset = 0, fRightInset = 0, fTopInset = 0, fBottomInset = 0; - marginData.GetLeftInset(fLeftInset); - marginData.GetRightInset(fRightInset); - marginData.GetTopInset(fTopInset); - marginData.GetBottomInset(fBottomInset); + float fLeftInset = 0; + float fRightInset = 0; + float fTopInset = 0; + float fBottomInset = 0; + marginData.TryLeftInset(fLeftInset); + marginData.TryRightInset(fRightInset); + marginData.TryTopInset(fTopInset); + marginData.TryBottomInset(fBottomInset); if (!pItem->GetPrev() && !pItem->GetNext()) { rtWidget.Deflate(fLeftInset, fTopInset, fRightInset, fBottomInset); } else { |