diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-03 12:01:24 -0500 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-03 18:20:01 +0000 |
commit | 39628b04bc86461426f2dcf5a568f5e9e70ad997 (patch) | |
tree | a0cf6c494a74b83e93a5cc526ab5897a08e3d743 /fxjs/xfa/cjx_object.cpp | |
parent | 9647614a5dcd00d50aacf4e000fd23a5ebb13931 (diff) | |
download | pdfium-39628b04bc86461426f2dcf5a568f5e9e70ad997.tar.xz |
Change CXFA_BoxData to CXFA_Box
This CL renames CXFA_BoxData to CXFA_Box and sets it to inhert from
CXFA_Node instead of CXFA_DataData. The CXFA_BoxData subclasses,
CXFA_RectangleData, CXFA_BorderData and CXFA_ArcData classes are removed
and the nodes now inherit from CXFA_Box.
Change-Id: Ia0df7d56535b5d90be29f040180a5d5fc55a64e0
Reviewed-on: https://pdfium-review.googlesource.com/22091
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/xfa/cjx_object.cpp')
-rw-r--r-- | fxjs/xfa/cjx_object.cpp | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/fxjs/xfa/cjx_object.cpp b/fxjs/xfa/cjx_object.cpp index b1324f1d5f..aa9a16956e 100644 --- a/fxjs/xfa/cjx_object.cpp +++ b/fxjs/xfa/cjx_object.cpp @@ -21,7 +21,9 @@ #include "third_party/base/ptr_util.h" #include "xfa/fxfa/cxfa_ffnotify.h" #include "xfa/fxfa/cxfa_ffwidget.h" +#include "xfa/fxfa/parser/cxfa_border.h" #include "xfa/fxfa/parser/cxfa_document.h" +#include "xfa/fxfa/parser/cxfa_filldata.h" #include "xfa/fxfa/parser/cxfa_layoutprocessor.h" #include "xfa/fxfa/parser/cxfa_measurement.h" #include "xfa/fxfa/parser/cxfa_node.h" @@ -1335,8 +1337,8 @@ void CJX_Object::Script_Som_FillColor(CFXJSE_Value* pValue, if (!widget_data_) return; - CXFA_BorderData borderData = widget_data_->GetBorderData(true); - CXFA_FillData borderfillData = borderData.GetFillData(true); + CXFA_Border* border = widget_data_->GetBorder(true); + CXFA_FillData borderfillData = border->GetFillData(true); CXFA_Node* pNode = borderfillData.GetNode(); if (!pNode) return; @@ -1367,8 +1369,8 @@ void CJX_Object::Script_Som_BorderColor(CFXJSE_Value* pValue, if (!widget_data_) return; - CXFA_BorderData borderData = widget_data_->GetBorderData(true); - int32_t iSize = borderData.CountEdges(); + CXFA_Border* border = widget_data_->GetBorder(true); + int32_t iSize = border->CountEdges(); if (bSetting) { int32_t r = 0; int32_t g = 0; @@ -1376,12 +1378,12 @@ void CJX_Object::Script_Som_BorderColor(CFXJSE_Value* pValue, std::tie(r, g, b) = StrToRGB(pValue->ToWideString()); FX_ARGB rgb = ArgbEncode(100, r, g, b); for (int32_t i = 0; i < iSize; ++i) - borderData.GetEdgeData(i).SetColor(rgb); + border->GetEdgeData(i).SetColor(rgb); return; } - FX_ARGB color = borderData.GetEdgeData(0).GetColor(); + FX_ARGB color = border->GetEdgeData(0).GetColor(); int32_t a; int32_t r; int32_t g; @@ -1397,16 +1399,16 @@ void CJX_Object::Script_Som_BorderWidth(CFXJSE_Value* pValue, if (!widget_data_) return; - CXFA_BorderData borderData = widget_data_->GetBorderData(true); + CXFA_Border* border = widget_data_->GetBorder(true); if (bSetting) { - CXFA_Measurement thickness = borderData.GetEdgeData(0).GetMSThickness(); + CXFA_Measurement thickness = border->GetEdgeData(0).GetMSThickness(); pValue->SetString(thickness.ToString().UTF8Encode().AsStringView()); return; } WideString wsThickness = pValue->ToWideString(); - for (int32_t i = 0; i < borderData.CountEdges(); ++i) { - borderData.GetEdgeData(i).SetMSThickness( + for (int32_t i = 0; i < border->CountEdges(); ++i) { + border->GetEdgeData(i).SetMSThickness( CXFA_Measurement(wsThickness.AsStringView())); } } |