New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 760522 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android , Mac
Pri: 1
Type: Bug



Sign in to add a comment

track.applyConstraints({advanced: []}) causes immediate crash

Project Member Reported by mscales@chromium.org, Aug 30 2017

Issue description

Chrome Version: 60.0.3112.101
Operating System: Mac OS X 10.12.6

Setting the 'advanced' constraints of a MediaStreamTrack to an empty array causes an immediate crash.

This was observed on stable and canary, on Mac and Android (this crash report is for stable on Mac)


Test case:
async function go() {
  const stream = await navigator.mediaDevices.getUserMedia({video: true});
  const track = stream.getVideoTracks()[0];
  track.applyConstraints({advanced: []});
}

go();


****DO NOT CHANGE BELOW THIS LINE****
Crash ID: crash/b8f6e42200f4073b

 
Description: Show this description
Labels: OS-Android
Cc: sandeepkumars@chromium.org
Labels: M-60
Owner: mcasas@chromium.org
Status: Assigned (was: Unconfirmed)
Tested the issue using #60.0.3112.113 on Mac 10.12.6 and was unable to reproduce the issue as per the steps mentioned in comment #0. No Crash is observed while running the above script in console.

Note:
1. This Crash has first started in M59.
2. This Crash is observed in Windows, Mac and Android as well

Using Code Search for the file, "MediaStreamTrack.cpp" assigning to the concern owner who might be related.

Suspecting Commit#
https://chromium.googlesource.com/chromium/src/+/1d5b6a1cd0258c12363b53f0d382471141463abe

@mcasas -- Could you please look into the issue, kindly re-assign if this is not related to your changes.

Link to the list of the builds
==============================
https://goto.google.com/zytlk

Thank You.


Comment 4 by mcasas@chromium.org, Aug 31 2017

Owner: guidou@chromium.org
guidou@ can you PTAL? I'm OOO and should be easily reproduceable. 

Comment 5 by guidou@chromium.org, Aug 31 2017

I will take a look.

Comment 6 by guidou@chromium.org, Aug 31 2017

Labels: Pri-1

Comment 8 by guidou@chromium.org, Aug 31 2017

Status: Fixed (was: Assigned)

Comment 9 by guidou@chromium.org, Aug 31 2017

I think we should request merge to M61 before it becomes stable.
mcasas@: WDYT?
Cc: pbomm...@chromium.org gov...@chromium.org
Labels: -M-60 M-61 TE-DesktopTriage
Looping to folks who works for M61.
Based on the crash_id provided in bug report, This crash had 106 number of crashes on Android during M59(59.0.3071.125) and just 10 crashes overall on Desktop(including Chrome OS).

I am not sure the functional impact and if we have to make a call just based on crash number it's too low crash rate to consider the merge to M61 at this stage, Please find the data here : https://goto.google.com/zrbos

Please do correct me and let us know why this should be merged to M61 before it hits stable(as per comment#9).
My main argument for merging to M61 is that it removes an easy way to crash the renderer with a very simple (and thus low-risk) patch. 

However, I will let mcasas@ decide if we should try to make the case for merging  to M61 since he is in a better position to judge the impact of both the bug and the fix.
Cc: mcasas@chromium.org
+ mcasas@, PTAL comment #12.


Merging sounds reasonable code-wise since it has been baked in M62 and it's 
a low risk feature, however since M61 is very close to cut and the crash
rate is quite reduced, we could let it pass and wait for 62 to roll. 

Sign in to add a comment