Extensions Docs Server: Efficient MemcacheFileSystem

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

Index: chrome/common/extensions/docs/server2/local_file_system.py
diff --git a/chrome/common/extensions/docs/server2/local_file_system.py b/chrome/common/extensions/docs/server2/local_file_system.py
index 89dfd9faba4bc729f1ca0b787779379100b70f2c..18fbe22b6753dba57d7155c75211767ccdb7ba05 100644
--- a/chrome/common/extensions/docs/server2/local_file_system.py
+++ b/chrome/common/extensions/docs/server2/local_file_system.py
@@ -52,10 +52,16 @@ class LocalFileSystem(file_system.FileSystem):
         result[path] = self._ReadFile(self._ConvertToFilepath(path), binary)
     return Future(value=result)
 
+  def _CreateStatInfo(self, path):
+    versions = dict((filename, os.stat(os.path.join(path, filename)).st_mtime)
+                    for filename in os.listdir(path))
+    return self.StatInfo(os.stat(path).st_mtime, versions)
+
   def Stat(self, path):
     try:
-      return self.StatInfo(
-          os.stat(os.path.join(self._base_path, path)).st_mtime)
+      if not path.endswith('/'):
+        return self.StatInfo(
+            os.stat(os.path.join(self._base_path, path)).st_mtime, {})

[not at google - send to devlin, 2012/08/13 05:34:15]

This logic is basically repeated in _CreateStatInfo, so put in there? This
method:

return self._CreateStatInfo(...)

above:

def _CreateStatInfo(self, path):
  def Mtime(path):
    return os.stat(path).st_mtime

  if path.endswith('/'):
    versions = dict((filename, Mtime(os.path.join(path, filename))
                     for filename in os.listdir(path))
  else:
    # note to cduvall: prefer None to {} here.
    versions = None
  return self.StatInfo(Mtime(path), versions)

[cduvall, 2012/08/13 19:45:45]

Done.

+      return self._CreateStatInfo(os.path.join(self._base_path, path))
     except OSError:
       raise file_system.FileNotFoundError(path)
-