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

Unified Diff: net/test/local_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/local_test_server.cc
diff --git a/net/test/local_test_server.cc b/net/test/local_test_server.cc
index e2d334f9557e2c5e5b6e0e237c44e7234bf1510c..73236b4c33ff105d363ed29fe5f3c80834cfe645 100644
--- a/net/test/local_test_server.cc
+++ b/net/test/local_test_server.cc
@@ -140,7 +140,7 @@ bool LocalTestServer::Stop() {
}
bool LocalTestServer::Init(const base::FilePath& document_root) {
- if (document_root.IsAbsolute())
+ if (!document_root.IsAbsolute())
return false;
// At this point, the port that the test server will listen on is unknown.
@@ -153,7 +153,7 @@ bool LocalTestServer::Init(const base::FilePath& document_root) {
base::FilePath src_dir;
if (!PathService::Get(base::DIR_SOURCE_ROOT, &src_dir))
return false;
- SetResourcePath(src_dir.Append(document_root),
+ SetResourcePath(document_root,
src_dir.AppendASCII("net")
.AppendASCII("data")
.AppendASCII("ssl")

Powered by Google App Engine
This is Rietveld 408576698