Results 131 comments of Elad Alon

> Can we not have a `removeCrop()` function on the track? Since it'd be functionally equivalent, I'd not object. But my own subjective preference is to have a single method...

You've mentioned precedents for this pattern. Could you please specify one or two precedents, so that I might examine the reasoning that led to that pattern being adopted there, and...

I believe `setInterval` and `setTimeout` return a handle[\*], and `clearInterval` accepts that handle, for a practical reason - there'd otherwise be no way to know **which** interval to clear. This...

> There are benefits to doing `cropTo(null)` in the fact you'd not need to keep track of that returned function - as long as you had access to the track...

> My issue was primarily with an empty string :D No harm in thinking out loud about other APIs :) Absolutely no harm indeed. I'm very glad to hear your...

Speaking of feedback, I am wondering - any objections to making a repeated call to `cropTo(null)`/`removeCrop()` **not** throw an error, but rather be a no-op? It simplifies both the implementation...

> That's what I said further up :) I appear to have misread your original message. Specifically, I believe I misread "and a noop if one doesn't" as "and an...

> Any reason why this would be limited to `HTMLDivElement` or `HTMLIFrameElement`? What if one wanted to share just a `HTMLVideoElement` for example? The use case could be to share...

Thanks, @murillo128! For **test automation** purposes, I believe [these flags](https://source.chromium.org/chromium/chromium/src/+/main:chrome/common/chrome_switches.cc;l=549-552;drc=3bfd3ba3af617152dfee61d03e93a773ddd3443b) should do the trick for you. Let me know if more is required.