summaryrefslogtreecommitdiff
path: root/xfa/fxgraphics/cfx_color.cpp
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-02-23 13:30:34 -0500
committerChromium commit bot <commit-bot@chromium.org>2017-02-23 18:53:44 +0000
commitb9c69ab7f1338684ffddbc1076052c546f1a95ad (patch)
tree59e5ecfcfaf1ae28956b725168854c1fb8adf684 /xfa/fxgraphics/cfx_color.cpp
parentefcf362b6658346ae0f8f3cdb73d871fdd82c8af (diff)
downloadpdfium-b9c69ab7f1338684ffddbc1076052c546f1a95ad.tar.xz
Cleanup CFX_Graphics.
This Cl removes unused methods, removes unused return values, removes unused hatch data. Change-Id: Id76240170a3bd1c3ef156fbda25a6ac96462cbda Reviewed-on: https://pdfium-review.googlesource.com/2827 Commit-Queue: dsinclair <dsinclair@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxgraphics/cfx_color.cpp')
-rw-r--r--xfa/fxgraphics/cfx_color.cpp13
1 files changed, 5 insertions, 8 deletions
diff --git a/xfa/fxgraphics/cfx_color.cpp b/xfa/fxgraphics/cfx_color.cpp
index 2044b76054..36025a82c8 100644
--- a/xfa/fxgraphics/cfx_color.cpp
+++ b/xfa/fxgraphics/cfx_color.cpp
@@ -24,26 +24,23 @@ CFX_Color::~CFX_Color() {
m_type = FX_COLOR_None;
}
-FWL_Error CFX_Color::Set(const FX_ARGB argb) {
+void CFX_Color::Set(const FX_ARGB argb) {
m_type = FX_COLOR_Solid;
m_info.argb = argb;
m_info.pattern = nullptr;
- return FWL_Error::Succeeded;
}
-FWL_Error CFX_Color::Set(CFX_Pattern* pattern, const FX_ARGB argb) {
+void CFX_Color::Set(CFX_Pattern* pattern, const FX_ARGB argb) {
if (!pattern)
- return FWL_Error::ParameterInvalid;
+ return;
m_type = FX_COLOR_Pattern;
m_info.argb = argb;
m_info.pattern = pattern;
- return FWL_Error::Succeeded;
}
-FWL_Error CFX_Color::Set(CFX_Shading* shading) {
+void CFX_Color::Set(CFX_Shading* shading) {
if (!shading)
- return FWL_Error::ParameterInvalid;
+ return;
m_type = FX_COLOR_Shading;
m_shading = shading;
- return FWL_Error::Succeeded;
}