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

Issue 616884 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Extension crash when click Chrome's stop sharing dialog

Project Member Reported by x...@chromium.org, Jun 2 2016

Issue description

Version: 52.0.2739.0

Steps:
1. Cast entire desktop to Chromecast
2. Click Stop sharing blue button in dialog that pops up along taskbar at bottom of screen.

Expected: Receiver immediately tears down session returning to home screen. Sender clears activity immediately as well.

Actual: receiver does not immediately return to homescreen. After ~ 10 seconds, a blue error screen appears and then the homescreen.
 

Comment 1 by amp@chromium.org, Jun 3 2016

Labels: M-52
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 4 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 6 2016

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

commit c4ef1a4e478937783c303b8a712e9fef0457ac60
Author: xjz <xjz@chromium.org>
Date: Mon Jun 06 18:34:27 2016

Avoid double deleting from |MediaStreamDispatcher::label_stream_map_|.
MediaStreamDispatcher::OnDeviceStopped() may delete the same element
twice from |label_stream_map_| and causes the crash.

BUG= 616884 

Review-Url: https://codereview.chromium.org/2035993002
Cr-Commit-Position: refs/heads/master@{#398075}

[modify] https://crrev.com/c4ef1a4e478937783c303b8a712e9fef0457ac60/content/renderer/media/media_stream_dispatcher.cc

Comment 4 by m...@chromium.org, Jun 6 2016

Labels: -M-53 Merge-Request-52 M-52
We should merge this crash fix into M52.

Comment 5 by x...@chromium.org, Jun 7 2016

Status: Fixed (was: Started)

Comment 6 by x...@chromium.org, Jun 7 2016

Labels: -MovedFrom-52

Comment 7 by tin...@google.com, Jun 7 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Please merge the CL in to M52 branch asap so that it gets picked for next beta promotion scheduled on 06/15.
Project Member

Comment 9 by bugdroid1@chromium.org, Jun 8 2016

Labels: -merge-approved-52 merge-merged-2743
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a279df07994a42c625120bbf20d7f3d4c67f4197

commit a279df07994a42c625120bbf20d7f3d4c67f4197
Author: Yuri Wiitala <miu@chromium.org>
Date: Wed Jun 08 21:30:35 2016

Avoid double deleting from |MediaStreamDispatcher::label_stream_map_|. MediaStreamDispatcher::OnDeviceStopped() may delete the same element twice from |label_stream_map_| and causes the crash.

BUG= 616884 
TBR=xjz@chromium.org

Review-Url: https://codereview.chromium.org/2035993002
Cr-Commit-Position: refs/heads/master@{#398075}
(cherry picked from commit c4ef1a4e478937783c303b8a712e9fef0457ac60)

Review URL: https://codereview.chromium.org/2049293002 .

Cr-Commit-Position: refs/branch-heads/2743@{#285}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/a279df07994a42c625120bbf20d7f3d4c67f4197/content/renderer/media/media_stream_dispatcher.cc

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 15 2016

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

commit a279df07994a42c625120bbf20d7f3d4c67f4197
Author: Yuri Wiitala <miu@chromium.org>
Date: Wed Jun 08 21:30:35 2016

Avoid double deleting from |MediaStreamDispatcher::label_stream_map_|. MediaStreamDispatcher::OnDeviceStopped() may delete the same element twice from |label_stream_map_| and causes the crash.

BUG= 616884 
TBR=xjz@chromium.org

Review-Url: https://codereview.chromium.org/2035993002
Cr-Commit-Position: refs/heads/master@{#398075}
(cherry picked from commit c4ef1a4e478937783c303b8a712e9fef0457ac60)

Review URL: https://codereview.chromium.org/2049293002 .

Cr-Commit-Position: refs/branch-heads/2743@{#285}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/a279df07994a42c625120bbf20d7f3d4c67f4197/content/renderer/media/media_stream_dispatcher.cc

Components: Internals>Cast
Components: -Internals>Cast>Extension

Sign in to add a comment