diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-11 14:46:32 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-11 14:46:32 +0000 |
| commit | 7d2c5f4c8d33d57b76af1c471e70af36db090072 (patch) | |
| tree | 929bc3f0aa098eacd987b6b7f9f8fd077f5cba66 /xfa/fxfa/parser | |
| parent | fc77dee8e5d0f941ea6050aa632254b588a21f87 (diff) | |
| download | pdfium-7d2c5f4c8d33d57b76af1c471e70af36db090072.tar.xz | |
Rename CreateInstance to CreateInstanceIfPossible
This CL makes it clear that CreateInstance may return nullptr.
Change-Id: I82f172a671761eae9055b0e9e55ab0e5a7d599aa
Reviewed-on: https://pdfium-review.googlesource.com/22712
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 2cf521f44b..c29e30347f 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -1284,7 +1284,7 @@ void CXFA_Node::RemoveItem(CXFA_Node* pRemoveInstance, } } -CXFA_Node* CXFA_Node::CreateInstance(bool bDataMerge) { +CXFA_Node* CXFA_Node::CreateInstanceIfPossible(bool bDataMerge) { CXFA_Document* pDocument = GetDocument(); CXFA_Node* pTemplateNode = GetTemplateNodeIfExists(); if (!pTemplateNode) diff --git a/xfa/fxfa/parser/cxfa_node.h b/xfa/fxfa/parser/cxfa_node.h index d8689859be..1e20ace09c 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -102,7 +102,7 @@ class CXFA_Node : public CXFA_Object { void SetFlag(uint32_t dwFlag, bool bNotify); void ClearFlag(uint32_t dwFlag); - CXFA_Node* CreateInstance(bool bDataMerge); + CXFA_Node* CreateInstanceIfPossible(bool bDataMerge); int32_t GetCount(); CXFA_Node* GetItem(int32_t iIndex); void RemoveItem(CXFA_Node* pRemoveInstance, bool bRemoveDataBinding); |