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

Unified Diff: chrome/browser/media/webrtc/webrtc_browsertest_base.cc

Issue 2985263002: Reland of RTCVideoEncoder: Report H264 profile information to WebRTC (Closed)
Patch Set: Add default argument to SetDefaultVideoCodec Created 3 years, 4 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/webrtc/webrtc_browsertest_base.cc
diff --git a/chrome/browser/media/webrtc/webrtc_browsertest_base.cc b/chrome/browser/media/webrtc/webrtc_browsertest_base.cc
index d7c6c9d1c40cb7ab9f6c94136f9aca00566cad3a..76f930332fdb2baaf790aaa1e006e96e27ffc9d8 100644
--- a/chrome/browser/media/webrtc/webrtc_browsertest_base.cc
+++ b/chrome/browser/media/webrtc/webrtc_browsertest_base.cc
@@ -559,11 +559,13 @@ void WebRtcTestBase::SetDefaultAudioCodec(
"setDefaultAudioCodec('" + audio_codec + "')", tab));
}
-void WebRtcTestBase::SetDefaultVideoCodec(
- content::WebContents* tab,
- const std::string& video_codec) const {
- EXPECT_EQ("ok", ExecuteJavascript(
- "setDefaultVideoCodec('" + video_codec + "')", tab));
+void WebRtcTestBase::SetDefaultVideoCodec(content::WebContents* tab,
+ const std::string& video_codec,
+ bool prefer_hw_codec) const {
+ EXPECT_EQ("ok",
+ ExecuteJavascript("setDefaultVideoCodec('" + video_codec + "'," +
+ (prefer_hw_codec ? "true" : "false") + ")",
+ tab));
}
void WebRtcTestBase::EnableOpusDtx(content::WebContents* tab) const {

Powered by Google App Engine
This is Rietveld 408576698