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

Unified Diff: net/test/local_sync_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, 4 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_sync_test_server.cc
diff --git a/net/test/local_sync_test_server.cc b/net/test/local_sync_test_server.cc
index 8d757f867ae3bdf07b0a1fba6d6480ce326a6a2c..432fde1d90169b3319d5a037f6431315de6814b2 100644
--- a/net/test/local_sync_test_server.cc
+++ b/net/test/local_sync_test_server.cc
@@ -29,7 +29,8 @@ LocalSyncTestServer::~LocalSyncTestServer() {}
bool LocalSyncTestServer::AddCommandLineArguments(
CommandLine* command_line) const {
- LocalTestServer::AddCommandLineArguments(command_line);
+ if (!LocalTestServer::AddCommandLineArguments(command_line))
+ return false;
if (xmpp_port_ != 0) {
std::string xmpp_port_str = base::IntToString(xmpp_port_);
command_line->AppendArg("--xmpp-port=" + xmpp_port_str);

Powered by Google App Engine
This is Rietveld 408576698