Docserver: Display API features that are available to content scripts

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

Index: chrome/common/extensions/docs/server2/api_models.py
diff --git a/chrome/common/extensions/docs/server2/api_models.py b/chrome/common/extensions/docs/server2/api_models.py
index 673dc434661fe18a297217db4f0bbf02d84716c2..e4e38c59f2d98fe0f7fd405167c10994a035d16e 100644
--- a/chrome/common/extensions/docs/server2/api_models.py
+++ b/chrome/common/extensions/docs/server2/api_models.py
@@ -5,17 +5,37 @@
 import posixpath
 
 from compiled_file_system import SingleFile, Unicode
-from docs_server_utils import StringIdentity
 from extensions_paths import API_PATHS
-from features_bundle import HasParentFeature
+from features_bundle import HasParentFeature, GetParentFeature
 from file_system import FileNotFoundError
 from future import Collect, Future
+from operator import itemgetter
 from platform_util import PlatformToExtensionType
 from schema_util import ProcessSchema
 from third_party.json_schema_compiler.json_schema import DeleteNodes
 from third_party.json_schema_compiler.model import Namespace, UnixName
 
 
+class ContentScriptAPI(object):
+  '''Represents an API available to content scripts.
+  '''
+  def __init__(self, name):
+    self.name = name
+    self.restrictedTo = None

[not at google - send to devlin, 2014/07/16 00:32:20]

document these

+
+  def __eq__(self, o):
+    return self.name == o.name and self.restrictedTo == o.restrictedTo
+
+  def __ne__(self, o):
+    return not (self == o)
+
+  def __repr__(self):
+    return '<ContentScriptAPI name=%s, restrictedTo=%s>' % (name, restrictedTo)
+
+  def __str__(self):
+    return repr(self)
+
+
 class APIModels(object):
   '''Tracks APIs and their Models.
   '''
@@ -24,11 +44,13 @@ class APIModels(object):
                features_bundle,
                compiled_fs_factory,
                file_system,
+               object_store_creator,
                platform):
     self._features_bundle = features_bundle
     self._platform = PlatformToExtensionType(platform)
     self._model_cache = compiled_fs_factory.Create(
         file_system, self._CreateAPIModel, APIModels, category=self._platform)
+    self._object_store = object_store_creator.Create(APIModels)
 
   @SingleFile
   @Unicode
@@ -97,6 +119,53 @@ class APIModels(object):
       futures[0].Get()
     return Future(callback=resolve)
 
+  def GetContentScriptAPIs(self):
+    '''Creates a dict of APIs and nodes supported by content scripts in
+    this format:
+
+      {
+        'extension': '<ContentScriptAPI name='extension', restrictedTo=None>',
+        ...
+      }
+
+    where a ContentScriptAPI object's restrictedTo attribute items look like:
+
+      {
+        'parent': 'extension',
+        'node': 'sendMessage'
+      }
+    '''
+    content_script_apis_future = self._object_store.Get('content_script_apis')
+    api_features_future = self._features_bundle.GetAPIFeatures()
+    def resolve():
+      content_script_apis = content_script_apis_future.Get()
+      if content_script_apis is not None:
+        return content_script_apis
+
+      api_features = api_features_future.Get()
+      content_script_apis = {}
+      for feature in api_features.itervalues():
+        if 'content_script' not in feature.get('contexts', ()):
+          continue
+        parent = GetParentFeature(feature['name'], feature, api_features)
+        # If it has no parent, then the API is fully supported.
+        if parent is None:
+          content_script_apis[feature['name']] = ContentScriptAPI(
+              feature['name'])
+        else:
+          # Creates a dict for the individual node.
+          node = {'parent': parent, 'node': feature['name'][len(parent) + 1:]}

[not at google - send to devlin, 2014/07/16 00:32:20]

this 'parent' property is still a concern of the templates, right? can you just
make restrictedTo be a list of the features it's restricted to, parent a
property of ContentScriptAPI (since it should be constant for all restrictedTo?)
and let the data source(s) figure it out when they do their transformation from
model -> dict.

+          if parent not in content_script_apis:
+            content_script_apis[parent] = ContentScriptAPI(parent)
+          if content_script_apis[parent].restrictedTo:
+            content_script_apis[parent].restrictedTo.append(node)
+          else:
+            content_script_apis[parent].restrictedTo = [node]
+
+      self._object_store.Set('content_script_apis', content_script_apis)
+      return content_script_apis
+    return Future(callback=resolve)
+
   def Cron(self):
     futures = [self.GetModel(name) for name in self.GetNames()]
     return Collect(futures, except_pass=(FileNotFoundError, ValueError))