Extensions Docs Server: Split apps from extensions

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

Index: chrome/common/extensions/docs/server2/branch_utility.py
diff --git a/chrome/common/extensions/docs/server2/branch_utility.py b/chrome/common/extensions/docs/server2/branch_utility.py
index b1470c2e38c43593282adf2962ec20a2ead361af..166c1a9b52f1ccd5e26041268a1cc6080eea47c0 100644
--- a/chrome/common/extensions/docs/server2/branch_utility.py
+++ b/chrome/common/extensions/docs/server2/branch_utility.py
@@ -15,15 +15,22 @@ class BranchUtility(object):
     self._memcache = memcache
 
   def SplitChannelNameFromPath(self, path):
+    prefix = ''
+    if path.startswith('extensions/'):
+      prefix = 'extensions/'
+      path = path[len('extensions/'):]
+    elif path.startswith('apps/'):
+      prefix = 'apps/'
+      path = path[len('apps/'):]

[not at google - send to devlin, 2012/08/06 08:38:33]

Is all of this necessary? The paths will be

http://developer.chrome.com/trunk/apps/storage.html vs
http://developer.chrome.com/trunk/apps/about_apps.storage

so the branch will come first?

or am I misunderstanding it?

[cduvall, 2012/08/06 18:36:16]

In the old docs, the branch came last. For example, the url would be
http://code.google.com/chrome/extensions/trunk/storage.html

Do you think we should change it to the branch coming first? I like it better,
but people switching from the old docs to the new docs might prefer to have it
the old way.

     try:
       first, second = path.split('/', 1)
     except ValueError:
       first = path
       second = ''
     if first in ['trunk', 'dev', 'beta', 'stable']:
-      return (first, second)
+      return (first, prefix + second)
     else:
-      return (self._default_branch, path)
+      return (self._default_branch, prefix + path)
 
   def GetBranchNumberForChannelName(self, channel_name):
     """Returns an empty string if the branch number cannot be found.