diff options
| author | Henrique Nakashima <hnakashima@chromium.org> | 2018-05-02 21:14:27 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-02 21:14:27 +0000 |
| commit | 6312bf94bb44e2edeb855008da0921c13103f66a (patch) | |
| tree | edafcb079272c4a20abc587861d5df492846a5fc /xfa/fxfa/cxfa_ffpushbutton.cpp | |
| parent | fceaf42fb5930070415703e448ecce39e9dc0522 (diff) | |
| download | pdfium-6312bf94bb44e2edeb855008da0921c13103f66a.tar.xz | |
Use pointer instead of ref for XFA_RectWithoutMargin param.chromium/3418
Removed many null checks since XFA_RectWithoutMargin already checks
if the margin is null internally.
Change-Id: I21f5101fe38559eb02f4c3aaf081ff29ce293e47
Reviewed-on: https://pdfium-review.googlesource.com/31993
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffpushbutton.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffpushbutton.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fxfa/cxfa_ffpushbutton.cpp b/xfa/fxfa/cxfa_ffpushbutton.cpp index 7c556b81a8..a40b1ef1f1 100644 --- a/xfa/fxfa/cxfa_ffpushbutton.cpp +++ b/xfa/fxfa/cxfa_ffpushbutton.cpp @@ -102,15 +102,13 @@ bool CXFA_FFPushButton::PerformLayout() { m_rtUI = rtWidget; CXFA_Margin* margin = m_pNode->GetMarginIfExists(); - if (margin) - XFA_RectWithoutMargin(rtWidget, margin); + XFA_RectWithoutMargin(&rtWidget, margin); m_rtCaption = rtWidget; CXFA_Caption* caption = m_pNode->GetCaptionIfExists(); CXFA_Margin* captionMargin = caption ? caption->GetMarginIfExists() : nullptr; - if (captionMargin) - XFA_RectWithoutMargin(m_rtCaption, captionMargin); + XFA_RectWithoutMargin(&m_rtCaption, captionMargin); LayoutHighlightCaption(); SetFWLRect(); |