diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-16 15:46:35 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-16 15:46:35 +0000 |
| commit | 54e4ccbbf09c496f8ab79a2b02fc4afc3d6abc42 (patch) | |
| tree | 4ad1768673054ccacf524e632dfae40fab87a609 /xfa/fxfa/cxfa_ffline.cpp | |
| parent | d3ae179dab3f63625e5dd3ddf0aa33176f8ee65c (diff) | |
| download | pdfium-54e4ccbbf09c496f8ab79a2b02fc4afc3d6abc42.tar.xz | |
Rename CXFA_Line methods to make clearer
This CL clarifies that he CXFA_Line::GetEdge method can return nullptr.
Change-Id: I785d4636126727e07a80afb001489bd00ec9ae9e
Reviewed-on: https://pdfium-review.googlesource.com/22771
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffline.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffline.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/xfa/fxfa/cxfa_ffline.cpp b/xfa/fxfa/cxfa_ffline.cpp index ee8ca1841a..cdaca4a88f 100644 --- a/xfa/fxfa/cxfa_ffline.cpp +++ b/xfa/fxfa/cxfa_ffline.cpp @@ -96,11 +96,11 @@ void CXFA_FFLine::RenderWidget(CXFA_Graphics* pGS, float fLineWidth = 1.0f; XFA_AttributeEnum iStrokeType = XFA_AttributeEnum::Unknown; XFA_AttributeEnum iCap = XFA_AttributeEnum::Unknown; - CXFA_Edge* edge = line->GetEdge(); - if (edge) { - if (!edge->IsVisible()) - return; + CXFA_Edge* edge = line->GetEdgeIfExists(); + if (edge && !edge->IsVisible()) + return; + if (edge) { lineColor = edge->GetColor(); iStrokeType = edge->GetStrokeType(); fLineWidth = edge->GetThickness(); |