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

Unified Diff: extensions/shell/browser/media_capture_util.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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: extensions/shell/browser/media_capture_util.cc
diff --git a/extensions/shell/browser/media_capture_util.cc b/extensions/shell/browser/media_capture_util.cc
index fe2db183f0be6fc5a828ea80401943c87082ba26..e48e881e747cfaab15009ef38118b32e07ee7471 100644
--- a/extensions/shell/browser/media_capture_util.cc
+++ b/extensions/shell/browser/media_capture_util.cc
@@ -76,12 +76,12 @@ void VerifyMediaAccessPermission(content::MediaStreamType type,
if (type == content::MEDIA_DEVICE_AUDIO_CAPTURE) {
// app_shell has no UI surface to show an error, and on an embedded device
// it's better to crash than to have a feature not work.
- CHECK(permissions_data->HasAPIPermission(APIPermission::kAudioCapture))
- << "Audio capture request but no audioCapture permission in manifest.";
+ // Audio capture request but no audioCapture permission in manifest.
+ CHECK(permissions_data->HasAPIPermission(APIPermission::kAudioCapture));
} else {
DCHECK(type == content::MEDIA_DEVICE_VIDEO_CAPTURE);
- CHECK(permissions_data->HasAPIPermission(APIPermission::kVideoCapture))
- << "Video capture request but no videoCapture permission in manifest.";
+ // Video capture request but no videoCapture permission in manifest.
+ CHECK(permissions_data->HasAPIPermission(APIPermission::kVideoCapture));
}
}

Powered by Google App Engine
This is Rietveld 408576698