From da212c98f0bba9c35aec3728bbcc13f8f3a7eb52 Mon Sep 17 00:00:00 2001
From: Anne van Kesteren <annevk@annevk.nl>
Date: Fri, 29 Nov 2024 09:21:08 +0100
Subject: [PATCH] Remove redundant domain to Unicode call from valid domain

I'm not entirely sure why this redundant check existed. Either because there was a difference back when this definition was introduced in 3bec3b89c4deb10842ba6c464c700df47c268f17 or (more likely) I wasn't sure if there was a difference.

Fixes #817.
---
 url.bs | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/url.bs b/url.bs
index 2f475eb..c51d9c0 100644
--- a/url.bs
+++ b/url.bs
@@ -952,20 +952,13 @@ concepts.
 <a>valid IPv4-address string</a>, or: U+005B ([), followed by a
 <a>valid IPv6-address string</a>, followed by U+005D (]).
 
-<p>A <var>domain</var> is a <dfn>valid domain</dfn> if these steps return success:
+<p>A <a for=/>string</a> <var>input</var> is a <dfn>valid domain</dfn> if these steps return true:
 
 <ol>
- <li><p>Let <var>result</var> be the result of running <a>domain to ASCII</a> with <var>domain</var>
+ <li><p>Let <var>domain</var> be the result of running <a>domain to ASCII</a> with <var>input</var>
  and true.
 
- <li><p>If <var>result</var> is failure, then return failure.
-
- <li><p>Set <var>result</var> to the result of running <a>domain to Unicode</a> with
- <var>result</var> and true.
-
- <li><p>If <var>result</var> contains any errors, return failure.
-
- <li><p>Return success.
+ <li><p>Return false if <var>domain</var> is failure; otherwise true.
 </ol>
 
 <p class=XXX>Ideally we define this in terms of a sequence of code points that make up a