Docserver: Make the hand-written Cron methods run first rather than last, since

chrome/common/extensions/docs/server2/permissions_data_source.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.
 
 from itertools import ifilter
 from operator import itemgetter
 
 from data_source import DataSource
 import features_utility as features
+from future import Gettable, Future
 from svn_constants import PRIVATE_TEMPLATE_PATH
 from third_party.json_schema_compiler.json_parse import Parse
 
 def _ListifyPermissions(permissions):
   '''Filter out any permissions that do not have a description or with a name
   that ends with Private then sort permissions features by name into a list.
   '''
   def filter_permissions(perm):
     return 'description' in perm and not perm['name'].endswith('Private')
 
@@ skipping 23 matching lines @@
   '''Load and format permissions features to be used by templates.
   '''
   def __init__(self, server_instance, request):
     self._features_bundle = server_instance.features_bundle
     self._object_store = server_instance.object_store_creator.Create(
         PermissionsDataSource)
     self._template_cache = server_instance.compiled_fs_factory.ForTemplates(
         server_instance.host_file_system_provider.GetTrunk())
 
   def _CreatePermissionsData(self):
-    api_features = self._features_bundle.GetAPIFeatures()
-    permission_features = self._features_bundle.GetPermissionFeatures()
+    api_features_future = self._features_bundle.GetAPIFeatures()
+    permission_features_future = self._features_bundle.GetPermissionFeatures()
+    def resolve():
+      permission_features = permission_features_future.Get()
+      _AddDependencyDescriptions(permission_features, api_features_future.Get())
 
-    def filter_for_platform(permissions, platform):
-      return _ListifyPermissions(features.Filtered(permissions, platform))
+      # Turn partial templates into descriptions, ensure anchors are set.
+      for permission in permission_features.values():
+        if not 'anchor' in permission:
+          permission['anchor'] = permission['name']
+        if 'partial' in permission:
+          permission['description'] = self._template_cache.GetFromFile('%s/%s' %
+              (PRIVATE_TEMPLATE_PATH, permission['partial'])).Get()
+          del permission['partial']
 
-    _AddDependencyDescriptions(permission_features, api_features)
-    # Turn partial templates into descriptions, ensure anchors are set.
-    for permission in permission_features.values():
-      if not 'anchor' in permission:
-        permission['anchor'] = permission['name']
-      if 'partial' in permission:
-        permission['description'] = self._template_cache.GetFromFile('%s/%s' %
-            (PRIVATE_TEMPLATE_PATH, permission['partial'])).Get()
-        del permission['partial']
-
-    return {
-      'declare_apps': filter_for_platform(permission_features, 'apps'),
-      'declare_extensions': filter_for_platform(
-          permission_features, 'extensions')
-    }
+      def filter_for_platform(permissions, platform):
+        return _ListifyPermissions(features.Filtered(permissions, platform))
+      return {
+        'declare_apps': filter_for_platform(permission_features, 'apps'),
+        'declare_extensions': filter_for_platform(
+            permission_features, 'extensions')
+      }
+    return Future(delegate=Gettable(resolve))
 
   def _GetCachedPermissionsData(self):
     data = self._object_store.Get('permissions_data').Get()
     if data is None:
-      data = self._CreatePermissionsData()
+      data = self._CreatePermissionsData().Get()
       self._object_store.Set('permissions_data', data)
     return data
 
   def Cron(self):
-    # TODO(kalman): Implement this.
-    pass
+    return self._CreatePermissionsData()
 
   def get(self, key):
     return self._GetCachedPermissionsData().get(key)