PPAPI/WS: use server providing hostname as test serevr hostname

chrome/test/ppapi/ppapi_test.cc

Index: chrome/test/ppapi/ppapi_test.cc
diff --git a/chrome/test/ppapi/ppapi_test.cc b/chrome/test/ppapi/ppapi_test.cc
index 2c1def44d16ace12c42195273474a742b450b5c3..5a6e7baaa4b1553ca077b013d02b094fe774da12 100644
--- a/chrome/test/ppapi/ppapi_test.cc
+++ b/chrome/test/ppapi/ppapi_test.cc
@@ -219,10 +219,13 @@ void PPAPITestBase::RunTestWithWebSocketServer(const std::string& test_case) {
   ASSERT_TRUE(http_server.BlockUntilStarted());
   ASSERT_TRUE(ws_server.BlockUntilStarted());
 
+  std::string host = ws_server.host_port_pair().host();
   uint16_t port = ws_server.host_port_pair().port();
   RunTestURL(GetTestURL(http_server,
                         test_case,
-                        StringPrintf("websocket_port=%d", port)));
+                        StringPrintf("websocket_host=%s&websocket_port=%d",
+                            host.c_str(),

[dmichael (off chromium), 2013/02/13 21:41:13]

nit: please line up the 2nd and 3rd params with the 1st, since they fit

[Ryan Sleevi, 2013/02/13 21:46:14]

Do you need to worry about canonicalizing the host, in the event of IPv6 (that
is: adding brackets).

See HostPortPair::HostForURL

[Takashi Toyoshima, 2013/02/14 08:28:28]

Thank you for catching this.
Yes, I should call HostForURL to get host string here.

[Takashi Toyoshima, 2013/02/14 08:28:28]

Done.

+                            port)));
 }
 
 void PPAPITestBase::RunTestIfAudioOutputAvailable(