diff options
| author | dsinclair <dsinclair@chromium.org> | 2016-06-23 13:34:10 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-06-23 13:34:10 -0700 |
| commit | b97784706ec898ff6b1c36f1564c0c66f9419b17 (patch) | |
| tree | 679a8602c52caee9efb7746f273b0959843c3be1 /xfa/fxfa/parser/xfa_document_imp.cpp | |
| parent | 85d1f2c2f9f1e746bedb1b6f03576613f54fbc27 (diff) | |
| download | pdfium-b97784706ec898ff6b1c36f1564c0c66f9419b17.tar.xz | |
Pass needed value to CXFA_Node constructor
Currently the CXFA_Node constructor will call out to lookup the object type for
a given element type. There is only one called of this constructor and it
already has the object type so just pass it through instead of getting the
element data a second time.
Review-Url: https://codereview.chromium.org/2092853002
Diffstat (limited to 'xfa/fxfa/parser/xfa_document_imp.cpp')
| -rw-r--r-- | xfa/fxfa/parser/xfa_document_imp.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xfa/fxfa/parser/xfa_document_imp.cpp b/xfa/fxfa/parser/xfa_document_imp.cpp index bcc37d9f65..7ba85bb35c 100644 --- a/xfa/fxfa/parser/xfa_document_imp.cpp +++ b/xfa/fxfa/parser/xfa_document_imp.cpp @@ -157,7 +157,8 @@ CXFA_Node* CXFA_Document::CreateNode(const XFA_PACKETINFO* pPacket, const XFA_ELEMENTINFO* pElement = XFA_GetElementByID(eElement); if (pElement && (pElement->dwPackets & pPacket->eName)) { - CXFA_Node* pNode = new CXFA_Node(this, pPacket->eName, pElement->eName); + CXFA_Node* pNode = new CXFA_Node(this, pPacket->eName, + pElement->eObjectType, pElement->eName); AddPurgeNode(pNode); return pNode; } |