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

Side by Side Diff: chrome/browser/safe_browsing/local_safebrowsing_test_server.cc

Issue 10073033: Run safebrowsing_service_test through the net testserver code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix warning Created 8 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/safe_browsing/local_safebrowsing_test_server.h"
6
7 #include "base/command_line.h"
8 #include "base/path_service.h"
9 #include "base/string_number_conversions.h"
10 #include "base/values.h"
11 #include "net/test/python_utils.h"
12 #include "net/test/test_server.h"
13
14 LocalSafeBrowsingTestServer::LocalSafeBrowsingTestServer(
15 const FilePath& data_file)
16 : net::LocalTestServer(net::TestServer::TYPE_HTTP,
17 net::TestServer::kLocalhost,
18 FilePath()),
19 data_file_(data_file) {}
Ryan Sleevi 2012/08/31 21:48:39 nit: { } Per http://google-styleguide.googlecode.
mattm 2012/09/01 01:44:54 Done.
20
21 LocalSafeBrowsingTestServer::~LocalSafeBrowsingTestServer() {}
22
23 bool LocalSafeBrowsingTestServer::GetTestServerPath(
24 FilePath* testserver_path) const {
25 FilePath testserver_dir;
26 if (!PathService::Get(base::DIR_SOURCE_ROOT, &testserver_dir)) {
27 LOG(ERROR) << "Failed to get DIR_SOURCE_ROOT";
28 return false;
29 }
30
31 testserver_dir = testserver_dir
32 .Append(FILE_PATH_LITERAL("chrome"))
33 .Append(FILE_PATH_LITERAL("browser"))
34 .Append(FILE_PATH_LITERAL("safe_browsing"));
35
36 *testserver_path = testserver_dir.Append(FILE_PATH_LITERAL(
37 "safe_browsing_testserver.py"));
38 return true;
39 }
40
41 bool LocalSafeBrowsingTestServer::SetPythonPath() const {
42 if (!net::LocalTestServer::SetPythonPath())
43 return false;
44
45 // Locate the Python code generated by the protocol buffers compiler.
46 FilePath pyproto_dir;
47 if (!GetPyProtoPath(&pyproto_dir)) {
48 LOG(ERROR) << "Cannot find pyproto dir for generated code.";
49 return false;
50 }
51
52 AppendToPythonPath(pyproto_dir.AppendASCII("google"));
53 return true;
54 }
55
56 bool LocalSafeBrowsingTestServer::GenerateAdditionalArguments(
57 base::DictionaryValue* arguments) const {
58 arguments->SetString("data-file", data_file_.value());
59 return true;
60 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698