Canvas2D Performance: fix the bottleneck of hasInstance during JS binding -- overloading

Source/bindings/scripts/v8_interface.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 # coding=utf-8
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 735 matching lines @@
     # • an interface type that V implements
     # (Unlike most of these tests, this can return multiple methods, since we
     #  test if it implements an interface. Thus we need a for loop, not a next.)
     # (We distinguish wrapper types from built-in interface types.)
     for idl_type, method in ((idl_type, method)
                              for idl_type, method in idl_types_methods
                              if idl_type.is_wrapper_type):
         test = 'V8{idl_type}::hasInstance({cpp_value}, info.GetIsolate())'.format(idl_type=idl_type.base_type, cpp_value=cpp_value)
         yield test, method
 
+    for argument, method in arguments_methods:
+        if argument['idl_type_object'].is_wrapper_type and method['number_of_required_arguments'] == method['number_of_arguments']:

[yunchao, 2014/10/02 10:51:56]

both number_of_required_arguments and number_of_arguments are elements in
method_context. But It reports an error for method['number_of_arguments'],
although method['number_of_required_arguments'] is OK. Three days ago, both of
method['number_of_required_arguments'] and method['number_of_arguments'] are OK.
Do you know why?

+            argument['type_checked_already'] = True
+
+
     # 8. Otherwise: if V is any kind of object except for a native Date object,
     # a native RegExp object, and there is an entry in S that has one of the
     # following types at position i of its type list,
     # • an array type
     # • a sequence type
     # ...
     # • a dictionary
     #
     # FIXME:
     # We don't strictly follow the algorithm here. The algorithm says "remove
@@ skipping 367 matching lines @@
         deleter = next(
             method
             for method in interface.operations
             if ('deleter' in method.specials and
                 len(method.arguments) == 1 and
                 str(method.arguments[0].idl_type) == 'DOMString'))
     except StopIteration:
         return None
 
     return property_deleter(deleter)