New issue
Advanced search Search tips

Issue 707691 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Merge to M58: potential crash in video call setup

Project Member Reported by sprang@chromium.org, Apr 3 2017

Issue description

Original bug here:
https://bugs.chromium.org/p/chromium/issues/detail?id=705505
Fix for crash bug was landed here: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+/90e3b7de4724e21e48f651a62e626fcc2b21b782
The code landed in chrome here: https://chromium.googlesource.com/chromium/src/+/6869066a6f9589b41fcb1857f25dfc9436ee69ef
...which was included from build 59.0.3057.0

This should be merged into M58 as well.
 
Cc: sprang@chromium.org
Owner: ilnik@chromium.org
ilnik@, can you take care of this while I'm off?
Project Member

Comment 2 by sheriffbot@chromium.org, Apr 4 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Your change is approved for M58. Please confirm whether the change is verified in canary. If all looks good merge ASAP so that it will be picked up for next Beta Release, RC cut today (Tuesday-04/04) at 4.00 PM PST.

Comment 4 by ilnik@chromium.org, Apr 4 2017

Re #3: It was merged just now.

Comment 5 by ilnik@chromium.org, Apr 4 2017

Status: Fixed (was: Untriaged)
If there is no pending work, please remove Merge-Approved-58 and 
add merge-merged-3029 label.

Comment 7 by ilnik@chromium.org, Apr 6 2017

Labels: -Merge-Approved-58 merge-merged-3029

Comment 8 by ilnik@chromium.org, Apr 6 2017

Re #6, Done.
Labels: M-59 M-58

Sign in to add a comment