diff options
author | 2016-09-14 12:04:30 -0700 | |
---|---|---|
committer | 2016-09-14 12:04:30 -0700 | |
commit | e1baa7c105dad0f301ce34e5d6d1aa329334ef8c (patch) | |
tree | c79b25d4ed835bffccd2cbac9a6da433f1e51245 /experimental/svg/model/SkSVGDOM.cpp | |
parent | be362774f9b9e8964544a579281603ed995e6e5a (diff) |
[SVGDom] Expose intrinsic size info
* expose intrinsic size info on <svg> nodes.
* tweak the SkSVGDOM constructor to no longer take an container size
param, but instead default to intrinsic size
* update clients to call SkSVGDOM::setContainerSize() explicitly, when
needed
R=robertphillips@google.com,stephana@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2345533002
Review-Url: https://codereview.chromium.org/2345533002
Diffstat (limited to 'experimental/svg/model/SkSVGDOM.cpp')
-rw-r--r-- | experimental/svg/model/SkSVGDOM.cpp | 29 |
1 files changed, 23 insertions, 6 deletions
diff --git a/experimental/svg/model/SkSVGDOM.cpp b/experimental/svg/model/SkSVGDOM.cpp index f467c5e6d0..c9745f1730 100644 --- a/experimental/svg/model/SkSVGDOM.cpp +++ b/experimental/svg/model/SkSVGDOM.cpp @@ -384,26 +384,29 @@ sk_sp<SkSVGNode> construct_svg_node(const SkDOM& dom, const ConstructionContext& } // anonymous namespace -SkSVGDOM::SkSVGDOM(const SkSize& containerSize) - : fContainerSize(containerSize) { +SkSVGDOM::SkSVGDOM() + : fContainerSize(SkSize::Make(0, 0)) { } -sk_sp<SkSVGDOM> SkSVGDOM::MakeFromDOM(const SkDOM& xmlDom, const SkSize& containerSize) { - sk_sp<SkSVGDOM> dom = sk_make_sp<SkSVGDOM>(containerSize); +sk_sp<SkSVGDOM> SkSVGDOM::MakeFromDOM(const SkDOM& xmlDom) { + sk_sp<SkSVGDOM> dom = sk_make_sp<SkSVGDOM>(); ConstructionContext ctx(&dom->fIDMapper); dom->fRoot = construct_svg_node(xmlDom, ctx, xmlDom.getRootNode()); + // Reset the default container size to match the intrinsic SVG size. + dom->setContainerSize(dom->intrinsicSize()); + return dom; } -sk_sp<SkSVGDOM> SkSVGDOM::MakeFromStream(SkStream& svgStream, const SkSize& containerSize) { +sk_sp<SkSVGDOM> SkSVGDOM::MakeFromStream(SkStream& svgStream) { SkDOM xmlDom; if (!xmlDom.build(svgStream)) { return nullptr; } - return MakeFromDOM(xmlDom, containerSize); + return MakeFromDOM(xmlDom); } void SkSVGDOM::render(SkCanvas* canvas) const { @@ -416,6 +419,20 @@ void SkSVGDOM::render(SkCanvas* canvas) const { } } +SkSize SkSVGDOM::intrinsicSize() const { + if (!fRoot || fRoot->tag() != SkSVGTag::kSvg) { + return SkSize::Make(0, 0); + } + + // Intrinsic sizes are never relative, so the viewport size is irrelevant. + const SkSVGLengthContext lctx(SkSize::Make(0, 0)); + return static_cast<const SkSVGSVG*>(fRoot.get())->intrinsicSize(lctx); +} + +const SkSize& SkSVGDOM::containerSize() const { + return fContainerSize; +} + void SkSVGDOM::setContainerSize(const SkSize& containerSize) { // TODO: inval fContainerSize = containerSize; |