Added tests to previously broken functionality and added null checks.

tools/dom/scripts/htmldartgenerator.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."""
 
 import emitter
 from generator import AnalyzeOperation, ConstantOutputOrder, \
@@ skipping 222 matching lines @@
       version[0] += 1
       generate_call(stmts_emitter, call_emitter,
           version[0], signature_index, argument_count)
 
     def GenerateChecksAndCall(signature_index, argument_count):
       checks = []
       for i in range(0, argument_count):
         argument = signatures[signature_index][i]
         parameter_name = parameter_names[i]
         test_type = self._DartType(argument.type.id)
+        nullable = False or argument.type.nullable

[Anton Muhin, 2013/05/21 13:24:08]

False or?

[Andrei Mouravski, 2013/05/21 18:39:14]

I am not a smart person... :-) 

On 2013/05/21 13:24:08, Anton Muhin wrote:

+
         if test_type in ['dynamic', 'Object']:
           checks.append('?%s' % parameter_name)
         elif not can_omit_type_check(test_type, i):
+          nullable = True
           checks.append('(%s is %s || %s == null)' % (
               parameter_name, test_type, parameter_name))
+
+        if nullable is False:

[Anton Muhin, 2013/05/21 13:24:08]

nit: if not nullable

[Andrei Mouravski, 2013/05/21 18:39:14]

Done.

+          for signature in signatures:
+            if (argument not in signature or (len(signature) > i and

[Anton Muhin, 2013/05/21 13:24:08]

why still argument not in signagture check

[Andrei Mouravski, 2013/05/21 18:39:14]

Done.

+                                              signature[i] != argument)):
+              checks.append('%s != null' % parameter_name)
+              break
+
       # There can be multiple presence checks.  We need them all since a later
       # optional argument could have been passed by name, leaving 'holes'.
       checks.extend(['!?%s' % name for name in parameter_names[argument_count:]])
 
       GenerateCall(signature_index, argument_count, checks)
 
     # TODO: Optimize the dispatch to avoid repeated checks.
     if len(signatures) > 1:
       index_swaps = {}
       for signature_index, signature in enumerate(signatures):
@@ skipping 322 matching lines @@
     if interface.parents:
       parent = interface.parents[0]
       if IsPureInterface(parent.type.id):
         walk(interface.parents)
       else:
         walk(interface.parents[1:])
     return result
 
   def _DartType(self, type_name):
     return self._type_registry.DartType(type_name)