An implementation of chrome.socket.secure().

chrome/browser/extensions/api/sockets_tcp/sockets_tcp_apitest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/dns/host_resolver_wrapper.h"
 #include "chrome/browser/extensions/api/dns/mock_host_resolver_creator.h"
 #include "chrome/browser/extensions/api/sockets_tcp/sockets_tcp_api.h"
@@ skipping 64 matching lines @@
   scoped_ptr<base::Value> result(utils::RunFunctionAndReturnSingleResult(
       socket_create_function.get(), "[]", browser(), utils::NONE));
   ASSERT_EQ(base::Value::TYPE_DICTIONARY, result->GetType());
   base::DictionaryValue *value =
       static_cast<base::DictionaryValue*>(result.get());
   int socketId = -1;
   EXPECT_TRUE(value->GetInteger("socketId", &socketId));
   ASSERT_TRUE(socketId > 0);
 }
 
 IN_PROC_BROWSER_TEST_F(SocketsTcpApiTest, SocketTcpExtension) {

[Ryan Sleevi, 2014/02/04 22:28:14]

Rather than modifying the existing test, you should be adding an additional test
modelled after it.

That is, test the Socket TCP extension, then test the Socket TCP + SSL
extension.

[lally, 2014/02/11 22:05:35]

Done.

-  scoped_ptr<net::SpawnedTestServer> test_server(
+  scoped_ptr<net::SpawnedTestServer> test_tcp_server(
       new net::SpawnedTestServer(
           net::SpawnedTestServer::TYPE_TCP_ECHO,
           net::SpawnedTestServer::kLocalhost,
           base::FilePath(FILE_PATH_LITERAL("net/data"))));
-  EXPECT_TRUE(test_server->Start());
+  scoped_ptr<net::SpawnedTestServer> test_https_server(
+      new net::SpawnedTestServer(
+          net::SpawnedTestServer::TYPE_HTTPS,
+          net::BaseTestServer::SSLOptions(),
+          base::FilePath(FILE_PATH_LITERAL("net/data"))));
+  // Start them in parallel.
+  EXPECT_TRUE(test_tcp_server->StartInBackground());
+  EXPECT_TRUE(test_https_server->StartInBackground());
+  EXPECT_TRUE(test_tcp_server->BlockUntilStarted());
+  EXPECT_TRUE(test_https_server->BlockUntilStarted());
 
-  net::HostPortPair host_port_pair = test_server->host_port_pair();
-  int port = host_port_pair.port();
-  ASSERT_TRUE(port > 0);
+  net::HostPortPair tcp_host_port_pair = test_tcp_server->host_port_pair();
+  int tcp_port = tcp_host_port_pair.port();
+  ASSERT_TRUE(tcp_port > 0);
+
+  net::HostPortPair https_host_port_pair = test_https_server->host_port_pair();
+  int https_port = https_host_port_pair.port();
+  ASSERT_TRUE(https_port > 0);
 
   // Test that connect() is properly resolving hostnames.
-  host_port_pair.set_host("lOcAlHoSt");
+  tcp_host_port_pair.set_host("lOcAlHoSt");
 
   ResultCatcher catcher;
   catcher.RestrictToProfile(browser()->profile());
 
   ExtensionTestMessageListener listener("info_please", true);
 
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("sockets_tcp/api")));
   EXPECT_TRUE(listener.WaitUntilSatisfied());
   listener.Reply(
-      base::StringPrintf("tcp:%s:%d", host_port_pair.host().c_str(), port));
+      base::StringPrintf("tcp:%s:%d,https:%s:%d",
+                         tcp_host_port_pair.host().c_str(), tcp_port,
+                         https_host_port_pair.host().c_str(), https_port));
 
   EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }