LGTM3

On Wed, Apr 24, 2024 at 4:46 PM Jxck <j...@chromium.org> wrote:

> @domenic
>
> I've updated finch name on ChromeStatus.
>
> Thanks
> Jxck
>
> On Wednesday, April 24, 2024 at 12:45:52 PM UTC+9 Domenic Denicola wrote:
>
>> LGTM2
>>
>> On Wed, Apr 24, 2024 at 1:50 AM Yoav Weiss (@Shopify) <
>> yoavwe...@chromium.org> wrote:
>>
>>> LGTM1
>>>
>>> Thanks for catching us up here! :)
>>>
>>> On Tue, Apr 23, 2024 at 10:20 AM Domenic Denicola <dome...@chromium.org>
>>> wrote:
>>>
>>>> Thanks. One more thing: our process requires all features have a Finch
>>>> feature name (i.e., a base::Feature) so that we can turn off the feature
>>>> remotely. Can you ensure your implementation has one, and then update
>>>> ChromeStatus with its name?
>>>>
>>>
>>> Looks like a flag
>>> <https://chromium-review.googlesource.com/c/chromium/src/+/5414853/14/third_party/blink/renderer/platform/runtime_enabled_features.json5>
>>> is already in place, but indeed please update the chromestatus entry with
>>> that flag's name.
>>>
>>>
>>>>
>>>> On Tue, Apr 23, 2024 at 2:29 PM Jxck <j...@chromium.org> wrote:
>>>>
>>>>> Thanks, I've done them.
>>>>>
>>>>> On Monday, April 22, 2024 at 3:53:27 PM UTC+9 Domenic Denicola wrote:
>>>>>
>>>>>> Can you request privacy, security, enterprise, debuggability, and
>>>>>> testing reviews on Chrome Status?
>>>>>>
>>>>>> On Thu, Apr 18, 2024 at 11:36 PM Jack J <j...@chromium.org> wrote:
>>>>>>
>>>>>>> Contact emailsj...@chromium.org
>>>>>>>
>>>>>>> ExplainerNone
>>>>>>>
>>>>>>> Specificationhttps://url.spec.whatwg.org/#dom-url-parse
>>>>>>>
>>>>>>> Design docs
>>>>>>> https://github.com/whatwg/url/pull/825
>>>>>>>
>>>>>>> Summary
>>>>>>>
>>>>>>> Parsing url string and returning URL object. If it's invalid url
>>>>>>> format, returning null instead of throwing Error.
>>>>>>>
>>>>>>>
>>>>>>> Blink componentBlink>Network
>>>>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ENetwork>
>>>>>>>
>>>>>>> Search tagsURL <https://chromestatus.com/features#tags:URL>
>>>>>>>
>>>>>>> TAG reviewNone
>>>>>>>
>>>>>>> TAG review statusNot applicable
>>>>>>>
>>>>>>> Risks
>>>>>>>
>>>>>>>
>>>>>>> Interoperability and Compatibility
>>>>>>>
>>>>>>> None
>>>>>>>
>>>>>>>
>>>>>>> *Gecko*: Shipped/Shipping (
>>>>>>> https://bugzilla.mozilla.org/show_bug.cgi?id=1887611)
>>>>>>>
>>>>>>> *WebKit*: Shipped/Shipping (
>>>>>>> https://github.com/WebKit/WebKit/pull/26403)
>>>>>>>
>>>>>>> *Web developers*: Positive (
>>>>>>> https://github.com/denoland/deno/issues/23069)
>>>>>>> https://github.com/nodejs/node/issues/52208
>>>>>>>
>>>>>>> *Other signals*:
>>>>>>>
>>>>>>> WebView application risks
>>>>>>>
>>>>>>> Does this intent deprecate or change behavior of existing APIs, such
>>>>>>> that it has potentially high risk for Android WebView-based 
>>>>>>> applications?
>>>>>>>
>>>>>>> None
>>>>>>>
>>>>>>>
>>>>>>> Debuggability
>>>>>>>
>>>>>>> None
>>>>>>>
>>>>>>>
>>>>>>> Will this feature be supported on all six Blink platforms (Windows,
>>>>>>> Mac, Linux, ChromeOS, 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
>>>>>>>
>>>>>>>
>>>>>>> https://wpt.fyi/results/url?label=master&label=experimental&aligned&q=url-statics-parse
>>>>>>>
>>>>>>>
>>>>>>> Flag name on chrome://flags
>>>>>>> chrome://flags/#enable-experimental-web-platform-features
>>>>>>>
>>>>>>> Finch feature nameNone
>>>>>>>
>>>>>>> Non-finch justificationNone
>>>>>>>
>>>>>>> Requires code in //chrome?False
>>>>>>>
>>>>>>> Tracking bughttps://issues.chromium.org/issues/331041242
>>>>>>>
>>>>>>> Estimated milestones
>>>>>>> Shipping on desktop 126
>>>>>>> Shipping on Android 126
>>>>>>> Shipping on WebView 126
>>>>>>> Shipping on WebView 126
>>>>>>>
>>>>>>> Anticipated spec changes
>>>>>>>
>>>>>>> Open questions about a feature may be a source of future web compat
>>>>>>> or interop issues. Please list open issues (e.g. links to known github
>>>>>>> issues in the project for the feature specification) whose resolution 
>>>>>>> may
>>>>>>> introduce web compat/interop risk (e.g., changing to naming or 
>>>>>>> structure of
>>>>>>> the API in a non-backward-compatible way).
>>>>>>> None
>>>>>>>
>>>>>>> Link to entry on the Chrome Platform Status
>>>>>>> https://chromestatus.com/feature/6301071388704768
>>>>>>>
>>>>>>> This intent message was generated by Chrome Platform Status
>>>>>>> <https://chromestatus.com/>.
>>>>>>>
>>>>>>> --
>>>>>>> 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/CAFBcQZ--hCtq-W37%2BkXCf89uKdEQ6Ot8RAdDO5LFXd7qReY_xw%40mail.gmail.com
>>>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFBcQZ--hCtq-W37%2BkXCf89uKdEQ6Ot8RAdDO5LFXd7qReY_xw%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/CAM0wra_800vnfaubD_W4X77V21MYLgwxhYQ-oG0pH_8BuFyk4Q%40mail.gmail.com
>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra_800vnfaubD_W4X77V21MYLgwxhYQ-oG0pH_8BuFyk4Q%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/4aa16455-8d59-41eb-917e-cfc38441bcd6n%40chromium.org
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/4aa16455-8d59-41eb-917e-cfc38441bcd6n%40chromium.org?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/CAARdPYcv8og4iNYJTrGmZwS%2BNn58g%3DnifpDtuQQnN4vekCm1Jg%40mail.gmail.com.

Reply via email to