Extensions Docs Server: Split apps from extensions

chrome/common/extensions/docs/server2/handler.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 logging
 import os
 import sys
 
 from appengine_wrappers import webapp
 from appengine_wrappers import memcache
@@ skipping 95 matching lines @@
     return BRANCH_URL + '/' + branch + '/src'
 
 class Handler(webapp.RequestHandler):
   def __init__(self, request, response, local_path=EXTENSIONS_PATH):
     self._local_path = local_path
     super(Handler, self).__init__(request, response)
 
   def _NavigateToPath(self, path):
     channel_name, real_path = BRANCH_UTILITY.SplitChannelNameFromPath(path)
     branch = BRANCH_UTILITY.GetBranchNumberForChannelName(channel_name)
-    if real_path == '':
-      real_path = 'index.html'
+    if real_path.strip('/') == 'apps':
+      real_path = 'apps/index.html'
+    if real_path.strip('/') == 'extensions':
+      real_path = 'extensions/index.html'

[not at google - send to devlin, 2012/08/06 22:01:24]

would be nice to do a webserver-y kind of thing and detect that these are
directories and serve index.html out of them (so like, generic handling code).
But that'd be a pain cos of all this come-from-subversion stuff.

TODO?

[cduvall, 2012/08/06 22:25:45]

Done.

     # TODO: This leaks Server instances when branch bumps.
     _GetInstanceForBranch(branch, self._local_path).Get(real_path,
                                                         self.request,
                                                         self.response)
 
   def get(self):
     path = self.request.path
     if '_ah/warmup' in path:
       logging.info('Warmup request.')
       if DEFAULT_BRANCH != 'local':
-        self._NavigateToPath('trunk/samples.html')
-      self._NavigateToPath('dev/samples.html')
-      self._NavigateToPath('beta/samples.html')
-      self._NavigateToPath('stable/samples.html')
+        self._NavigateToPath('extensions/trunk/samples.html')
+      self._NavigateToPath('extensions/dev/samples.html')
+      self._NavigateToPath('extensions/beta/samples.html')
+      self._NavigateToPath('extensions/stable/samples.html')
       return
 
     # Redirect paths like "directory" to "directory/". This is so relative file
     # paths will know to treat this as a directory.
     if os.path.splitext(path)[1] == '' and path[-1] != '/':
       self.redirect(path + '/')
-    path = path.replace('/chrome/extensions/', '')
+    path = path.replace('/chrome/', '')
     path = path.strip('/')
     self._NavigateToPath(path)