Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Unified Diff: chrome/common/extensions/docs/server2/content_providers.py

Issue 63203002: Docserver: Make the hand-written Cron methods run first rather than last, since (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: logging Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/server2/content_providers.py
diff --git a/chrome/common/extensions/docs/server2/content_providers.py b/chrome/common/extensions/docs/server2/content_providers.py
index 97e9390343284ca793a0945aaf88844f295cacbb..067cac33dca29e5a4709c5dbd432d0b747a86a42 100644
--- a/chrome/common/extensions/docs/server2/content_providers.py
+++ b/chrome/common/extensions/docs/server2/content_providers.py
@@ -8,6 +8,7 @@ import posixpath
from chroot_file_system import ChrootFileSystem
from content_provider import ContentProvider
+from future import Gettable, Future
from svn_constants import JSON_PATH
from third_party.json_schema_compiler.memoize import memoize
@@ -88,5 +89,6 @@ class ContentProviders(object):
supports_zip=supports_zip)
def Cron(self):
- for name, config in self._GetConfig().iteritems():
- self._CreateContentProvider(name, config).Cron()
+ futures = [self._CreateContentProvider(name, config).Cron()
+ for name, config in self._GetConfig().iteritems()]
+ return Future(delegate=Gettable(lambda: [f.Get() for f in futures]))

Powered by Google App Engine
This is Rietveld 408576698