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

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

Issue 1917133002: WebRtcBrowserTest tests with RSA and ECDSA certificates. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed if-statement Created 4 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
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_base.h ('k') | chrome/test/data/webrtc/peerconnection.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/webrtc_browsertest_base.cc
diff --git a/chrome/browser/media/webrtc_browsertest_base.cc b/chrome/browser/media/webrtc_browsertest_base.cc
index 378c80d7887f9c605407c735b806e98d967f660e..06e452afa69c5f357d392a494b0e4b38c026087a 100644
--- a/chrome/browser/media/webrtc_browsertest_base.cc
+++ b/chrome/browser/media/webrtc_browsertest_base.cc
@@ -55,6 +55,8 @@ const char WebRtcTestBase::kAudioVideoCallConstraints360p[] =
const char WebRtcTestBase::kAudioVideoCallConstraints720p[] =
"{audio: true, video: {mandatory: {minWidth: 1280, maxWidth: 1280, "
" minHeight: 720, maxHeight: 720}}}";
+const char WebRtcTestBase::kUseDefaultCertKeygen[] = "null";
+const char WebRtcTestBase::kUseDefaultVideoCodec[] = "";
namespace {
@@ -303,15 +305,19 @@ std::string WebRtcTestBase::ExecuteJavascript(
}
void WebRtcTestBase::SetupPeerconnectionWithLocalStream(
- content::WebContents* tab) const {
- SetupPeerconnectionWithoutLocalStream(tab);
+ content::WebContents* tab,
+ std::string certificate_keygen_algorithm) const {
+ SetupPeerconnectionWithoutLocalStream(tab, certificate_keygen_algorithm);
EXPECT_EQ("ok-added", ExecuteJavascript("addLocalStream()", tab));
}
void WebRtcTestBase::SetupPeerconnectionWithoutLocalStream(
- content::WebContents* tab) const {
+ content::WebContents* tab,
+ std::string certificate_keygen_algorithm) const {
+ std::string javascript = base::StringPrintf(
+ "preparePeerConnection(%s)", certificate_keygen_algorithm.c_str());
EXPECT_EQ("ok-peerconnection-created",
- ExecuteJavascript("preparePeerConnection()", tab));
+ ExecuteJavascript(javascript, tab));
}
std::string WebRtcTestBase::CreateLocalOffer(
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_base.h ('k') | chrome/test/data/webrtc/peerconnection.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698