diff options
author | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-27 01:52:52 +0000 |
---|---|---|
committer | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-27 01:52:52 +0000 |
commit | b2cc96d0f23779ed8b2491227d13cc8f984f4585 (patch) | |
tree | 358b4d0ffa7ff1aa273411a1ed5350cffbaf1697 /net/test/test_server.cc | |
parent | 893d7214c49df970ed4abfc69060a48c9b880b22 (diff) | |
download | chromium_src-b2cc96d0f23779ed8b2491227d13cc8f984f4585.zip chromium_src-b2cc96d0f23779ed8b2491227d13cc8f984f4585.tar.gz chromium_src-b2cc96d0f23779ed8b2491227d13cc8f984f4585.tar.bz2 |
Made testserver communicate to parent process with JSON
This is so that if the testserver needs to communicate anything more than
the port in the future (e.g., xmpp port for the test sync server), it
can do so in a flexible manner.
BUG=53934
TEST=manually
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=66879
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67018
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67386
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67398
Review URL: http://codereview.chromium.org/5196001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67481 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/test/test_server.cc')
-rw-r--r-- | net/test/test_server.cc | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/net/test/test_server.cc b/net/test/test_server.cc index 1ae2c5b..1fd20c9 100644 --- a/net/test/test_server.cc +++ b/net/test/test_server.cc @@ -17,11 +17,14 @@ #include "base/base64.h" #include "base/command_line.h" #include "base/debug/leak_annotations.h" +#include "base/json/json_reader.h" #include "base/file_util.h" #include "base/logging.h" #include "base/path_service.h" +#include "base/scoped_ptr.h" #include "base/string_number_conversions.h" #include "base/utf_string_conversions.h" +#include "base/values.h" #include "googleurl/src/gurl.h" #include "net/base/cert_test_util.h" #include "net/base/host_port_pair.h" @@ -378,4 +381,28 @@ bool TestServer::AddCommandLineArguments(CommandLine* command_line) const { return true; } +bool TestServer::ParseServerData(const std::string& server_data) { + VLOG(1) << "Server data: " << server_data; + base::JSONReader json_reader; + scoped_ptr<Value> value(json_reader.JsonToValue(server_data, true, false)); + if (!value.get() || + !value->IsType(Value::TYPE_DICTIONARY)) { + LOG(ERROR) << "Could not parse server data: " + << json_reader.GetErrorMessage(); + return false; + } + DictionaryValue* dict = static_cast<DictionaryValue*>(value.get()); + int port = 0; + if (!dict->GetInteger("port", &port)) { + LOG(ERROR) << "Could not find port value"; + return false; + } + if ((port <= 0) || (port >= kuint16max)) { + LOG(ERROR) << "Invalid port value: " << port; + return false; + } + host_port_pair_.set_port(port); + return true; +} + } // namespace net |