diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-02 19:09:58 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-02 19:09:58 +0000 |
| commit | 3ac07ffafea1ee4d3cf985e32692849dd3fcfdfb (patch) | |
| tree | 3edbd3db87541a57fa0ee172bd50b70d35d45a11 /fxjs/cjx_node.cpp | |
| parent | bf29e5b532e4ef9a309906083b33b6d4c48ff757 (diff) | |
| download | pdfium-3ac07ffafea1ee4d3cf985e32692849dd3fcfdfb.tar.xz | |
Cleanup params to CJX_Node::TryCData
This CL removes the bProto param as it was never set and inlines the
value where needed in the methods. The default value was removed from
bUseDefault and inlined into callsites as needed.
Change-Id: I773261d19aa3799bc607e7df482b5f5e5217bee6
Reviewed-on: https://pdfium-review.googlesource.com/17533
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs/cjx_node.cpp')
| -rw-r--r-- | fxjs/cjx_node.cpp | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp index 0c78f99551..c6e60933f0 100644 --- a/fxjs/cjx_node.cpp +++ b/fxjs/cjx_node.cpp @@ -3031,7 +3031,7 @@ CXFA_Measurement CJX_Node::GetMeasure(XFA_ATTRIBUTE eAttr) const { WideStringView CJX_Node::GetCData(XFA_ATTRIBUTE eAttr) { WideStringView wsValue; - return TryCData(eAttr, wsValue) ? wsValue : WideStringView(); + return TryCData(eAttr, wsValue, true) ? wsValue : WideStringView(); } bool CJX_Node::SetCData(XFA_ATTRIBUTE eAttr, @@ -3166,11 +3166,10 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue, bool CJX_Node::TryCData(XFA_ATTRIBUTE eAttr, WideString& wsValue, - bool bUseDefault, - bool bProto) { + bool bUseDefault) { void* pKey = GetMapKey_Element(GetXFANode()->GetElementType(), eAttr); if (eAttr == XFA_ATTRIBUTE_Value) { - WideString* pStr = (WideString*)GetUserData(pKey, bProto); + WideString* pStr = (WideString*)GetUserData(pKey, true); if (pStr) { wsValue = *pStr; return true; @@ -3197,11 +3196,10 @@ bool CJX_Node::TryCData(XFA_ATTRIBUTE eAttr, bool CJX_Node::TryCData(XFA_ATTRIBUTE eAttr, WideStringView& wsValue, - bool bUseDefault, - bool bProto) { + bool bUseDefault) { void* pKey = GetMapKey_Element(GetXFANode()->GetElementType(), eAttr); if (eAttr == XFA_ATTRIBUTE_Value) { - WideString* pStr = (WideString*)GetUserData(pKey, bProto); + WideString* pStr = (WideString*)GetUserData(pKey, true); if (pStr) { wsValue = pStr->AsStringView(); return true; @@ -3556,7 +3554,7 @@ bool CJX_Node::TryContent(WideString& wsContent, GetXFANode()->InsertChild(pContentRawDataNode); } return pContentRawDataNode->JSNode()->TryContent(wsContent, bScriptModify, - bProto); + true); } case XFA_ObjectType::NodeC: case XFA_ObjectType::NodeV: @@ -3575,7 +3573,7 @@ bool CJX_Node::TryContent(WideString& wsContent, GetDocument()->GetScriptContext()->AddNodesOfRunScript(GetXFANode()); } } - return TryCData(XFA_ATTRIBUTE_Value, wsContent, false, bProto); + return TryCData(XFA_ATTRIBUTE_Value, wsContent, false); } return false; } |