Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(322)

Unified Diff: net/test/remote_test_server.cc

Issue 12210088: Make the TestServer use an absolute document root path. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: testserver uses absolute docroot path Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/test/remote_test_server.cc
diff --git a/net/test/remote_test_server.cc b/net/test/remote_test_server.cc
index a4eab058005307543ba2c6eca476f0b6bae14be5..c883514641c294d3e97310dbd01b85293079ab14 100644
--- a/net/test/remote_test_server.cc
+++ b/net/test/remote_test_server.cc
@@ -161,7 +161,7 @@ base::FilePath RemoteTestServer::GetDocumentRoot() const {
}
bool RemoteTestServer::Init(const base::FilePath& document_root) {
- if (document_root.IsAbsolute())
+ if (!document_root.IsAbsolute())
return false;
// Gets ports information used by test server spawner and Python test server.
@@ -194,9 +194,9 @@ bool RemoteTestServer::Init(const base::FilePath& document_root) {
SetPort(test_server_port);
SetResourcePath(document_root, base::FilePath().AppendASCII("net")
- .AppendASCII("data")
- .AppendASCII("ssl")
- .AppendASCII("certificates"));
+ .AppendASCII("data")
+ .AppendASCII("ssl")
+ .AppendASCII("certificates"));
return true;
}

Powered by Google App Engine
This is Rietveld 408576698