diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-02 18:18:18 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-02 18:18:18 +0000 |
| commit | ea708c02877df7038c5f2300b037dd0eedff6f54 (patch) | |
| tree | e1a4e0139f6889e436d1eb9e31ee56906a152380 /fxjs | |
| parent | 019c1c61b7bf3f3573ce158c9735f3c1b1db7c68 (diff) | |
| download | pdfium-ea708c02877df7038c5f2300b037dd0eedff6f54.tar.xz | |
Remove default values from CJX_Node::SetAttributeValue
This CL removes the default parameters from SetAttributeValue and
inlines into the call sites as needed.
Change-Id: Ibb4e4747141a49c2367743421254c251aff71eeb
Reviewed-on: https://pdfium-review.googlesource.com/17490
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs')
| -rw-r--r-- | fxjs/cjx_node.cpp | 2 | ||||
| -rw-r--r-- | fxjs/cjx_node.h | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp index 4cebf44cd7..59ddab5db0 100644 --- a/fxjs/cjx_node.cpp +++ b/fxjs/cjx_node.cpp @@ -3396,7 +3396,7 @@ bool CJX_Node::SetScriptContent(const WideString& wsContent, pValueNode; pValueNode = pValueNode->GetNodeItem( XFA_NODEITEM_NextSibling)) { pValueNode->JSNode()->SetAttributeValue( - wsSaveTextArray[i], wsSaveTextArray[i], false); + wsSaveTextArray[i], wsSaveTextArray[i], false, false); i++; } } diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h index 442e0d57e3..1b8ad5c079 100644 --- a/fxjs/cjx_node.h +++ b/fxjs/cjx_node.h @@ -65,8 +65,8 @@ class CJX_Node : public CJX_Object { bool GetAttribute(XFA_ATTRIBUTE eAttr, WideString& wsValue, bool bUseDefault); bool SetAttributeValue(const WideString& wsValue, const WideString& wsXMLValue, - bool bNotify = false, - bool bScriptModify = false); + bool bNotify, + bool bScriptModify); bool RemoveAttribute(const WideStringView& wsAttr); CXFA_Node* GetProperty(int32_t index, |