diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-10 17:48:57 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-10 17:48:57 +0000 |
| commit | 7d36c3e835bc32898f0064226d79a8f0f89e15ec (patch) | |
| tree | 7b9df79705fbeb8eeaa9fe537fc23d7acd30565e /xfa/fxfa/parser | |
| parent | 50bd8fa8a9505487c1109e2645ef785ad73b1cbc (diff) | |
| download | pdfium-7d36c3e835bc32898f0064226d79a8f0f89e15ec.tar.xz | |
Rename CXFA_Node::GetBorder to GetBorderIfExists
Renaming GetBorder makes it clear that the method may return a nullptr.
Change-Id: I50a8b511e273552e3a4a226ba72a7c0e2ae74ea3
Reviewed-on: https://pdfium-review.googlesource.com/22653
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 2 | ||||
| -rw-r--r-- | xfa/fxfa/parser/cxfa_node.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index 6d77f74c4e..02d1c0f6f3 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -1530,7 +1530,7 @@ int32_t CXFA_Node::GetRotate() { return degrees ? XFA_MapRotation(*degrees) / 90 * 90 : 0; } -CXFA_Border* CXFA_Node::GetBorder() const { +CXFA_Border* CXFA_Node::GetBorderIfExists() const { return JSObject()->GetProperty<CXFA_Border>(0, XFA_Element::Border); } diff --git a/xfa/fxfa/parser/cxfa_node.h b/xfa/fxfa/parser/cxfa_node.h index c1074ffaad..b175a83a84 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -236,7 +236,7 @@ class CXFA_Node : public CXFA_Object { bool IsOpenAccess(); - CXFA_Border* GetBorder() const; + CXFA_Border* GetBorderIfExists() const; CXFA_Border* GetOrCreateBorder(); CXFA_Caption* GetCaption() const; |