Run safebrowsing_service_test through the net testserver code.

net/test/local_test_server.h

Index: net/test/local_test_server.h
diff --git a/net/test/local_test_server.h b/net/test/local_test_server.h
index 63ea00e611deba6efba38601389ca7cb3d05b37b..91b285e68a858e13cf739ab94119f96e0eb2d860 100644
--- a/net/test/local_test_server.h
+++ b/net/test/local_test_server.h
@@ -58,9 +58,10 @@ class LocalTestServer : public BaseTestServer {
   // Waits for the server to start. Returns true on success.
   bool WaitToStart() WARN_UNUSED_RESULT;
 
-  // Add the command line arguments for the Python test server to
-  // |command_line|. Return true on success.

[Ryan Sleevi, 2012/04/18 00:45:02]

This comment is no longer accurate.

While I realize you changed it for Pawel's comments, this now does two things:
It adds the command line arguments for Python AND the command line arguments for
the Python test server.

Perhaps split into two methods, to keep that separation?

[mattm, 2012/08/03 19:41:03]

Done.

-  bool AddCommandLineArguments(CommandLine* command_line) const;
+  // Add the command line arguments for the Python test server
+  // |testserver_path| to |command_line|. Return true on success.
+  bool AddCommandLineArguments(const FilePath& testserver_path,
+                               CommandLine* command_line) const;
 
   // Handle of the Python process running the test server.
   base::ProcessHandle process_handle_;