Thanks for the LGTMs. I appreciate it. Let me implement and ship with a feature flag so that we can have a kill switch.
On Wed, Aug 16, 2023 at 9:47 PM Daniel Bratell <bratel...@gmail.com> wrote: > Compatibility is nice. > > LGTM3, but note Mike's request > > /Daniel > On 2023-08-11 17:15, Mike Taylor wrote: > > I'm having a hard time assessing the risk, despite the very low usage (it > has doubled since the original email was sent - but still very low) and > other browsers shipping it. > > That said, LGTM2 % having a base::Feature we can use as a killswitch, in > case we discover something we didn't anticipate. > On 8/9/23 8:12 PM, 'TAMURA, Kent' via blink-dev wrote: > > LGTM1. It seems to have very low risk. > > > On Fri, Aug 4, 2023 at 4:53 PM Hayato Ito <hay...@chromium.org> wrote: > >> Contact emails hay...@chromium.org >> >> Specification https://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 component Blink>Network >> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ENetwork> >> >> TAG review None >> >> TAG review status Not 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 bug https://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 > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqHcCModrUAQ%3DGJx-oiLcEmBwi%2BjU0ONCpnNWh%3Dp_THRdg%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > > -- > 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/b674e07c-7804-41d5-b8f5-7ada6295651b%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/b674e07c-7804-41d5-b8f5-7ada6295651b%40chromium.org?utm_medium=email&utm_source=footer> > . > > -- 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_3_a5ZoZbPFw4V-CHH2%2BmCdqmQfD8yjQOuC8tJRSt%2B7Gw%40mail.gmail.com.