summaryrefslogtreecommitdiffstats
path: root/net/tools
diff options
context:
space:
mode:
authorakalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-11-25 07:54:10 +0000
committerakalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-11-25 07:54:10 +0000
commitf73ecb005462a2d7de56979ef9168452559ea71c (patch)
tree01df584fe7872bb556407ed6b656199881777ec5 /net/tools
parent2ff778c5748038872c0c6b7df04c834b005167fa (diff)
downloadchromium_src-f73ecb005462a2d7de56979ef9168452559ea71c.zip
chromium_src-f73ecb005462a2d7de56979ef9168452559ea71c.tar.gz
chromium_src-f73ecb005462a2d7de56979ef9168452559ea71c.tar.bz2
Revert 67386 - 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 Review URL: http://codereview.chromium.org/5196001 TBR=akalin@chromium.org Review URL: http://codereview.chromium.org/5343003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67393 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/tools')
-rwxr-xr-xnet/tools/testserver/testserver.py22
1 files changed, 4 insertions, 18 deletions
diff --git a/net/tools/testserver/testserver.py b/net/tools/testserver/testserver.py
index b69eb2f..2551139 100755
--- a/net/tools/testserver/testserver.py
+++ b/net/tools/testserver/testserver.py
@@ -27,13 +27,6 @@ import time
import urlparse
import warnings
-# If we use simplejson always, we get some warnings when we run under
-# 2.6.
-if sys.version_info < (2, 6):
- import simplejson as json
-else:
- import json
-
# Ignore deprecation warnings, they make our output more cluttered.
warnings.filterwarnings("ignore", category=DeprecationWarning)
@@ -1328,22 +1321,15 @@ def main(options, args):
# Notify the parent that we've started. (BaseServer subclasses
# bind their sockets on construction.)
if options.startup_pipe is not None:
- server_data = {
- 'port': listen_port
- }
- server_data_json = json.dumps(server_data)
- debug('sending server_data: %s' % server_data_json)
- server_data_len = len(server_data_json)
if sys.platform == 'win32':
fd = msvcrt.open_osfhandle(options.startup_pipe, 0)
else:
fd = options.startup_pipe
startup_pipe = os.fdopen(fd, "w")
- # First write the data length as an unsigned 4-byte value. This
- # is _not_ using network byte ordering since the other end of the
- # pipe is on the same machine.
- startup_pipe.write(struct.pack('=L', server_data_len))
- startup_pipe.write(server_data_json)
+ # Write the listening port as a 2 byte value. This is _not_ using
+ # network byte ordering since the other end of the pipe is on the same
+ # machine.
+ startup_pipe.write(struct.pack('@H', listen_port))
startup_pipe.close()
try: