Add fetchpriority attribute

NiedziolkaMichal · sideshowbarker · commit 7c9991405e7b · 2023-02-27T18:43:29.000+09:00
diff --git a/schema/html5/common.rnc b/schema/html5/common.rnc
@@ -216,6 +216,11 @@ common.attrs.other =
 			list { w:string "render" }
 		}
 
+	common.attrs.fetchpriority =
+		attribute fetchpriority {
+			w:string "high" | w:string "low" | w:string "auto"
+		}
+
 	common.attrs.nonce =
 		attribute nonce {
 			string
diff --git a/schema/html5/core-scripting.rnc b/schema/html5/core-scripting.rnc
@@ -34,6 +34,7 @@ datatypes w = "http://whattf.org/datatype-draft"
 		&	script.attrs.integrity?
 		&	embedded.content.attrs.crossorigin?
 		&	common.attrs.blocking?
+		&	common.attrs.fetchpriority?
 		&	referrerpolicy?
 		)
 		script.attrs.src =
diff --git a/schema/html5/embed.rnc b/schema/html5/embed.rnc
@@ -22,6 +22,7 @@ namespace local = ""
 		&	img.attrs.ismap?
 		&	img.attrs.decoding?
 		&	img.attrs.loading?
+		&	common.attrs.fetchpriority?
 		&	img.attrs.border? # obsolete
 		&	referrerpolicy?
 		&	embedded.content.attrs.crossorigin?
diff --git a/schema/html5/meta.rnc b/schema/html5/meta.rnc
@@ -164,6 +164,7 @@ datatypes w = "http://whattf.org/datatype-draft"
 		&	link.attrs.workertype?
 		&	link.attrs.imagesrcset?
 		&	link.attrs.imagesizes?
+		&	common.attrs.fetchpriority?
 		#	link.attrs.title included in common.attrs
 		&	embedded.content.attrs.crossorigin?
 		&	common.attrs.blocking?
diff --git a/src/nu/validator/checker/schematronequiv/Assertions.java b/src/nu/validator/checker/schematronequiv/Assertions.java
@@ -678,6 +678,13 @@ private static void registerRequiredAncestorRole(String parent,
         ATTRIBUTES_WITH_IMPLICIT_STATE_OR_PROPERTY.add("required");
         ATTRIBUTES_WITH_IMPLICIT_STATE_OR_PROPERTY.add("rowspan");
     }
+    
+    /**
+     * Names of link types that are considered as "External Resource" according to https://html.spec.whatwg.org/multipage/links.html#linkTypes
+     */
+    private static final String[] EXTERNAL_RESOURCE_LINK_REL = new String[] {
+            "dns-prefetch", "icon", "manifest", "modulepreload", "pingback", "preconnect", "prefetch", "preload", "prerender", "stylesheet"
+    };
 
     private static final String h1WarningMessage = "Consider using the"
             + " \u201Ch1\u201D element as a top-level heading only (all"
@@ -2908,6 +2915,11 @@ else if ("script" == localName) {
                                 + " \u201Cintegrity\u201D unless attribute"
                                 + " \u201Csrc\u201D is also specified.");
                     }
+                    if (atts.getIndex("", "fetchpriority") > -1) {
+                        warn("Element \u201Cscript\u201D should not have attribute"
+                                     + " \u201Cfetchpriority\u201D unless attribute"
+                                     + " \u201Csrc\u201D is also specified.");
+                    }
                 }
                 if (atts.getIndex("", "type") > -1) {
                     String scriptType = atts.getValue("", "type").toLowerCase();
@@ -3365,6 +3377,12 @@ else if ("bdo" == localName && atts.getIndex("", "dir") < 0) {
                                 + " \u201Ctitle\u201D attribute with a"
                                 + " non-empty value.");
                 }
+                if (atts.getIndex("", "fetchpriority") > -1
+                        && Arrays.stream(EXTERNAL_RESOURCE_LINK_REL).noneMatch(relList::contains)) {
+                    warn("A \u201Clink\u201D element with \u201Cfetchpriority\u201D"
+                                + " attribute should have a \u201Crel\u201D"
+                                + " attribute containing external resource type.");
+                }
                 if ((ancestorMask & BODY_MASK) != 0
                         && (relList != null
                                 && !(relList.contains("dns-prefetch")

Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,7 @@ datatypes w = "http://whattf.org/datatype-draft"`
`34`	`34`	`& script.attrs.integrity?`
`35`	`35`	`& embedded.content.attrs.crossorigin?`
`36`	`36`	`& common.attrs.blocking?`
	`37`	`+ & common.attrs.fetchpriority?`
`37`	`38`	`& referrerpolicy?`
`38`	`39`	`)`
`39`	`40`	`script.attrs.src =`