Do not spawn a thread in browser/interactive ui tests before spawning sandbox host process

net/test/embedded_test_server/embedded_test_server.h

 // Copyright (c) 2012 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.
 
 #ifndef NET_TEST_EMBEDDED_TEST_SERVER_EMBEDDED_TEST_SERVER_H_
 #define NET_TEST_EMBEDDED_TEST_SERVER_EMBEDDED_TEST_SERVER_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 21 matching lines @@
 
 // This class is required to be able to have composition instead of inheritance,
 class HttpListenSocket : public TCPListenSocket {
  public:
   HttpListenSocket(const SocketDescriptor socket_descriptor,
                    StreamListenSocket::Delegate* delegate);
   virtual ~HttpListenSocket();
   virtual void Listen();
 
  private:
+  friend class EmbeddedTestServer;
+
+  // Detaches the current from |thread_checker_|.
+  void DetachFromThread();
 
   base::ThreadChecker thread_checker_;
 };
 
 // Class providing an HTTP server for testing purpose. This is a basic server
 // providing only an essential subset of HTTP/1.1 protocol. Especially,
 // it assumes that the request syntax is correct. It *does not* support
 // a Chunked Transfer Encoding.
 //
-// The common use case is below:
+// The common use case for unit tests is below:

[oshima, 2013/11/26 02:26:02]

Forgot to mention. I'll update the comment and add another examples for browser
test case.

 //
 // base::Thread io_thread_;
 // scoped_ptr<EmbeddedTestServer> test_server_;
 //
 // void SetUp() {
 //   base::Thread::Options thread_options;
 //   thread_options.message_loop_type = base::MessageLoop::TYPE_IO;
 //   ASSERT_TRUE(io_thread_.StartWithOptions(thread_options));
 //
 //   test_server_.reset(
@@ skipping 53 matching lines @@
   // For instance, a request to "/foo.html" is served by "foo.html" under
   // |directory|. Files under sub directories are also handled in the same way
   // (i.e. "/foo/bar.html" is served by "foo/bar.html" under |directory|).
   void ServeFilesFromDirectory(const base::FilePath& directory);
 
   // The most general purpose method. Any request processing can be added using
   // this method. Takes ownership of the object. The |callback| is called
   // on UI thread.
   void RegisterRequestHandler(const HandleRequestCallback& callback);
 
+  // Starts/Stops IO thread that handles http requests.
+  void StartThread();
+  void StopThread();
+
  private:
   // Initializes and starts the server. If initialization succeeds, Starts()
   // will return true.
   void InitializeOnIOThread();
 
   // Shuts down the server.
   void ShutdownOnIOThread();
 
   // Handles a request when it is parsed. It passes the request to registed
   // request handlers and sends a http response.
@@ skipping 32 matching lines @@
 
   base::ThreadChecker thread_checker_;
 
   DISALLOW_COPY_AND_ASSIGN(EmbeddedTestServer);
 };
 
 }  // namespace test_servers
 }  // namespace net
 
 #endif  // NET_TEST_EMBEDDED_TEST_SERVER_EMBEDDED_TEST_SERVER_H_