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/cxfa_widgetacc.cpp | |
| 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/cxfa_widgetacc.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_widgetacc.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xfa/fxfa/cxfa_widgetacc.cpp b/xfa/fxfa/cxfa_widgetacc.cpp index 5a925da833..39aa378a52 100644 --- a/xfa/fxfa/cxfa_widgetacc.cpp +++ b/xfa/fxfa/cxfa_widgetacc.cpp @@ -1486,7 +1486,7 @@ XFA_CHECKSTATE CXFA_WidgetAcc::GetCheckState() { } void CXFA_WidgetAcc::SetCheckState(XFA_CHECKSTATE eCheckState, bool bNotify) { - CXFA_Node* node = m_pNode->GetExclGroup(); + CXFA_Node* node = m_pNode->GetExclGroupIfExists(); if (!node) { CXFA_Items* pItems = m_pNode->GetChild<CXFA_Items>(0, XFA_Element::Items, false); @@ -1506,6 +1506,7 @@ void CXFA_WidgetAcc::SetCheckState(XFA_CHECKSTATE eCheckState, bool bNotify) { } if (m_pNode) m_pNode->SyncValue(wsContent, bNotify); + return; } |