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

Unified Diff: chrome/test/data/webrtc/jsep01_call.js

Issue 10958074: Added workaround to the current API revision chaos. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webrtc/jsep01_call.js
diff --git a/chrome/test/data/webrtc/jsep01_call.js b/chrome/test/data/webrtc/jsep01_call.js
index 647179a1ce9e734c8e6431a40e18f12ce489ea75..7a934ca1054bf7927a3718198c029e7a756ab4c9 100644
--- a/chrome/test/data/webrtc/jsep01_call.js
+++ b/chrome/test/data/webrtc/jsep01_call.js
@@ -37,7 +37,16 @@ function createPeerConnection(stun_server) {
try {
peerConnection = new webkitRTCPeerConnection(servers, null);
} catch (exception) {
- failTest('Failed to create peer connection: ' + exception);
+ // TODO(phoglund): Remove once the URI-requiring revisions are gone.
+ debug('Failed to create connection: maybe we need to fall ' +
+ 'back to the old API?');
+ servers = {iceServers:[{uri:"stun:" + stun_server}]};
+ try {
+ peerConnection = new webkitRTCPeerConnection(servers, null);
+ debug('Yeah, we could use the old API.');
+ } catch (exception) {
+ failTest('Failed to create peer connection: ' + exception);
+ }
}
peerConnection.onaddstream = addStreamCallback_;
peerConnection.onremovestream = removeStreamCallback_;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698