diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-12-13 18:29:02 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-12-13 18:29:02 +0000 |
commit | 2614250d4f11ed02033a7da095a1694ece12ab78 (patch) | |
tree | cc41c5770c17ea47537856991fa44cd1e2acfeb8 /xfa/fxfa/parser/cxfa_containerlayoutitem.cpp | |
parent | 14768f3264a4cff8ca0096a27e41f1861b2c422b (diff) | |
download | pdfium-2614250d4f11ed02033a7da095a1694ece12ab78.tar.xz |
Methods are on CJX_Object not CJX_Node now
This CL converts all of the JSNode() to JSObject() calls as all of the
original CJX_Node methods have been moved to CJX_Object. This fixes
potential bad casts from things like CJX_Content which do not inhert
from CJX_Node.
Bug: pdfium:793372
Change-Id: I6c7f63a78f3d47bb6bad74faed3fd8c535bf095e
Reviewed-on: https://pdfium-review.googlesource.com/21090
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_containerlayoutitem.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_containerlayoutitem.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/cxfa_containerlayoutitem.cpp b/xfa/fxfa/parser/cxfa_containerlayoutitem.cpp index 07203caa85..28c0c2eedd 100644 --- a/xfa/fxfa/parser/cxfa_containerlayoutitem.cpp +++ b/xfa/fxfa/parser/cxfa_containerlayoutitem.cpp @@ -31,10 +31,13 @@ CFX_SizeF CXFA_ContainerLayoutItem::GetPageSize() const { if (!pMedium) return size; - size = CFX_SizeF( - pMedium->JSNode()->GetMeasure(XFA_Attribute::Short).ToUnit(XFA_Unit::Pt), - pMedium->JSNode()->GetMeasure(XFA_Attribute::Long).ToUnit(XFA_Unit::Pt)); - if (pMedium->JSNode()->GetEnum(XFA_Attribute::Orientation) == + size = CFX_SizeF(pMedium->JSObject() + ->GetMeasure(XFA_Attribute::Short) + .ToUnit(XFA_Unit::Pt), + pMedium->JSObject() + ->GetMeasure(XFA_Attribute::Long) + .ToUnit(XFA_Unit::Pt)); + if (pMedium->JSObject()->GetEnum(XFA_Attribute::Orientation) == XFA_AttributeEnum::Landscape) { size = CFX_SizeF(size.height, size.width); } |