LGTM3 On Tue, Apr 23, 2024 at 6:45 PM Mike Taylor <miketa...@chromium.org> wrote:
> LGTM2 > On 4/23/24 1:18 AM, Domenic Denicola wrote: > > LGTM1 > > On Tue, Apr 23, 2024 at 7:36 AM Ken Russell <k...@chromium.org> wrote: > >> Yes. N/As and explanations have been requested for all of these now. >> >> >> On Sun, Apr 21, 2024 at 7:37 PM Domenic Denicola <dome...@chromium.org> >> wrote: >> >>> Can you request Privacy, Security, Enterprise, Debuggability, and >>> Testing reviews by following the procedure on ChromeStatus? Most of them >>> should be pretty simple, or even N/A, for this feature. But we're not >>> supposed to give API owner approvals until they're underway. >>> >>> On Sat, Apr 20, 2024 at 12:42 PM Ken Russell <k...@chromium.org> wrote: >>> >>>> Correction: this API change is already tested in the Web Platform >>>> Tests, by wpt/webgl/idlharness.any and wpt/webgl/idlharness.any.worker. >>>> I've updated the Chromestatus entry to indicate this; these tests are >>>> progressed by this change. >>>> >>>> https://chromium-review.googlesource.com/c/chromium/src/+/5466430 , >>>> implementing this, passes the trybots. Would appreciate LGTMs from OWNERs >>>> for this small change to minimize the number of rebases that need to be >>>> done before it lands. >>>> >>>> Thanks in advance. >>>> >>>> -Ken >>>> >>>> >>>> >>>> On Thu, Apr 18, 2024 at 5:45 PM Ken Russell <k...@chromium.org> wrote: >>>> >>>>> Contact emails k...@chromium.org >>>>> >>>>> Explainer None >>>>> >>>>> Specification https://registry.khronos.org/webgl/specs/latest/1.0 >>>>> >>>>> Summary >>>>> >>>>> The WebGL specification has defined a WebGLObject superinterface for >>>>> many years, but Chromium's implementation never exposed it. This did not >>>>> significantly affect applications in practice, but prevented enabling >>>>> WebGL >>>>> IDL tests in the Interop suite. The WebGL working group also aims to >>>>> finally utilize this superinterface to improve application-level >>>>> debugging. >>>>> This feature exposes the WebGLObject type in the same contexts where the >>>>> WebGL API is exposed - on the main thread and workers. >>>>> >>>>> >>>>> Blink component Blink>WebGL >>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EWebGL> >>>>> >>>>> TAG review None >>>>> >>>>> TAG review status Not applicable >>>>> >>>>> Risks >>>>> >>>>> >>>>> Interoperability and Compatibility >>>>> >>>>> No risk. Chrome, Firefox and Safari all intend to expose this >>>>> interface. >>>>> >>>>> >>>>> *Gecko*: Positive (https://github.com/KhronosGroup/WebGL/pull/3637) >>>>> Firefox approves of and proposes using this superinterface to add >>>>> application-level debugging functionality. >>>>> >>>>> *WebKit*: Positive (https://github.com/KhronosGroup/WebGL/pull/3637) >>>>> Safari approves of using this superinterface to add application-level >>>>> debugging functionality. >>>>> >>>>> *Web developers*: Positive ( >>>>> https://github.com/KhronosGroup/WebGL/issues/3514) Web developers >>>>> request debugging functionality built on this superinterface, per the link >>>>> above. >>>>> >>>>> *Other signals*: >>>>> >>>>> Ergonomics >>>>> >>>>> N/A for this change; follow-on ones will improve developer ergonomics >>>>> and integration with platform-specific graphics debugging tools. >>>>> >>>>> >>>>> Activation >>>>> >>>>> N/A for this change. >>>>> >>>>> >>>>> Security >>>>> >>>>> None >>>>> >>>>> >>>>> 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 >>>>> >>>>> N/A for this change. >>>>> >>>>> >>>>> 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> >>>>> ? No >>>>> >>>>> Not tested by web-platform-tests yet. This change enables automated >>>>> WebGL IDL tests, which were previously failing, to be added to the Interop >>>>> suite. It will also be tested in the WebGL conformance suite, which is >>>>> separate from web-platform-tests. >>>>> >>>>> >>>>> Flag name on chrome://flags None >>>>> >>>>> Finch feature name None >>>>> >>>>> Non-finch justification >>>>> >>>>> Trivial addition of a superinterface to multiple WebGL object types. >>>>> >>>>> >>>>> Requires code in //chrome? False >>>>> >>>>> Measurement Does not apply; this is currently an empty web-exposed >>>>> interface which can be used in "instanceof" checks. >>>>> >>>>> Availability expectation Feature is available on Web Platform >>>>> mainline within 12 months of launch in Chrome. >>>>> >>>>> Adoption expectation Feature is considered a best practice for some >>>>> use case within 12 months of reaching Web Platform baseline. >>>>> >>>>> Adoption plan Chrome, Firefox and Safari plan to ship debugging >>>>> functionality for application developers on top of this feature as soon as >>>>> possible. >>>>> >>>>> Non-OSS dependencies >>>>> >>>>> Does the feature depend on any code or APIs outside the Chromium open >>>>> source repository and its open-source dependencies to function? >>>>> None >>>>> >>>>> Estimated milestones >>>>> Shipping on desktop 126 >>>>> Shipping on Android 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 - has been specified for many years >>>>> >>>>> Link to entry on the Chrome Platform Status >>>>> https://chromestatus.com/feature/5119115615535104 >>>>> >>>>> 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/CAMYvS2fATLXkxCJEf1Us8DWDH5XWRcBvjogqZqr22TwotbmJcQ%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAMYvS2fATLXkxCJEf1Us8DWDH5XWRcBvjogqZqr22TwotbmJcQ%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_ajr5HL%2B%3D2MiiScv3d2Ayp%3DwwUZ%2BatQyH-hjS8YXVgqw%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra_ajr5HL%2B%3D2MiiScv3d2Ayp%3DwwUZ%2BatQyH-hjS8YXVgqw%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/e21e8f34-ba12-4db6-8680-1901c98d5f46%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/e21e8f34-ba12-4db6-8680-1901c98d5f46%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/CAOmohSLQeqW29p4%2B0TYNKXoyFeU5XMhPewEPqSnQuhtk5OAO4g%40mail.gmail.com.