From 037bad7497ec5a07e7d3bc0f14c40927cef33826 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Bostr=C3=B6m?= Date: Thu, 2 Apr 2015 12:10:13 +0200 Subject: [PATCH] ~CaptureManager: DCHECK(capture_states_.empty()) instead of CHECK until we fix not empty bug. BUG=chromium:320200 R=perkj@webrtc.org, tommi@webrtc.org Review URL: https://webrtc-codereview.appspot.com/49579004 Cr-Commit-Position: refs/heads/master@{#8922} --- talk/media/base/capturemanager.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/talk/media/base/capturemanager.cc b/talk/media/base/capturemanager.cc index a68c91c489..95612187fa 100644 --- a/talk/media/base/capturemanager.cc +++ b/talk/media/base/capturemanager.cc @@ -176,7 +176,9 @@ CaptureManager::~CaptureManager() { // cleaned up before we get here. In fact, in the normal shutdown sequence, // all capturers *will* be shut down by now, so trying to stop them here // will crash. If we're still tracking any, it's a dangling pointer. - CHECK(capture_states_.empty()); + // TODO(hbos): DCHECK instead of CHECK until we figure out why capture_states_ + // is not always empty here. + DCHECK(capture_states_.empty()); } bool CaptureManager::StartVideoCapture(VideoCapturer* video_capturer,