New issue
Advanced search Search tips

Issue 685345 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Component move: Internals>Media>Capture* to Blink>GetUserMedia

Project Member Reported by mcasas@chromium.org, Jan 25 2017

Issue description

Internals>Media>Capture and Internals>Media>Capture>Tab>v2 
should not exist, instead all the capture-related bugs should
be filed under Blink>GetUserMedia.

The former are confusing for issue reporters because
the boundary between categories is fuzzy and non intuitive, 
and anyway all uses of capture should ultimately come from a 
getUserMedia() JS call.

If you guys agree, I'll do the component reshuffling.
 

Comment 1 by chfremer@google.com, Jan 25 2017

I am okay with it, as it has been confusing for me as well.
However, when faced with a decision which tag to assign to video capture issues, I usually ended up choosing Internals>Media>Capture over Blink>GetUserMedia, because I felt that the video capture code that I am touching has nothing to do with Blink directly (other than that Blink happens to expose GetUserMedia, whose implementation requires the internal video capture code).

That may be enough reason to file video capture under this tag, but Blink would not naturally be the first place for me to look for it.
Cc: dalecur...@chromium.org
Status: Available (was: Untriaged)
Add Dale. change status to available.

Comment 3 by m...@chromium.org, Feb 7 2017

I've actually argued (many times in the past) for the opposite: That we get stop using Blink>GetUserMedia and Blink>GetUserMedia>Tab, and instead use the Internals>... ones. The main reason: For tab capture, there is no web platform access to it, only an extension API, and so it makes no sense for it to be in the "Blink" bucket.

Comment 4 by m...@chromium.org, Feb 1 2018

Status: WontFix (was: Available)
Going to move tab capture bugs out of Internals>Media>Capture and place them in a new Internals>Media>ScreenCapture component.

Sign in to add a comment