[svg] Convert text-anchor to a presentation attribute
https://www.w3.org/TR/SVG11/text.html#TextAnchorProperty
Bug: skia:10840
Change-Id: Iff647b62243c42150e873f06215401b5e33705fd
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/330125
Commit-Queue: Florin Malita <fmalita@google.com>
Reviewed-by: Tyler Denniston <tdenniston@google.com>
diff --git a/modules/svg/src/SkSVGAttribute.cpp b/modules/svg/src/SkSVGAttribute.cpp
index 0ccea49..c9acb46 100644
--- a/modules/svg/src/SkSVGAttribute.cpp
+++ b/modules/svg/src/SkSVGAttribute.cpp
@@ -31,7 +31,8 @@
result.fFontFamily.init("Sans");
result.fFontStyle.init(SkSVGFontStyle::Type::kNormal);
result.fFontSize.init(SkSVGLength(24));
- result.fFontWeight.init(SkSVGFontWeight(SkSVGFontWeight::Type::kNormal));
+ result.fFontWeight.init(SkSVGFontWeight::Type::kNormal);
+ result.fTextAnchor.init(SkSVGTextAnchor::Type::kStart);
return result;
}
diff --git a/modules/svg/src/SkSVGAttributeParser.cpp b/modules/svg/src/SkSVGAttributeParser.cpp
index a56e8e7..05c585b 100644
--- a/modules/svg/src/SkSVGAttributeParser.cpp
+++ b/modules/svg/src/SkSVGAttributeParser.cpp
@@ -822,6 +822,26 @@
return parsedValue && this->parseEOSToken();
}
+// https://www.w3.org/TR/SVG11/text.html#TextAnchorProperty
+bool SkSVGAttributeParser::parseTextAnchor(SkSVGTextAnchor* anchor) {
+ static constexpr std::tuple<const char*, SkSVGTextAnchor::Type> gAnchorMap[] = {
+ { "start" , SkSVGTextAnchor::Type::kStart },
+ { "middle" , SkSVGTextAnchor::Type::kMiddle },
+ { "end" , SkSVGTextAnchor::Type::kEnd },
+ { "inherit", SkSVGTextAnchor::Type::kInherit},
+ };
+
+ bool parsedValue = false;
+ SkSVGTextAnchor::Type type;
+
+ if (this->parseEnumMap(gAnchorMap, &type)) {
+ *anchor = SkSVGTextAnchor(type);
+ parsedValue = true;
+ }
+
+ return parsedValue && this->parseEOSToken();
+}
+
// https://www.w3.org/TR/SVG11/coords.html#PreserveAspectRatioAttribute
bool SkSVGAttributeParser::parsePreserveAspectRatio(SkSVGPreserveAspectRatio* par) {
static constexpr std::tuple<const char*, SkSVGPreserveAspectRatio::Align> gAlignMap[] = {
diff --git a/modules/svg/src/SkSVGDOM.cpp b/modules/svg/src/SkSVGDOM.cpp
index b989b70..f9327a3 100644
--- a/modules/svg/src/SkSVGDOM.cpp
+++ b/modules/svg/src/SkSVGDOM.cpp
@@ -308,6 +308,19 @@
return true;
}
+bool SetTextAnchorAttribute(const sk_sp<SkSVGNode>& node, SkSVGAttribute attr,
+ const char* stringValue) {
+ SkSVGTextAnchor anchor;
+ SkSVGAttributeParser parser(stringValue);
+
+ if (!parser.parseTextAnchor(&anchor)) {
+ return false;
+ }
+
+ node->setAttribute(attr, SkSVGTextAnchorValue(anchor));
+ return true;
+}
+
bool SetPreserveAspectRatioAttribute(const sk_sp<SkSVGNode>& node, SkSVGAttribute attr,
const char* stringValue) {
SkSVGPreserveAspectRatio par;
@@ -439,7 +452,7 @@
{ "stroke-width" , { SkSVGAttribute::kStrokeWidth , SetLengthAttribute }},
{ "style" , { SkSVGAttribute::kUnknown , SetStyleAttributes }},
{ "text" , { SkSVGAttribute::kText , SetStringAttribute }},
- { "text-anchor" , { SkSVGAttribute::kTextAnchor , SetStringAttribute }},
+ { "text-anchor" , { SkSVGAttribute::kTextAnchor , SetTextAnchorAttribute }},
{ "transform" , { SkSVGAttribute::kTransform , SetTransformAttribute }},
{ "viewBox" , { SkSVGAttribute::kViewBox , SetViewBoxAttribute }},
{ "visibility" , { SkSVGAttribute::kVisibility , SetVisibilityAttribute }},
diff --git a/modules/svg/src/SkSVGNode.cpp b/modules/svg/src/SkSVGNode.cpp
index 07f3a47..21b5a76 100644
--- a/modules/svg/src/SkSVGNode.cpp
+++ b/modules/svg/src/SkSVGNode.cpp
@@ -238,6 +238,11 @@
this->setStrokeWidth(*strokeWidth);
}
break;
+ case SkSVGAttribute::kTextAnchor:
+ if (const SkSVGTextAnchorValue* anchor = v.as<SkSVGTextAnchorValue>()) {
+ this->setTextAnchor(*anchor);
+ }
+ break;
case SkSVGAttribute::kVisibility:
if (const SkSVGVisibilityValue* visibility = v.as<SkSVGVisibilityValue>()) {
this->setVisibility(*visibility);
diff --git a/modules/svg/src/SkSVGRenderContext.cpp b/modules/svg/src/SkSVGRenderContext.cpp
index 4b1b93b..a1b7deb 100644
--- a/modules/svg/src/SkSVGRenderContext.cpp
+++ b/modules/svg/src/SkSVGRenderContext.cpp
@@ -302,6 +302,13 @@
// Not part of the SkPaint state; applied at render time.
}
+template <>
+void commitToPaint<SkSVGAttribute::kTextAnchor>(const SkSVGPresentationAttributes&,
+ const SkSVGRenderContext&,
+ SkSVGPresentationContext*) {
+ // Not part of the SkPaint state; applied at render time.
+}
+
} // namespace
SkSVGPresentationContext::SkSVGPresentationContext()
@@ -403,6 +410,7 @@
ApplyLazyInheritedAttribute(StrokeMiterLimit);
ApplyLazyInheritedAttribute(StrokeOpacity);
ApplyLazyInheritedAttribute(StrokeWidth);
+ ApplyLazyInheritedAttribute(TextAnchor);
ApplyLazyInheritedAttribute(Visibility);
ApplyLazyInheritedAttribute(Color);
diff --git a/modules/svg/src/SkSVGText.cpp b/modules/svg/src/SkSVGText.cpp
index 4dde8b8..6cbf5d6 100644
--- a/modules/svg/src/SkSVGText.cpp
+++ b/modules/svg/src/SkSVGText.cpp
@@ -16,22 +16,6 @@
SkSVGText::SkSVGText() : INHERITED(SkSVGTag::kText) {}
-void SkSVGText::setX(const SkSVGLength& x) { fX = x; }
-
-void SkSVGText::setY(const SkSVGLength& y) { fY = y; }
-
-void SkSVGText::setText(const SkSVGStringType& text) { fText = text; }
-
-void SkSVGText::setTextAnchor(const SkSVGStringType& text_anchor) {
- if (strcmp(text_anchor.c_str(), "start") == 0) {
- fTextAlign = SkTextUtils::Align::kLeft_Align;
- } else if (strcmp(text_anchor.c_str(), "middle") == 0) {
- fTextAlign = SkTextUtils::Align::kCenter_Align;
- } else if (strcmp(text_anchor.c_str(), "end") == 0) {
- fTextAlign = SkTextUtils::Align::kRight_Align;
- }
-}
-
SkFont SkSVGText::resolveFont(const SkSVGRenderContext& ctx) const {
auto weight = [](const SkSVGFontWeight& w) {
switch (w.type()) {
@@ -92,14 +76,27 @@
void SkSVGText::onRender(const SkSVGRenderContext& ctx) const {
const auto font = this->resolveFont(ctx);
+ const auto text_align = [](const SkSVGTextAnchor& anchor) {
+ switch (anchor.type()) {
+ case SkSVGTextAnchor::Type::kStart : return SkTextUtils::Align::kLeft_Align;
+ case SkSVGTextAnchor::Type::kMiddle: return SkTextUtils::Align::kCenter_Align;
+ case SkSVGTextAnchor::Type::kEnd : return SkTextUtils::Align::kRight_Align;
+ case SkSVGTextAnchor::Type::kInherit:
+ SkASSERT(false);
+ return SkTextUtils::Align::kLeft_Align;
+ }
+ SkUNREACHABLE;
+ };
+
+ const auto align = text_align(*ctx.presentationContext().fInherited.fTextAnchor);
if (const SkPaint* fillPaint = ctx.fillPaint()) {
SkTextUtils::DrawString(ctx.canvas(), fText.c_str(), fX.value(), fY.value(), font,
- *fillPaint, fTextAlign);
+ *fillPaint, align);
}
if (const SkPaint* strokePaint = ctx.strokePaint()) {
SkTextUtils::DrawString(ctx.canvas(), fText.c_str(), fX.value(), fY.value(), font,
- *strokePaint, fTextAlign);
+ *strokePaint, align);
}
}
@@ -129,12 +126,6 @@
this->setText(*text);
}
break;
- case SkSVGAttribute::kTextAnchor:
- if (const auto* text_anchor = v.as<SkSVGStringValue>()) {
- this->setTextAnchor(*text_anchor);
- }
- break;
- break;
default:
this->INHERITED::onSetAttribute(attr, v);
}