Primary eng (and PM) emails
bi...@chromium.org, bradn...@chromium.org
Summary
SharedArrayBuffer.isView is a function that was accidentally shipped when the SharedArrayBuffer feature was shipped.
It is not in the ECMAscript spec, and it was not implemented by any other browser.
Motivation
This function should be removed for spec and browser compliance.
Interoperability and Compatibility Risk
Edge: not supported
Firefox: not supported
Safari: not supported
Alternative implementation suggestion for web developers
This behavior of this function is a duplication of ArrayBuffer.isView.
Usage information from UseCounter
A use counter for this function has not been implemented.
OWP launch tracking bug
Link to bug (not OWP launch tracking): https://crbug.com/793095
Entry on the feature dashboard
Requesting approval to remove too?
Yes
--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/c4a993ca-7d79-4f68-92aa-96bd6ce7323d%40chromium.org.
Given that nobody else supports this, removal without measuring the usage is probably safe.I don't quite understand the suggested alternative. SharedArrayBuffer doesn't seem to inherit from ArrayBuffer, so I assume that after this removal there could be some non-exotic code that breaks? (Exotic being relying on exactly where in the prototype chain a thing is.)
On Monday, December 11, 2017 at 2:21:46 PM UTC-8, Philip Jägenstedt wrote:Given that nobody else supports this, removal without measuring the usage is probably safe.I don't quite understand the suggested alternative. SharedArrayBuffer doesn't seem to inherit from ArrayBuffer, so I assume that after this removal there could be some non-exotic code that breaks? (Exotic being relying on exactly where in the prototype chain a thing is.)Sorry, what I meant by the suggestion is that, as an implementation detail, the current implementation of SharedArrayBuffer.isView is exactly the same as ArrayBuffer.isView. So if someone did happen to use SharedArrayBuffer.isView, they could just switch to that.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/82c8d20e-b874-4472-802c-14994ee661b7%40chromium.org.To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.
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/82c8d20e-b874-4472-802c-14994ee661b7%40chromium.org.
--
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/CAOMQ%2Bw9CGer9Y9hvrnARyy8AQQGhKE-_j3W%2Bhn8Kvb1Zr4vVcw%40mail.gmail.com.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/82c8d20e-b874-4472-802c-14994ee661b7%40chromium.org.
--
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+unsubscribe@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw9CGer9Y9hvrnARyy8AQQGhKE-_j3W%2Bhn8Kvb1Zr4vVcw%40mail.gmail.com.
--/* Opera Software, Linköping, Sweden: CET (UTC+1) */
--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/op.za8qu3z6rbppqq%40cicero2.linkoping.osa.
Apologies for the late response. Comments inline:
LGTM3Any lessons learned for avoiding such accidental shipping in the future? IDL tooling being discussed wouldn't help since this is ecma262. I don't see SharedArrayBuffer.isView in the webexposed test outputs, why?
Yes, I was wondering about this too. :-)
Apparently JS builtins aren't meant to be in the webexposed list anyway: see comment here. SharedArrayBuffer isn't in this blacklist (though it should be), so it is displayed. Even so, it appears that the interfaces only display properties on the prototype, if one exists. Since SharedArrayBuffer.isView is not on the prototype it isn't displayed.
Apologies for the late response. Comments inline:
On Thursday, December 14, 2017 at 7:44:20 AM UTC-8, Rick Byers wrote:LGTM3Any lessons learned for avoiding such accidental shipping in the future? IDL tooling being discussed wouldn't help since this is ecma262. I don't see SharedArrayBuffer.isView in the webexposed test outputs, why?Yes, I was wondering about this too. :-)
Apparently JS builtins aren't meant to be in the webexposed list anyway: see comment here. SharedArrayBuffer isn't in this blacklist (though it should be), so it is displayed. Even so, it appears that the interfaces only display properties on the prototype, if one exists. Since SharedArrayBuffer.isView is not on the prototype it isn't displayed.
As for how to avoid shipping the wrong thing in the future -- this is a good question. Perhaps v8 needs something similar to the webexposed test? I spoke to Adam and he said that this would be a reasonable addition.
On Wed, Jan 10, 2018 at 10:53 AM, Ben Smith <bi...@chromium.org> wrote:Apologies for the late response. Comments inline:
On Thursday, December 14, 2017 at 7:44:20 AM UTC-8, Rick Byers wrote:LGTM3Any lessons learned for avoiding such accidental shipping in the future? IDL tooling being discussed wouldn't help since this is ecma262. I don't see SharedArrayBuffer.isView in the webexposed test outputs, why?Yes, I was wondering about this too. :-)
Apparently JS builtins aren't meant to be in the webexposed list anyway: see comment here. SharedArrayBuffer isn't in this blacklist (though it should be), so it is displayed. Even so, it appears that the interfaces only display properties on the prototype, if one exists. Since SharedArrayBuffer.isView is not on the prototype it isn't displayed.
As for how to avoid shipping the wrong thing in the future -- this is a good question. Perhaps v8 needs something similar to the webexposed test? I spoke to Adam and he said that this would be a reasonable addition.+1. This has saved us from accidental launches in several other cases.