Extensions Docs Server: Samples zip files

chrome/common/extensions/docs/server2/server_instance.py

 # Copyright (c) 2012 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.
 
 import mimetypes
 import os
 
 STATIC_DIR_PREFIX = 'docs/server2/'
 
 class ServerInstance(object):
   """This class is used to hold a data source and fetcher for an instance of a
   server. Each new branch will get its own ServerInstance.
   """
-  def __init__(self, template_data_source, cache_builder):
+  def __init__(self, template_data_source, zip_data_source, cache_builder):
     self._template_data_source = template_data_source
+    self._zip_data_source = zip_data_source
     self._cache = cache_builder.build(lambda x: x)
     mimetypes.init()
 
   def _FetchStaticResource(self, path, request_handler):
     """Fetch a resource in the 'static' directory.
     """
     try:
       result = self._cache.get(STATIC_DIR_PREFIX + path)
       base, ext = os.path.splitext(path)
       request_handler.response.headers['content-type'] = (
           mimetypes.types_map[ext])
       return result
     except:
       return ''
 
   def Get(self, path, request_handler):
-    if path.startswith('static'):
+    if path.endswith('.zip'):

[not at google - send to devlin, 2012/07/11 00:35:09]

I'd be more comfortable if this was restricted specifically to the examples
directory (injected of course).

Maybe a glob like "%s/**/*.zip" % examples_dir (there's a glob library
somewhere). Or whatever glob refers to "directory under examples that's not the
toplevel directory".

We also need to check for a manifest.json in that directory before we zip it,
and make sure we return None if it isn't found.

A job for another class obviously; perhaps ZipDataSource can turn into that.
Call like ExampleZipper or something (hehe).

[cduvall, 2012/07/11 20:56:30]

Done. I used re because glob is only for the filesystem.

+      content = self._zip_data_source[path]
+      request_handler.response.headers['content-type'] = (
+          mimetypes.types_map['.zip'])
+    elif path.startswith('static'):
       content = self._FetchStaticResource(path, request_handler)
     else:
       content = self._template_data_source.Render(path)
     if content:
       request_handler.response.out.write(content)
     else:
       # TODO: Actual 404 page.
       request_handler.response.set_status(404);
       request_handler.response.out.write('File not found.')