New issue
Advanced search Search tips

Issue 877430 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Remove MediaStreamSource constraints accessors

Project Member Reported by guidou@chromium.org, Aug 24

Issue description

Constraints do not have sources, so this is dead code.
 
Components: Blink>GetUserMedia
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 24

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9

commit 7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9
Author: Guido Urdaneta <guidou@chromium.org>
Date: Fri Aug 24 14:04:47 2018

Remove dead code related to MediaStreamSource constraints.

MediaStreamSources do not have constraints, so remove old accessors
that provided that functionality.

Bug:  877430 
Change-Id: Iee3973f5ddcb8d85608a798565784a59e474231a
Reviewed-on: https://chromium-review.googlesource.com/1188312
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585829}
[modify] https://crrev.com/7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9/third_party/blink/public/platform/web_media_stream_source.h
[modify] https://crrev.com/7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9/third_party/blink/renderer/modules/mediastream/user_media_request.cc
[modify] https://crrev.com/7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9/third_party/blink/renderer/platform/exported/web_media_stream_source.cc
[modify] https://crrev.com/7e63ed80b1a09ebd6d008563c5ae0aa530a5e0c9/third_party/blink/renderer/platform/mediastream/media_stream_source.h

Status: Fixed (was: Started)

Sign in to add a comment