LGTM1. It seems to have very low risk.
On Fri, Aug 4, 2023 at 4:53 PM Hayato Ito <hay...@chromium.org> wrote: > Contact emailshay...@chromium.org > > Specificationhttps://url.spec.whatwg.org/ > > Summary > > Make URL parser to not decode percent-encoded ASCII characters in URL's > path, such as "%41" ('A'). Before this change: > const url = new URL(" > http://example.com/%41";); > url.href "http://example.com/A"; After this > change: > const url = new URL("http://example.com/%41";); > url.href " > http://example.com/%41"; > > > Blink componentBlink>Network > <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ENetwork> > > TAG reviewNone > > TAG review statusNot applicable > > Risks > > Interoperability and Compatibility > *Gecko*: Shipped/Shipping > > *WebKit*: Shipped/Shipping > > > There are risks. Please see the WIP CL's description for details ( > https://crrev.com/c/4607744). > I'd like to collect feedback about possible risks widely through this > thread. > > The usage (Canary): 0.000106% (URL.Path.UnescapeEscapedChar > <https://uma.googleplex.com/p/chrome/timeline_v2/?sid=1bb9e227dc4889fd2efbf5755d256c62> > ). > This usage is not specific to any particular use case and can be > considered a theoretical upper bound. The actual breakage is likely much > lower than this number. > > Will this feature be supported on all six Blink platforms (Windows, Mac, > Linux, Chrome OS, Android, and Android WebView)?Yes > > Is this feature fully tested by web-platform-tests > <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> > ?Yes > > > Tracking bughttps://crbug.com/1252531 > > Link to entry on the Chrome Platform Status > https://chromestatus.com/feature/6389236793606144 > > -- > Hayato > > -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFpjS_2-4PAY47VbDdd%2BHS%2BchmNUc9dW3BsRtW33LDr1QOeLGw%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFpjS_2-4PAY47VbDdd%2BHS%2BchmNUc9dW3BsRtW33LDr1QOeLGw%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- TAMURA, Kent Software Engineer, Google -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqHcCModrUAQ%3DGJx-oiLcEmBwi%2BjU0ONCpnNWh%3Dp_THRdg%40mail.gmail.com.
