Extensions Docs Server: Proper path handling

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

Index: chrome/common/extensions/docs/server2/server_instance.py
diff --git a/chrome/common/extensions/docs/server2/server_instance.py b/chrome/common/extensions/docs/server2/server_instance.py
index 53bf38cc9debbb70b1178babdfe76edfcb7f7bcd..2fa2a953c5b4bbce80b7d23d5c2c48eecfec9001 100644
--- a/chrome/common/extensions/docs/server2/server_instance.py
+++ b/chrome/common/extensions/docs/server2/server_instance.py
@@ -36,11 +36,13 @@ class ServerInstance(object):
   def Get(self, path, request, response):
     templates = self._template_data_source_factory.Create(request)
 
-    if fnmatch(path, 'examples/*.zip'):
-      content = self._example_zipper.Create(path[:-len('.zip')])
+    if fnmatch(path, 'extensions/examples/*.zip'):
+      content = self._example_zipper.Create(
+          path[len('extensions/'):-len('.zip')])
       response.headers['content-type'] = mimetypes.types_map['.zip']
-    elif path.startswith('examples/'):
-      content = self._cache.GetFromFile(DOCS_PATH + '/' + path)
+    elif path.startswith('extensions/examples/'):
+      content = self._cache.GetFromFile(
+          DOCS_PATH + '/' + path[len('extensions/'):])

[not at google - send to devlin, 2012/08/06 23:13:46]

I presume the apps samples page will be set up to serve .zip files differently
then?

I mean, yeah, ideally we should just let github deal with it. I'll need to look
at the apps samples review again (will definitely get around to doing that
today..).

[cduvall, 2012/08/06 23:14:55]

Right now they're just linked to the github page.

       response.headers['content-type'] = 'text/plain'
     elif path.startswith('static/'):
       content = self._FetchStaticResource(path, response)