Re: [mediacapture-region] Why expose produceCropTarget at MediaDevices level? (#11)

> @eladalon1983 the way I interpreted the compromise above was we agreed to move the point of exposure from MediaDevices (a SecureContext object) to CropTarget (which is not). Adding SecureContext to fromElement would appear to not honor that.

Then we have different interpretations. I still suggest we start by merging #50, then continue the discussion about SecureContext separately. Should we decide to move away from SecureContext, we'll then have the path for it (which we didn't with mediaDevices.produceCropTarget).

> @youennf already mentioned in https://github.com/w3c/mediacapture-region/issues/11#issuecomment-1073988276 that this was part of the issue

I understand Youenn's reasoning and I have [addressed](https://github.com/w3c/mediacapture-region/issues/11#issuecomment-1132675929) it.

> I'll accept the PR with a note on the lack of consensus over SecureContext.

That's great.

-- 
GitHub Notification of comment by eladalon1983
Please view or discuss this issue at https://github.com/w3c/mediacapture-region/issues/11#issuecomment-1139414575 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 27 May 2022 08:49:35 UTC