diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-20 20:28:43 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-20 20:28:43 +0000 |
| commit | 03d9180f30f2ba79ef48b4003306224428f4ad47 (patch) | |
| tree | b2d9daf2b6b026bac946589c2781ea555ce93059 /fxjs/cjx_node.h | |
| parent | e74ae8c02f6b5efa37ba11bcad6b714046cfc7b0 (diff) | |
| download | pdfium-03d9180f30f2ba79ef48b4003306224428f4ad47.tar.xz | |
Remove return value from SetAttributeValue
The CJX_Node::SetAttributeValue method always returns |true| and we
never check the value. This CL changes the method to return |void|
instead.
Change-Id: Ia3162627dee926d602e7ffe74f9dca2c1fd81ca8
Reviewed-on: https://pdfium-review.googlesource.com/18770
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/cjx_node.h')
| -rw-r--r-- | fxjs/cjx_node.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h index d0f8279e7f..148f1a6378 100644 --- a/fxjs/cjx_node.h +++ b/fxjs/cjx_node.h @@ -59,7 +59,7 @@ class CJX_Node : public CJX_Object { pdfium::Optional<WideString> TryAttribute(XFA_Attribute eAttr, bool bUseDefault); - bool SetAttributeValue(const WideString& wsValue, + void SetAttributeValue(const WideString& wsValue, const WideString& wsXMLValue, bool bNotify, bool bScriptModify); |