Refactoring DOM annotations and comment generation, adding DOM API triage list

tools/dom/scripts/htmleventgenerator.py

 #!/usr/bin/python
 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 """This module provides functionality to generate dart:html event classes."""
 
 import logging
 import monitored
-from generator import GetAnnotationsAndComments, FormatAnnotationsAndComments
 
 _logger = logging.getLogger('dartgenerator')
 
 # Events without onEventName attributes in the  IDL we want to support.
 # We can automatically extract most event names by checking for
 # onEventName methods in the IDL but some events aren't listed so we need
 # to manually add them here so that they are easy for users to find.
 _html_manual_events = monitored.Dict('htmleventgenerator._html_manual_events', {
   'Element': ['touchleave', 'touchenter', 'webkitTransitionEnd'],
   'Window': ['DOMContentLoaded']
@@ skipping 186 matching lines @@
   'XMLHttpRequestUpload.progress': ('progress', 'ProgressEvent'),
 })
 
 # These classes require an explicit declaration for the "on" method even though
 # they don't declare any unique events, because the concrete class hierarchy
 # doesn't match the interface hierarchy.
 _html_explicit_event_classes = set(['DocumentFragment'])
 
 class HtmlEventGenerator(object):
 
-  def __init__(self, database, renamer, template_loader):
+  def __init__(self, database, renamer, metadata, template_loader):
     self._event_classes = set()
     self._database = database
     self._renamer = renamer
+    self._metadata = metadata
     self._template_loader = template_loader
 
   def EmitStreamProviders(self, interface, custom_events,
       members_emitter, library_name):
     events = self._GetEvents(interface, custom_events)
     if not events:
       return
 
     for event_info in events:
       (dom_name, html_name, event_type) = event_info
       annotation_name = dom_name + 'Event'
 
       # If we're using a different provider, then don't declare one.
       if self._GetEventRedirection(interface, html_name, event_type):
         continue
 
-      annotations = FormatAnnotationsAndComments(
-          GetAnnotationsAndComments(library_name, interface.id,
-                                    annotation_name), '  ')
+      annotations = self._metadata.FormatMetadata(
+          self._metadata.GetMetadata(library_name, interface.id,
+              annotation_name, 'on' + dom_name), '  ')
 
       members_emitter.Emit(
           "\n"
           "  $(ANNOTATIONS)static const EventStreamProvider<$TYPE> "
           "$(NAME)Event = const EventStreamProvider<$TYPE>('$DOM_NAME');\n",
           ANNOTATIONS=annotations,
           NAME=html_name,
           DOM_NAME=dom_name,
           TYPE=event_type)
 
   def EmitStreamGetters(self, interface, custom_events,
       members_emitter, library_name):
     events = self._GetEvents(interface, custom_events)
     if not events:
       return
 
     for event_info in events:
       (dom_name, html_name, event_type) = event_info
       getter_name = 'on%s%s' % (html_name[:1].upper(), html_name[1:])
       annotation_name = 'on' + dom_name
 
       # If the provider is declared elsewhere, point to that.
       redirection = self._GetEventRedirection(interface, html_name, event_type)
       if redirection:
         provider = '%s.%sEvent' % (redirection, html_name)
       else:
         provider = html_name + 'Event'
 
-      annotations = FormatAnnotationsAndComments(
-          GetAnnotationsAndComments(library_name, interface.id,
-                                    annotation_name), '  ')
+      annotations = self._metadata.GetFormattedMetadata(
+          library_name, interface.id, annotation_name, '  ')
 
       members_emitter.Emit(
           "\n"
           "  $(ANNOTATIONS)Stream<$TYPE> get $(NAME) => "
           "$PROVIDER.forTarget(this);\n",
           ANNOTATIONS=annotations,
           NAME=getter_name,
           PROVIDER=provider,
           TYPE=event_type)
 
@@ skipping 67 matching lines @@
     """
     key = '%s.%s' % (html_interface_name, dom_event_name)
     if key in _html_event_types:
       return _html_event_types[key]
     key = '*.%s' % dom_event_name
     if key in _html_event_types:
       return _html_event_types[key]
     _logger.warn('Cannot resolve event type for %s.%s' %
         (html_interface_name, dom_event_name))
     return None