Docs: Remove unnecessary redirects from directories to 'index' files since we

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

 # Copyright 2013 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 mimetypes
 import posixpath
 import traceback
 
 from compiled_file_system import SingleFile
 from directory_zipper import DirectoryZipper
 from docs_server_utils import ToUnicode
 from file_system import FileNotFoundError
 from future import Future
 from path_canonicalizer import PathCanonicalizer
-from path_util import AssertIsValid, Join, ToDirectory
+from path_util import AssertIsValid, IsDirectory, Join, ToDirectory
 from special_paths import SITE_VERIFICATION_FILE
 from third_party.handlebar import Handlebar
 from third_party.markdown import markdown
 
 
 _MIMETYPE_OVERRIDES = {
   # SVG is not supported by mimetypes.guess_type on AppEngine.
   '.svg': 'image/svg+xml',
 }
 
@@ skipping 114 matching lines @@
           lambda: ContentAndType(zip_future.Get(), 'application/zip', None))
 
     # If there is no file extension, look for a file with one of the default
     # extensions. If one cannot be found, check if the path is a directory.
     # If it is, then check for an index file with one of the default
     # extensions.
     if not ext:
       new_path = self._AddExt(path)
       # Add a trailing / to check if it is a directory and not a file with
       # no extension.
-      if new_path is None and self.file_system.Exists(path + '/').Get():
-        new_path = self._AddExt(path + '/index')
+      if new_path is None and self.file_system.Exists(ToDirectory(path)).Get():
+        new_path = self._AddExt(Join(path, 'index'))
+        # If an index file wasn't found in this directly then we're never going
+        # to find a file.
+        if new_path is None:
+          return FileNotFoundError.RaiseInFuture('"%s" is a directory' % path)
       if new_path is not None:
         path = new_path
 
     return self._content_cache.GetFromFile(path)
 
   def _AddExt(self, path):
     '''Tries to append each of the default file extensions to path and returns
     the first one that is an existing file.
     '''
     for default_ext in self._default_extensions:
@@ skipping 16 matching lines @@
                           self.GetContentAndType(Join(root, base))))
       # TODO(kalman): Cache .zip files for each directory (if supported).
     def resolve():
       for label, future in futures:
         try: future.Get()
         except: logging.error('%s: %s' % (label, traceback.format_exc()))
     return Future(callback=resolve)
 
   def __repr__(self):
     return 'ContentProvider of <%s>' % repr(self.file_system)