diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-24 20:05:29 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-24 20:05:29 +0000 |
| commit | b657e87fa1e25997c8277fa49ed33ff245391f31 (patch) | |
| tree | c95f75693ea9a03e82ac0b696ca992ef5bd714ac /fxjs/xfa/cjx_object.cpp | |
| parent | 4327b48d249ad61020e353c794a08b2969949eba (diff) | |
| download | pdfium-b657e87fa1e25997c8277fa49ed33ff245391f31.tar.xz | |
Cleanup CreateUIChild node creation
This CL cleans up some of the code around creating the XML nodes related
to the UI widgets.
Change-Id: Ib91364439ab039f46e44690e92cc0cb93a8da203
Reviewed-on: https://pdfium-review.googlesource.com/23770
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/xfa/cjx_object.cpp')
| -rw-r--r-- | fxjs/xfa/cjx_object.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fxjs/xfa/cjx_object.cpp b/fxjs/xfa/cjx_object.cpp index 46af66ae28..d663fcce70 100644 --- a/fxjs/xfa/cjx_object.cpp +++ b/fxjs/xfa/cjx_object.cpp @@ -377,7 +377,7 @@ Optional<int32_t> CJX_Object::TryInteger(XFA_Attribute eAttr, } Optional<XFA_AttributeEnum> CJX_Object::TryEnum(XFA_Attribute eAttr, - bool bUseDefault) { + bool bUseDefault) const { void* pKey = GetMapKey_Element(GetXFAObject()->GetElementType(), eAttr); void* pValue = nullptr; if (GetMapModuleValue(pKey, pValue)) { @@ -402,7 +402,7 @@ bool CJX_Object::SetEnum(XFA_Attribute eAttr, return true; } -XFA_AttributeEnum CJX_Object::GetEnum(XFA_Attribute eAttr) { +XFA_AttributeEnum CJX_Object::GetEnum(XFA_Attribute eAttr) const { return TryEnum(eAttr, true).value_or(XFA_AttributeEnum::Unknown); } @@ -969,8 +969,8 @@ void CJX_Object::SetMapModuleValue(void* pKey, void* pValue) { CreateMapModuleData()->m_ValueMap[pKey] = pValue; } -bool CJX_Object::GetMapModuleValue(void* pKey, void*& pValue) { - for (CXFA_Node* pNode = ToNode(GetXFAObject()); pNode; +bool CJX_Object::GetMapModuleValue(void* pKey, void*& pValue) const { + for (const CXFA_Node* pNode = ToNode(GetXFAObject()); pNode; pNode = pNode->GetTemplateNodeIfExists()) { XFA_MAPMODULEDATA* pModule = pNode->JSObject()->GetMapModuleData(); if (pModule) { |