diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-16 15:15:05 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-16 15:15:05 +0000 |
commit | d3ae179dab3f63625e5dd3ddf0aa33176f8ee65c (patch) | |
tree | 151c01afa3404c846b1ab323607b244427b57ae9 /xfa/fxfa/parser/cxfa_color.cpp | |
parent | b36c7e1f84ea7402b7576d2a03a219d469735434 (diff) | |
download | pdfium-d3ae179dab3f63625e5dd3ddf0aa33176f8ee65c.tar.xz |
Cleanup CXFA_Fill
This CL cleans up the CXFA_Fill internal getters to handle nullptr
returns correctly and moves some of the logic to the specific XFA
classes.
Change-Id: Icac487105a026a25cc9981d00fbc152e459ad0b8
Reviewed-on: https://pdfium-review.googlesource.com/22770
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_color.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_color.cpp | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/xfa/fxfa/parser/cxfa_color.cpp b/xfa/fxfa/parser/cxfa_color.cpp index a072d0302b..95c0ad345b 100644 --- a/xfa/fxfa/parser/cxfa_color.cpp +++ b/xfa/fxfa/parser/cxfa_color.cpp @@ -37,3 +37,23 @@ CXFA_Color::CXFA_Color(CXFA_Document* doc, XFA_PacketType packet) pdfium::MakeUnique<CJX_Color>(this)) {} CXFA_Color::~CXFA_Color() {} + +FX_ARGB CXFA_Color::GetValue() { + Optional<WideString> val = JSObject()->TryCData(XFA_Attribute::Value, false); + return val ? StringToFXARGB(val->AsStringView()) : 0xFF000000; +} + +FX_ARGB CXFA_Color::GetValueOrDefault(FX_ARGB defaultValue) { + Optional<WideString> val = JSObject()->TryCData(XFA_Attribute::Value, false); + return val ? StringToFXARGB(val->AsStringView()) : defaultValue; +} + +void CXFA_Color::SetValue(FX_ARGB color) { + int a; + int r; + int g; + int b; + std::tie(a, r, g, b) = ArgbDecode(color); + JSObject()->SetCData(XFA_Attribute::Value, + WideString::Format(L"%d,%d,%d", r, g, b), false, false); +} |