diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-11 16:29:42 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-11 16:29:42 +0000 |
| commit | b800d149de09ac7d73aa3d1ff8558e33a4c56009 (patch) | |
| tree | c1bd0bfea008ffe1c1f1bc357d36af980b07e288 /xfa/fxfa/parser | |
| parent | 61dfd72ab3a913b143c03bdcf4ec5816dcb09cd1 (diff) | |
| download | pdfium-b800d149de09ac7d73aa3d1ff8558e33a4c56009.tar.xz | |
Rename GetExclGroup to GetExclGroupIfExists
This CL makes it clearer that GetExclGroup can return nullptr.
Change-Id: Ica9bdfb05be36b3b6ede20f8b54daac1154506fd
Reviewed-on: https://pdfium-review.googlesource.com/22739
Reviewed-by: Henrique Nakashima <hnakashima@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 075ee46836..5d72cf7cb9 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -1654,7 +1654,7 @@ Optional<float> CXFA_Node::TryMaxHeight() { return JSObject()->TryMeasureAsFloat(XFA_Attribute::MaxH); } -CXFA_Node* CXFA_Node::GetExclGroup() { +CXFA_Node* CXFA_Node::GetExclGroupIfExists() { CXFA_Node* pExcl = GetParent(); if (!pExcl || pExcl->GetElementType() != XFA_Element::ExclGroup) return nullptr; diff --git a/xfa/fxfa/parser/cxfa_node.h b/xfa/fxfa/parser/cxfa_node.h index 8108eeccc4..dec324300d 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -263,7 +263,7 @@ class CXFA_Node : public CXFA_Object { Optional<float> TryMaxWidth(); Optional<float> TryMaxHeight(); - CXFA_Node* GetExclGroup(); + CXFA_Node* GetExclGroupIfExists(); int32_t ProcessEvent(CXFA_FFDocView* docView, XFA_AttributeEnum iActivity, |