LGTM3

On 5/31/24 4:07 AM, Yoav Weiss (@Shopify) wrote:
LGTM2

On Fri, May 31, 2024 at 7:35 AM Domenic Denicola <dome...@chromium.org> wrote:

    In that case, LGTM1.

    On Fri, May 31, 2024 at 2:31 PM François Beaufort
    <fbeauf...@google.com> wrote:



        On Fri, May 31, 2024 at 3:16 AM Domenic Denicola
        <dome...@chromium.org> wrote:



            On Thursday, May 30, 2024 at 1:34:03 PM UTC+9
            fbea...@google.com wrote:

                Contact emailsfbeauf...@google.com

                ExplainerNone

                
Specificationhttps://gpuweb.github.io/gpuweb/#dom-gpuadapter-info
                <https://gpuweb.github.io/gpuweb/#dom-gpuadapter-info>

                Summary

                Functionality added to the WebGPU spec after its first
                shipment in a browser. Adds a synchronous GPUAdapter
                info attribute to retrieve the same information about
                the physical adapter as with the asynchronous
                GPUAdapter requestAdapterInfo() method. A separate
                Intent will be sent to deprecate and remove the
                asynchronous GPUAdapter requestAdapterInfo() method.



                Blink componentBlink>WebGPU
                
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EWebGPU>

                TAG reviewNone

                TAG review statusNot applicable

                Risks


                Interoperability and Compatibility

                The GPUAdapter info attribute has not yet been
                implemented in any browser. It has been approved by
                the GPU for the Web Community Group, with
                representatives from Chrome, Firefox, and Safari. See
                minutes at
                
https://github.com/gpuweb/gpuweb/wiki/GPU-Web-2024-05-22#add-synchronous-gpuadapterinfo-4550
                
<https://github.com/gpuweb/gpuweb/wiki/GPU-Web-2024-05-22#add-synchronous-gpuadapterinfo-4550>




                /Gecko/: No signal
                (https://github.com/mozilla/standards-positions/issues/1033
                <https://github.com/mozilla/standards-positions/issues/1033>)


                /WebKit/: Closed Without a Position
                
(https://github.com/WebKit/standards-positions/issues/294#issuecomment-1877411933
                
<https://github.com/WebKit/standards-positions/issues/294#issuecomment-1877411933>)

                /Web developers/: Positive
                (https://github.com/gpuweb/gpuweb/issues/4536
                <https://github.com/gpuweb/gpuweb/issues/4536>)

                /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)?No

                All platforms will eventually have support. Will
                immediately be available on Android, Android WebView,
                ChromeOS, Mac, and Windows, since those platforms
                already support WebGPU. Linux is planned to have
                WebGPU support in the future, so this feature will
                become available when WebGPU does.



                Is this feature fully tested by web-platform-tests
                
<https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>?Yes


                See https://github.com/gpuweb/cts/pull/3679
                <https://github.com/gpuweb/cts/pull/3679> WebGPU/WGSL
                have a conformance test suite
                (https://github.com/gpuweb/cts) that is regularly
                pulled into Chromium and part of the testing of
                Dawn/Tint in Chromium.



                Flag name on chrome://flagsNone

                Finch feature nameNone


            Per
            
https://chromium.googlesource.com/chromium/src/+/main/docs/flag_guarding_guidelines.md#when-is-a-flag-required
            , a feature flag is required for any web platform addition.


        We'll have a blink runtime feature flag named
        "WebGPUAdapterInfoAttribute" for this.
        Thanks for catching Domenic!



                Non-finch justificationNone

                Requires code in //chrome?False

                Tracking
                bughttps://issues.chromium.org/issues/335383516
                <https://issues.chromium.org/issues/335383516>

                Estimated milestonesShipping on desktop127Shipping on
                Android127Shipping on WebView127

                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
                
Statushttps://chromestatus.com/feature/5087914701881344?gate=5141800569536512
                
<https://chromestatus.com/feature/5087914701881344?gate=5141800569536512>

                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/CAM0wra8zntEfiC5SBHcPeOAKV2SOfUM%3DTn9P8Wrq4L3d%3D_pXyQ%40mail.gmail.com
    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra8zntEfiC5SBHcPeOAKV2SOfUM%3DTn9P8Wrq4L3d%3D_pXyQ%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/CAOmohSL53_EFUAGrbT%2B%2BbmnPBM33%2ByuGuVraXwzo_iPOrqPCsg%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOmohSL53_EFUAGrbT%2B%2BbmnPBM33%2ByuGuVraXwzo_iPOrqPCsg%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/4ce61c3c-d774-4cc6-a964-9d70a8c8cbd9%40chromium.org.

Reply via email to