LGTM1 On Thu, Oct 28, 2021 at 10:56 PM Andreu Botella <and...@andreubotella.com> wrote:
> I don't think the differences are listed anywhere. I know there are some > because of the failures in > https://wpt.fyi/results/html/infrastructure/safe-passing-of-structured-data?label=experimental&label=master&aligned, > but there might be others that aren't tested. Although it seems like some > of the failures in the shared-array-buffer folder seem to be bugs with > the tests rather than with the implementations. > OK, as these differences are already exposed, I don't think shipping this significantly increases risk. The fact that they're covered by WPTs makes it more likely we'd (eventually) converge on the specified behavior. > On Wednesday, October 27, 2021 at 11:12:32 PM UTC+2 fs...@chromium.org > wrote: > >> This is amazing! :) >> >> I agree it shouldn't block this, but do we have anywhere written what >> are the browser's differences on structured clone algorithms? Is it a spec >> issue? Could we add WPT tests for it? >> >> On Wed, Oct 27, 2021 at 2:45 PM Andreu Botella <and...@andreubotella.com> >> wrote: >> >>> * Contact emails* >>> and...@andreubotella.com, jbr...@chromium.org, su...@chromium.org >>> >>> *Explainer* >>> https://github.com/whatwg/html/issues/793 >>> >>> *Specification* >>> https://html.spec.whatwg.org/#structured-cloning >>> >>> * Summary* >>> Enables using the HTML structured clone algorithm synchronously for >>> cloning and transferring objects within a single realm. >>> >>> * Initial public proposal* >>> https://github.com/whatwg/html/issues/793 >>> https://github.com/whatwg/html/pull/3414 >>> >>> *Blink component* >>> Blink>Messaging >>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EMessaging> >>> >>> * TAG review* >>> This is just exposing existing browser functionality, with a two-line >>> spec. It doesn’t seem like there’s much to discuss architecturally, but >>> I’ll file for review if the community thinks it would help. >>> >>> *TAG review status* >>> Not applicable >>> >>> * Risks* >>> >>> * Interoperability and Compatibility* >>> Low. There are some differences across the browsers’ implementations of >>> the structured cloning algorithm, but they are very minor and already >>> present in other APIs that use it. >>> >>> Gecko: Shipped/Shipping ( >>> https://bugzilla.mozilla.org/show_bug.cgi?id=1722576) >>> Edge: No signal >>> WebKit: Shipped/Shipping (https://bugs.webkit.org/show_bug.cgi?id=228331) >>> >>> >>> Web developers: Positive ( >>> https://github.com/whatwg/html/pull/3414#issuecomment-854051942 and >>> following comments). There seems to be a lot of demand for a built-in deep >>> clone, and while structured clone is not exactly that, it fulfills many of >>> the use cases. >>> >>> * Debuggability* >>> n/a >>> >>> * Is this feature fully tested by web-platform-tests >>> <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md>?* >>> Yes >>> <https://wpt.fyi/results/html/webappapis/structured-clone?label=experimental&label=master&aligned> >>> >>> * Requires code in //chrome?* >>> False >>> >>> * Tracking bug* >>> https://bugs.chromium.org/p/chromium/issues/detail?id=1233571 >>> >>> *Estimated milestones* >>> No milestones specified >>> >>> * Link to entry on the Chrome Platform Status* >>> https://chromestatus.com/feature/5630001077551104 >>> >>> *Requesting approval to ship? * >>> Yes. This is a relatively small feature which exposes existing >>> functionality. >>> >>> -- >>> 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+...@chromium.org. >>> To view this discussion on the web visit >>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/e7299674-54df-4f4d-8c30-d922ebf4e47cn%40chromium.org >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/e7299674-54df-4f4d-8c30-d922ebf4e47cn%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/74ab6933-2925-455a-9e24-a95ae08f3cf5n%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/74ab6933-2925-455a-9e24-a95ae08f3cf5n%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/CAL5BFfVnABD048Xi6r3J9%2BGwBmYUX6pM1Auqp6MQSuwJUaNejg%40mail.gmail.com.