New issue
Advanced search Search tips

Issue 739393 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Add sequence checks to content::MediaStreamVideoCapturerSource

Project Member Reported by guidou@chromium.org, Jul 5 2017

Issue description

Most methods don't have sequence checks.
Internal classes use thread checks instead of sequence checks.
 
Labels: -Type-Bug Type-Task

Comment 2 by orphis@chromium.org, Oct 11 2017

Owner: orphis@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 16 2017

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

commit 5aa3813736e571bd3091a89e4a401eca1082a0d6
Author: Florent Castelli <orphis@chromium.org>
Date: Mon Oct 16 12:06:59 2017

Add missing sequence check to remaining MSVCS function.

Bug: 739393
Change-Id: I923995bda4576ab921f1d122eacb663ba8568873
Reviewed-on: https://chromium-review.googlesource.com/716484
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Florent Castelli <orphis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509024}
[modify] https://crrev.com/5aa3813736e571bd3091a89e4a401eca1082a0d6/content/renderer/media/media_stream_video_capturer_source.cc

Sign in to add a comment