Issue 848314 link

Starred by 2 users

Issue metadata

Status:	Fixed
Owner:	mmoroz@chromium.org
Closed:	Jun 2018
Cc:	xhw...@chromium.org dalecur...@chromium.org wolenetz@chromium.org
Components:	Blink>Media
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	1
Type:	Bug

Blocking:
issue 845774

GN flag enable_mse_mpeg2ts_stream_parser is not always correct

Project Member Reported by mmoroz@chromium.org, May 31 2018

Issue description

https://cs.chromium.org/chromium/src/media/media_options.gni?type=cs&q=enable_mse_mpeg2ts_stream_parser&sq=package:chromium&g=0&l=60

use_fuzzing_engine is not enough, it also needs proprietary_codecs

Project Member

Comment 1 by bugdroid1@chromium.org, Jun 4 2018

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

commit 2ac3e7330141fd68ad651987f5b6e31d39e6a34a
Author: Max Moroz <mmoroz@chromium.org>
Date: Mon Jun 04 05:30:58 2018

media: fix enable_mse_mpeg2ts_stream_parser GN flag calculation for fuzzing builds.

Bug:  848314 
Change-Id: I0c7f2140d9ba680bfddb43d6b8b07f7544de17fb
Reviewed-on: https://chromium-review.googlesource.com/1080930
Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564000}
[modify] https://crrev.com/2ac3e7330141fd68ad651987f5b6e31d39e6a34a/media/media_options.gni

Comment 2 by mmoroz@chromium.org, Jun 4 2018

Status: Fixed (was: Started)

►

Issue 848314 link

Issue metadata

GN flag enable_mse_mpeg2ts_stream_parser is not always correct

Issue description

Comment 1 by bugdroid1@chromium.org, Jun 4 2018

Comment 1 Deleted

Comment 2 by mmoroz@chromium.org, Jun 4 2018

Comment 2 Deleted

Sign in to add a comment