New issue
Advanced search Search tips

Issue 895017 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Annotate thread-safetiness in ChunkDemuxer{Stream}

Project Member Reported by wolenetz@chromium.org, Oct 12

Issue description

Now that GUARDED_BY, etc are available, upgrade our ChunkDemuxer internals to use these static checks for safety (and fix any missing locking as part of this).
 
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 13

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

commit a0810e4e9e8ffa5475acd49373367423d64e1740
Author: Matt Wolenetz <wolenetz@chromium.org>
Date: Sat Oct 13 00:55:37 2018

MSE: Annotate thread-safetiness related to ChunkDemuxerStream::lock_

Adds GUARDED_BY and related annotations to ChunkDemuxerStream.
Adding the annotations makes it clear which fields are protected by
ChunkDemuxerStream::lock_ and enables compile-time verification.

These additions confirmed previous lack of appropriate locking in four
ancillary ChunkDemuxerStream methods; this CL also fixes those cases.

Credit to jrummell@ for identifying the missing locking, and to
chcunningham@ for promoting the new static annotations' usage.

BUG=895017,887645
R=jrummell@chromium.org

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I0c14ac2fedf60df84e616cf3c268b876d8f8c514
Reviewed-on: https://chromium-review.googlesource.com/c/1279242
Reviewed-by: John Rummell <jrummell@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599452}
[modify] https://crrev.com/a0810e4e9e8ffa5475acd49373367423d64e1740/media/filters/chunk_demuxer.cc
[modify] https://crrev.com/a0810e4e9e8ffa5475acd49373367423d64e1740/media/filters/chunk_demuxer.h

Sign in to add a comment