[sync] Divorce python sync test server chromiumsync.py from testserver.py

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 3c6bddec38753183d47dac181e006d82b35ddb57..2f053c76ac51fd024aca8c42786be03bdf767101 100644
--- a/net/test/local_test_server.h
+++ b/net/test/local_test_server.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -45,17 +45,13 @@ class LocalTestServer : public BaseTestServer {
   // Modify PYTHONPATH to contain libraries we need.
   virtual bool SetPythonPath() const WARN_UNUSED_RESULT;
 
-  // This is a static version so that RunSyncTest in run_testserver.cc can use
-  // it.
-  // TODO(mattm): We should refactor so this isn't necessary (crbug.com/159731).
-  static bool SetPythonPathStatic() WARN_UNUSED_RESULT;
-
-  // Returns true if successfully stored the FilePath for the directory of the
-  // testserver python script in |*directory|.
-  static bool GetTestServerDirectory(FilePath* directory) WARN_UNUSED_RESULT;
+  // Returns true if the FilePath for the directory of the testserver python
+  // script is successfully stored in |*directory|.
+  virtual bool GetTestServerDirectory(FilePath* directory) const

[Paweł Hajdan Jr., 2013/01/17 23:38:49]

Why not just let the callers extract the directory from GetTestServerPath? This
used to be static before, so thank you for converting that. I think we should go
further and remove it completely. Overriding any of
GetTestServer{Directory,Path} without overriding the other is a recipe for
weirdness.

[Raghu Simha, 2013/01/18 02:17:09]

Agreed. I've removed GetTestServerDirectory. Done.

+      WARN_UNUSED_RESULT;
 
-  // Returns true if successfully stored the FilePath for the testserver python
-  // script in |*testserver_path|.
+  // Returns true if the FilePath for the testserver python script is
+  // successfully stored  in |*testserver_path|.
   virtual bool GetTestServerPath(FilePath* testserver_path) const
       WARN_UNUSED_RESULT;