[bindings] partial interfaces should not violate componentization

Source/bindings/scripts/idl_definitions.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # 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
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 257 matching lines @@
 class IdlInterface(object):
     def __init__(self, idl_name, node=None):
         self.attributes = []
         self.constants = []
         self.constructors = []
         self.custom_constructors = []
         self.extended_attributes = {}
         self.operations = []
         self.parent = None
         self.stringifier = None
+        self.original_interface = None

[bashi, 2014/10/10 09:48:11]

This looks strange to me.. My understanding is that idl definitions only have
information which come from idl parser. All other properties are set in this
initializer, but |original_interface| and |partial_interfaces| are not. I think
this implies that there would be a better place to hold such information.

[tasak, 2014/10/10 11:43:40]

So interface_dependency_resolver updates idl definitions. This looks also
strange?

[bashi, 2014/10/15 05:29:22]

That means we have two paths; the first path reads an idl file and create
definitions, the second path updates (or additionary creates) definitions based
on dependencies. I feel that it might be clear that if we could explicitly treat
these information separately (information come from each path). But I'm not sure
it's worth doing, so I'm ok with this.

[tasak, 2014/10/15 11:24:18]

As far as I know, we can obtain valid IdlDefinition by using only idl_reader,
not idl_parser.

+        self.partial_interfaces = []
         if not node:  # Early exit for IdlException.__init__
             return
 
         self.is_callback = node.GetProperty('CALLBACK') or False
         self.is_exception = False
         # FIXME: uppercase 'Partial' => 'PARTIAL' in base IDL parser
         self.is_partial = node.GetProperty('Partial') or False
         self.idl_name = idl_name
         self.name = node.GetName()
 
@@ skipping 527 matching lines @@
     child_class = child.GetClass()
     if child_class != 'Type':
         raise ValueError('Unrecognized node class: %s' % child_class)
     return type_node_to_type(child)
 
 
 def union_type_node_to_idl_union_type(node):
     member_types = [type_node_to_type(member_type_node)
                     for member_type_node in node.GetChildren()]
     return IdlUnionType(member_types)