Docserver: Serve docs out of src/ not src/chrome/common/extensions. This allows

chrome/common/extensions/docs/server2/subversion_file_system.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 posixpath
 import traceback
 import xml.dom.minidom as xml
 from xml.parsers.expat import ExpatError
 
 from appengine_url_fetcher import AppEngineUrlFetcher
 from docs_server_utils import StringIdentity
 from file_system import (
     FileNotFoundError, FileSystem, FileSystemError, StatInfo, ToUnicode)
 from future import Future
-import svn_constants
 import url_constants
 
+
 def _ParseHTML(html):
   '''Unfortunately, the viewvc page has a stray </div> tag, so this takes care
   of all mismatched tags.
   '''
   try:
     return xml.parseString(html)
   except ExpatError as e:
     return _ParseHTML('\n'.join(
         line for (i, line) in enumerate(html.split('\n'))
         if e.lineno != i + 1))
@@ skipping 108 matching lines @@
     if self._error is not None:
       raise self._error
     return self._value
 
 class SubversionFileSystem(FileSystem):
   '''Class to fetch resources from src.chromium.org.
   '''
   @staticmethod
   def Create(branch='trunk', revision=None):
     if branch == 'trunk':
-      svn_path = 'trunk/src/%s' % svn_constants.EXTENSIONS_PATH
+      svn_path = 'trunk/src'
     else:
-      svn_path = 'branches/%s/src/%s' % (branch, svn_constants.EXTENSIONS_PATH)
+      svn_path = 'branches/%s/src' % branch
     return SubversionFileSystem(
         AppEngineUrlFetcher('%s/%s' % (url_constants.SVN_URL, svn_path)),
         AppEngineUrlFetcher('%s/%s' % (url_constants.VIEWVC_URL, svn_path)),
         svn_path,
         revision=revision)
 
   def __init__(self, file_fetcher, stat_fetcher, svn_path, revision=None):
     self._file_fetcher = file_fetcher
     self._stat_fetcher = stat_fetcher
     self._svn_path = svn_path
     self._revision = revision
 
   def Read(self, paths, binary=False):
     args = None
     if self._revision is not None:
       # |fetcher| gets from svn.chromium.org which uses p= for version.
       args = 'p=%s' % self._revision
     return Future(delegate=_AsyncFetchFuture(paths,
                                              self._file_fetcher,
                                              binary,
                                              args=args))
 
   def Refresh(self):
     return Future(value=())
 
   def Stat(self, path):
     directory, filename = posixpath.split(path)
-    directory += '/'
     if self._revision is not None:
       # |stat_fetch| uses viewvc which uses pathrev= for version.
       directory += '?pathrev=%s' % self._revision
 
     try:
       result = self._stat_fetcher.Fetch(directory)
     except Exception as e:
       raise FileSystemError('Error fetching %s for Stat: %s' %
           (path, traceback.format_exc()))
 
     if result.status_code == 404:
       raise FileNotFoundError('Got 404 when fetching %s for Stat, content %s' %
           (path, result.content))
     if result.status_code != 200:
       raise FileNotFoundError('Got %s when fetching %s for Stat, content %s' %
           (result.status_code, path, result.content))
 
     stat_info = _CreateStatInfo(result.content)
     if stat_info.version is None:
       raise FileSystemError('Failed to find version of dir %s' % directory)
-    if path.endswith('/'):
+    if path == '' or path.endswith('/'):
       return stat_info
     if filename not in stat_info.child_versions:
       raise FileNotFoundError(
           '%s from %s was not in child versions for Stat' % (filename, path))
     return StatInfo(stat_info.child_versions[filename])
 
   def GetIdentity(self):
     # NOTE: no revision here, since it would mess up the caching of reads. It
     # probably doesn't matter since all the caching classes will use the result
     # of Stat to decide whether to re-read - and Stat has a ceiling of the
     # revision - so when the revision changes, so might Stat. That is enough.
     return '@'.join((self.__class__.__name__, StringIdentity(self._svn_path)))