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

Unified Diff: ppapi/tests/test_websocket.cc

Issue 12252004: PPAPI/WS: use server providing hostname as test serevr hostname (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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: ppapi/tests/test_websocket.cc
diff --git a/ppapi/tests/test_websocket.cc b/ppapi/tests/test_websocket.cc
index acbad3e4de0280d3a80a73086198292a32afeeae..5b400eeb38f6707775b917399d23b2690275a0b8 100644
--- a/ppapi/tests/test_websocket.cc
+++ b/ppapi/tests/test_websocket.cc
@@ -240,8 +240,13 @@ void TestWebSocket::RunTests(const std::string& filter) {
}
std::string TestWebSocket::GetFullURL(const char* url) {
- std::string rv = "ws://localhost";
- // Some WebSocket tests don't start the server so there'll be no port.
+ std::string rv = "ws://";
+ // Some WebSocket tests don't start the server so there'll be no host and
+ // port.
+ if (instance_->websocket_host().empty())
+ rv += "127.0.0.1";
+ else
+ rv += instance_->websocket_host();
if (instance_->websocket_port() != -1) {
char buffer[10];
sprintf(buffer, ":%d", instance_->websocket_port());

Powered by Google App Engine
This is Rietveld 408576698