Fixed clean URLs for GCS file system provider in the documentation server.

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

 # Copyright 2014 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.
 
 from third_party.cloudstorage import cloudstorage_api
 from third_party.cloudstorage import common
 from third_party.cloudstorage import errors
 
 from docs_server_utils import StringIdentity
 from file_system import FileSystem, FileNotFoundError, StatInfo
@@ skipping 21 matching lines @@
   try:
     with cloudstorage_api.open('/' + filename, 'r') as f:
       return f.read()
   except errors.Error:
     raise FileNotFoundError('Read failed for %s: %s' % (filename,
         traceback.format_exc()))
 
 def _ListDir(dir_name):
   AssertIsDirectory(dir_name)
   try:
-    files = cloudstorage_api.listbucket('/' + dir_name)
-    return [os_path.filename.lstrip('/') for os_path in files]
+    files = cloudstorage_api.listbucket('/' + dir_name, delimiter='/')
+    return [os_path.filename.lstrip('/')[len(dir_name):] for os_path in files]
   except errors.Error:
     raise FileNotFoundError('cloudstorage.listbucket failed for %s: %s' %
                             (dir_name, traceback.format_exc()))
 
 def _CreateStatInfo(bucket, path):
   full_path = Join(bucket, path)
   last_commit_file = Join(bucket, LAST_COMMIT_HASH_FILENAME)
   try:
     last_commit = _ReadFile(last_commit_file)
     if IsDirectory(full_path):
       child_versions = dict()
       # Fetching stats for all files under full_path, recursively. The
       # listbucket method uses a prefix approach to simulate hierarchy,
       # but calling it without the "delimiter" argument searches for prefix,
       # which means, for directories, everything beneath it.
       for _file in cloudstorage_api.listbucket('/' + full_path):

[not at google - send to devlin, 2014/02/14 23:03:55]

p.s. why doesn't this use ListDir(..)?

[Renato Mangini (chromium), 2014/02/15 02:01:52]

Old code, didn't go through it. Fixing now, thanks for pointing it out.

         filename = _file.filename.lstrip('/')[len(full_path):]
         child_versions[filename] = last_commit
     else:
       child_versions = None
     return StatInfo(last_commit, child_versions)
   except (TypeError, errors.Error):
     raise FileNotFoundError('cloudstorage.stat failed for %s: %s' % (path,
                             traceback.format_exc()))
 
 class CloudStorageFileSystem(FileSystem):
@@ skipping 50 matching lines @@
   def _warnAboutAuthError(self):
     logging.warn(('Authentication error on Cloud Storage. Check if your'
                   ' appengine project has permissions to Read the GCS'
                   ' buckets. If you are running a local appengine server,'
                   ' you need to set an access_token in'
                   ' local_debug/gcs_debug.conf.'
                   ' Remember that this token expires in less than 10'
                   ' minutes, so keep it updated. See'
                   ' gcs_file_system_provider.py for instructions.'));
     logging.debug(traceback.format_exc())