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_ffpageview.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_ffpageview.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffpageview.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/cxfa_ffpageview.cpp b/xfa/fxfa/cxfa_ffpageview.cpp index 39f4dbe03b..53b39fd3fa 100644 --- a/xfa/fxfa/cxfa_ffpageview.cpp +++ b/xfa/fxfa/cxfa_ffpageview.cpp @@ -351,12 +351,12 @@ void CXFA_FFTabOrderPageWidgetIterator::CreateTabOrderWidgetArray() { ? it - SpaceOrderWidgetArray.begin() + 1 : 0; while (true) { - CXFA_FFWidget* pRadio = + CXFA_FFWidget* radio = SpaceOrderWidgetArray[iWidgetIndex % nWidgetCount]; - if (pRadio->GetNode()->GetExclGroup() != pWidgetAcc->GetNode()) + if (radio->GetNode()->GetExclGroupIfExists() != pWidgetAcc->GetNode()) break; if (!pdfium::ContainsValue(m_TabOrderWidgetArray, hWidget)) - m_TabOrderWidgetArray.push_back(pRadio); + m_TabOrderWidgetArray.push_back(radio); iWidgetIndex++; } |