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

Issue 595970 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

FATAL:media_web_contents_observer.cc(218)] Check failed: std::find((*player_map)[id.first].begin(), (*player_map)[id.first].end(), id.second) == (*player_map)[id.first].end().

Project Member Reported by ukai@chromium.org, Mar 18 2016

Issue description

Version: 51.0.2680.0 (Developer Build) (64-bit) with dcheck_always_on=1
OS: Linux

What steps will reproduce the problem?
don't know.
(1)
(2)
(3)

What is the expected output? What do you see instead?

browser process crashed for some reason.
[6639:6639:0318/142124:FATAL:media_web_contents_observer.cc(218)] Check failed: 
std::find((*player_map)[id.first].begin(), (*player_map)[id.first].end(), id.second) == (*player_map)[id.first].end().

Program received signal SIGABRT, Aborted.
0x00007fffeeec8cc9 in __GI_raise (sig=sig@entry=6)
    at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
56      ../nptl/sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb)
(gdb)
(gdb) bt
#0  0x00007fffeeec8cc9 in __GI_raise (sig=sig@entry=6)
    at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1  0x00007fffeeecc0d8 in __GI_abort () at abort.c:89
#2  0x00007ffff7ec0202 in base::debug::BreakDebugger() ()
   from /usr/local/google/home/ukai/src/chromium-git/src/out.0/Release/./libbase.so
#3  0x00000000000000c8 in ?? ()
#4  0x00007ffff7ee175a in logging::LogMessage::~LogMessage() ()
   from /usr/local/google/home/ukai/src/chromium-git/src/out.0/Release/./libbase.so
#5  0x000024b18bffb300 in ?? ()
#6  0x000024b186e06d38 in ?? ()
#7  0x00007fffffffad98 in ?? ()
#8  0x00007ffff7f6b201 in tracked_objects::TaskStopwatch::Stop() ()
   from /usr/local/google/home/ukai/src/chromium-git/src/out.0/Release/./libbase.so
#9  0x36363a393336365b in ?? ()
#10 0x2f383133303a3933 in ?? ()
#11 0x463a343231323431 in ?? ()
#12 0x64656d3a4c415441 in ?? ()
#13 0x635f6265775f6169 in ?? ()
#14 0x5f73746e65746e6f in ?? ()
#15 0x726576726573626f in ?? ()
---Type <return> to continue, or q <return> to quit---
#16 0x293831322863632e in ?? ()
#17 0x206b63656843205d in ?? ()
#18 0x203a64656c696166 in ?? ()
#19 0x6e69663a3a647473 in ?? ()
#20 0x79616c702a282864 in ?? ()
#21 0x5b2970616d5f7265 in ?? ()
#22 0x74737269662e6469 in ?? ()
#23 0x286e696765622e5d in ?? ()
#24 0x616c702a28202c29 in ?? ()
#25 0x2970616d5f726579 in ?? ()
#26 0x737269662e64695b in ?? ()
#27 0x2928646e652e5d74 in ?? ()
#28 0x6365732e6469202c in ?? ()
#29 0x203d3d2029646e6f in ?? ()
#30 0x726579616c702a28 in ?? ()
#31 0x64695b2970616d5f in ?? ()
#32 0x2e5d74737269662e in ?? ()
#33 0x0a202e2928646e65 in ?? ()
#34 0x00007fffffffa660 in ?? ()
#35 0x00007fff00000001 in ?? ()
#36 0x000024b18304dcf0 in ?? ()
#37 0x000024b1858ef330 in ?? ()
#38 0x0000000000000000 in ?? ()


Please use labels and text to provide additional information.
https://chromium.googlesource.com/chromium/src/+/88c24007f6dc13d7e9e6b249e734ba5b668e8023
 
Components: Internals>Media
Owner: w...@chromium.org
Status: Assigned (was: Untriaged)
watk@ were you going to work on this? If not I can pick it up.

Comment 2 by w...@chromium.org, Mar 18 2016

I'm working on  http://crbug.com/595545  right now. I can get to this afterwards, or you can pick it up. Totally up to you.
Cc: -dalecur...@chromium.org w...@chromium.org
Owner: dalecur...@chromium.org
I'll pick it up.
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 18 2016

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

commit 73fd2eb18f1ed48a1047d737a9db170608ac6f47
Author: watk <watk@chromium.org>
Date: Fri Mar 18 21:45:57 2016

media: Allow WMPI to send playing started notification while playing

Previously MediaWebContentsObserver DCHECKed that an active player
was never added twice. To avoid complicated logic in WMPI to keep track
of when it's okay to send a playing notification to
MediaWebContentsObserver, now it's allowed to send it more than once.

BUG= 595903 ,  595970 

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

Cr-Commit-Position: refs/heads/master@{#382098}

[modify] https://crrev.com/73fd2eb18f1ed48a1047d737a9db170608ac6f47/content/browser/media/media_web_contents_observer.cc
[modify] https://crrev.com/73fd2eb18f1ed48a1047d737a9db170608ac6f47/content/browser/media/media_web_contents_observer.h

Status: Fixed (was: Assigned)
Cc: -w...@chromium.org dalecur...@chromium.org
Owner: w...@chromium.org
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 21 2016

Labels: merge-merged-2661
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/217c5e96c255d046fce4a1743b560e7ee5625a82

commit 217c5e96c255d046fce4a1743b560e7ee5625a82
Author: Chris Watkins <watk@chromium.org>
Date: Mon Mar 21 18:08:05 2016

media: Allow WMPI to send playing started notification while playing

Previously MediaWebContentsObserver DCHECKed that an active player
was never added twice. To avoid complicated logic in WMPI to keep track
of when it's okay to send a playing notification to
MediaWebContentsObserver, now it's allowed to send it more than once.

BUG= 595903 ,  595970 

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

Cr-Commit-Position: refs/heads/master@{#382098}
(cherry picked from commit 73fd2eb18f1ed48a1047d737a9db170608ac6f47)

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

Cr-Commit-Position: refs/branch-heads/2661@{#321}
Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081}

[modify] https://crrev.com/217c5e96c255d046fce4a1743b560e7ee5625a82/content/browser/media/media_web_contents_observer.cc
[modify] https://crrev.com/217c5e96c255d046fce4a1743b560e7ee5625a82/content/browser/media/media_web_contents_observer.h

Comment 8 by sshru...@google.com, May 24 2016

Components: -Blink>MediaStreamRecording Blink>MediaStream>Recording
Renamed component Blink>MediaStreamRecording to Blink>MediaStream>Recording. Moving issues to the new component. 

Comment 9 by mcasas@chromium.org, Jan 18 2017

Components: Blink>MediaRecording
Components: -Blink>MediaStream>Recording

Sign in to add a comment