Allows start, end, justify-all and match-parent as text-align values.

[martin-jackson]

After this change, all values currently listed on MDN for text-align are allowed: https://developer.mozilla.org/en-US/docs/Web/CSS/text-align#values

left and right are not in textAlignLiterals0, but they are already included through azimuthLiterals1:

java-html-sanitizer/owasp-java-html-sanitizer/src/main/java/org/owasp/html/CssSchema.java

Line 315 in f729a08

Set<String> azimuthLiterals1 = j8().setOf("left", "right");

java-html-sanitizer/owasp-java-html-sanitizer/src/main/java/org/owasp/html/CssSchema.java

Lines 714 to 716 in f729a08

	Property textAlign = new Property(
	0, union(azimuthLiterals1, textAlignLiterals0), zeroFns);
	builder.put("text-align", textAlign);

[eivinhb]

LGTM

[martin-jackson]

@mikesamuel Hi! Does this look ok to merge and release? We need to allow the missing values, and I haven't found a way to override the existing text-align rule. The union() method of CssSchema throws an exception when trying to combine the default schema with my own schema:

var updatedTextAlign = Map.of("text-align", new CssSchema.Property(0, Set.of("center", "end", "inherit", "justify", "justify-all", "match-parent", "start", "left", "right"), Collections.emptyMap())); var schema = CssSchema.union( CssSchema.DEFAULT, CssSchema.withProperties(updatedTextAlign) );

java-html-sanitizer/owasp-java-html-sanitizer/src/main/java/org/owasp/html/CssSchema.java

Lines 204 to 207 in f729a08

	if (oldProp != null && !oldProp.equals(newProp)) {
	throw new IllegalArgumentException(
	"Duplicate irreconcilable definitions for " + name);
	}