Refactor the event-generation code to locate it in systemhtml.py.

client/dom/scripts/systemhtml.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 shared functionality for the system to generate
 Dart:html APIs from the IDL database."""
 
 from systemfrog import *
 from systeminterface import *
@@ skipping 365 matching lines @@
   'waiting': 'waiting',
   'webkitAnimationEnd': 'animationEnd',
   'webkitAnimationIteration': 'animationIteration',
   'webkitAnimationStart': 'animationStart',
   'webkitfullscreenchange': 'fullscreenChange',
   'webkitfullscreenerror': 'fullscreenError',
   'webkitSpeechChange': 'speechChange',
   'webkitTransitionEnd': 'transitionEnd'
 }
 
+# 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'])
+
 def _OnAttributeToEventName(on_method):
   event_name = on_method.id[2:]
   if event_name in _on_attribute_to_event_name_mapping:
     return _on_attribute_to_event_name_mapping[event_name]
   else:
     return event_name
 
 def _DomToHtmlEvents(interface_id, events):
   event_names = set(map(_OnAttributeToEventName, events)) 
   if interface_id in _html_manual_events:
@@ skipping 93 matching lines @@
       return ['Events']
     else:
       names = []
       for interface in interfaces_with_events:
         names.append(interface.id + 'Events')
       return names
 
   def _ImplClassName(self, type_name):
     return '_' + type_name + 'Impl'
 
+  def GetEventAttributes(self, interface):
+    events =  set([attr for attr in interface.attributes
+                   if self._generator._IsEventAttribute(interface, attr)])
+
+    # Hack to generate no-op Element events for DocumentFragment.

[Jacob, 2012/03/07 23:24:05]

this comment can now be removed.

[nweiz, 2012/03/08 01:03:51]

Done.

+    if events or interface.id in _html_explicit_event_classes:
+      return events
+
 class HtmlSystem(System):
 
   def __init__(self, templates, database, emitters, output_dir, generator):
     super(HtmlSystem, self).__init__(
         templates, database, emitters, output_dir)
     self._shared = HtmlSystemShared(database, generator)
 
 class HtmlInterfacesSystem(HtmlSystem):
 
   def __init__(self, templates, database, emitters, output_dir, generator):
@@ skipping 112 matching lines @@
       self._members_emitter.Emit(
           '\n'
           '  $CTOR(int length);\n'
           '\n'
           '  $CTOR.fromList(List<$TYPE> list);\n'
           '\n'
           '  $CTOR.fromBuffer(ArrayBuffer buffer);\n',
         CTOR=self._interface.id,
         TYPE=DartType(element_type))
 
+    events = self._shared.GetEventAttributes(self._interface)
+    if events != None: self.AddEventAttributes(events)

[Jacob, 2012/03/07 23:24:05]

I believe we're using the style of placing the if clause on the next line.

[nweiz, 2012/03/08 01:03:51]

Done.

+
   def AddAttribute(self, getter, setter):
     html_getter_name = self._shared.RenameInHtmlLibrary(
       self._interface, getter.id, 'get:')
     html_setter_name = self._shared.RenameInHtmlLibrary(
       self._interface, getter.id, 'set:')
 
     if not html_getter_name:
       getter = None
     if not html_setter_name:
       setter = None
@@ skipping 110 matching lines @@
         NATIVESPEC=' native "' + native_spec + '"')
 
     if element_type:
       self.AddTypedArrayConstructors(element_type)
 
     # Emit a factory provider class for the constructor.
     constructor_info = AnalyzeConstructor(interface)
     if constructor_info:
       self._EmitFactoryProvider(interface_name, constructor_info)
 
+    events = self._shared.GetEventAttributes(interface)
+    if events != None: self.AddEventAttributes(events)

[Jacob, 2012/03/07 23:24:05]

why not just
if events:
  self.AddEventAttributes(events)

[nweiz, 2012/03/08 01:03:51]

Because we want to run AddEventAttributes even if events is empty.

[Jacob, 2012/03/08 18:38:09]

same comment as above for all cases where == set() is used

+
   def _EmitFactoryProvider(self, interface_name, constructor_info):
     template_file = 'factoryprovider_%s.darttemplate' % interface_name
     template = self._system._templates.TryLoad(template_file)
     if not template:
       template = self._system._templates.Load('factoryprovider.darttemplate')
 
     factory_provider = '_' + interface_name + 'FactoryProvider'
     emitter = self._system._ImplFileEmitter(factory_provider)
     emitter.Emit(
         template,
@@ skipping 459 matching lines @@
     if dom_name != 'Document':
       self._members_emitter.Emit(
           '  $(CLASSNAME)._wrap(ptr) : super._wrap(ptr);\n',
           CLASSNAME=self._class_name)
 
     # Emit a factory provider class for the constructor.
     constructor_info = AnalyzeConstructor(interface)
     if constructor_info:
       self._EmitFactoryProvider(interface_name, constructor_info)
 
+    events = self._shared.GetEventAttributes(self._interface)
+    if events != None: self.AddEventAttributes(events)
+
   def _EmitFactoryProvider(self, interface_name, constructor_info):
     template_file = 'factoryprovider_%s.darttemplate' % interface_name
     template = self._system._templates.TryLoad(template_file)
     if not template:
       template = self._system._templates.Load('factoryprovider.darttemplate')
 
     factory_provider = '_' + interface_name + 'FactoryProvider'
     emitter = self._system._ImplFileEmitter(factory_provider)
     emitter.Emit(
         template,
@@ skipping 465 matching lines @@
     # dispatch has removed f(X), leaving only f(Y), but there is no guarantee
     # that Y = Z-X, so we need to check for Y.
     true_code = emitter.Emit(
         '$(INDENT)if ($COND) {\n'
         '$!TRUE'
         '$(INDENT)}\n',
         COND=test, INDENT=indent)
     self.GenerateDispatch(
         true_code, info, indent + '  ', position + 1, positive)
     return True