Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5716)

Unified Diff: chrome/browser/media/desktop_streams_registry.cc

Issue 1099803003: [chrome/browser/media] favor DCHECK_CURRENTLY_ON for better logs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed build issue Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media/desktop_streams_registry.cc
diff --git a/chrome/browser/media/desktop_streams_registry.cc b/chrome/browser/media/desktop_streams_registry.cc
index 9f6a0ebec9c193fa49d1ff80a5b76cf542ee4c21..bd21240cdc56f3117fc2fe2f0a1e2052ef5cd7cd 100644
--- a/chrome/browser/media/desktop_streams_registry.cc
+++ b/chrome/browser/media/desktop_streams_registry.cc
@@ -36,7 +36,7 @@ std::string DesktopStreamsRegistry::RegisterStream(
const GURL& origin,
const content::DesktopMediaID& source,
const std::string& extension_name) {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
std::string id = GenerateRandomStreamId();
DCHECK(approved_streams_.find(id) == approved_streams_.end());
@@ -62,7 +62,7 @@ content::DesktopMediaID DesktopStreamsRegistry::RequestMediaForStreamId(
int render_frame_id,
const GURL& origin,
std::string* extension_name) {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
StreamsMap::iterator it = approved_streams_.find(id);
@@ -82,7 +82,7 @@ content::DesktopMediaID DesktopStreamsRegistry::RequestMediaForStreamId(
}
void DesktopStreamsRegistry::CleanupStream(const std::string& id) {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+ DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
approved_streams_.erase(id);
}
« no previous file with comments | « chrome/browser/media/desktop_media_list_ash.cc ('k') | chrome/browser/media/media_capture_devices_dispatcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698