Add a PDF test to load all the pdfs in a test directory, using the test serve...

net/tools/testserver/testserver.py

 #!/usr/bin/python2.4
 # Copyright (c) 2006-2010 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """This is a simple HTTP server used for testing Chrome.
 
 It supports several test URLs, as specified by the handlers in TestPageHandler.
 By default, it listens on an ephemeral port and sends the port number back to
 the originating process over a pipe. The originating process can specify an
@@ skipping 203 matching lines @@
     put_handlers = [
       self.EchoTitleHandler,
       self.EchoAllHandler,
       self.EchoHandler] + get_handlers
 
     self._mime_types = {
       'crx' : 'application/x-chrome-extension',
       'gif': 'image/gif',
       'jpeg' : 'image/jpeg',
       'jpg' : 'image/jpeg',
-      'xml' : 'text/xml'
+      'xml' : 'text/xml',
+      'pdf' : 'application/pdf'
     }
     self._default_mime_type = 'text/html'
 
     BasePageHandler.__init__(self, request, client_address, socket_server,
                              connect_handlers, get_handlers, post_handlers,
                              put_handlers)
 
   def GetMIMETypeFromName(self, file_name):
     """Returns the mime type for the specified file_name. So far it only looks
     at the file extension."""
@@ skipping 456 matching lines @@
       for line in f:
         header_values = re.findall('(\S+):\s*(.*)', line)
         if len(header_values) > 0:
           # "name: value"
           name, value = header_values[0]
           self.send_header(name, value)
       f.close()
     else:
       # Could be more generic once we support mime-type sniffing, but for
       # now we need to set it explicitly.
-      self.send_response(200)
+
+      range = self.headers.get('Range')
+      if range and range.startswith('bytes='):
+        # Note this doesn't handle all valid byte range values (i.e. open ended
+        # ones), just enough for what we needed so far.
+        range = range[6:].split('-')
+        start = int(range[0])
+        end = int(range[1])
+
+        self.send_response(206)
+        content_range = 'bytes ' + str(start) + '-' + str(end) + '/' + \
+                        str(len(data))
+        self.send_header('Content-Range', content_range)
+        data = data[start: end + 1]
+      else:
+        self.send_response(200)

[Nico, 2010/11/17 14:02:19]

should you send some error code if range and not range.startswith('bytes=')?

[jam, 2010/11/17 16:53:15]

given how primitive this server is, I don't think it's worth our time to start
making it to be spec compliant...  if chrome suddenly starts sending different
byte range requests, the test I added will fail and it'll be easy to track down.

+
       self.send_header('Content-type', self.GetMIMETypeFromName(file_path))
+      self.send_header('Accept-Ranges', 'bytes')
       self.send_header('Content-Length', len(data))
+      self.send_header('ETag', '\'' + file_path + '\'')
     self.end_headers()
 
     self.wfile.write(data)
 
     return True
 
   def RealFileWithCommonHeaderHandler(self):
     """This handler sends the contents of the requested file without the pseudo
     http head!"""
 
@@ skipping 641 matching lines @@
                            'option may appear multiple times, indicating '
                            'multiple algorithms should be enabled.');
   option_parser.add_option('', '--file-root-url', default='/files/',
                            help='Specify a root URL for files served.')
   option_parser.add_option('', '--startup-pipe', type='int',
                            dest='startup_pipe',
                            help='File handle of pipe to parent process')
   options, args = option_parser.parse_args()
 
   sys.exit(main(options, args))