[bindings] partial interfaces should not violate componentization

Source/bindings/scripts/v8_attributes.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 416 matching lines @@
     return '%s::%sAttr' % (namespace, content_attribute_name)
 
 
 ################################################################################
 # Attribute configuration
 ################################################################################
 
 # [Replaceable]
 def setter_callback_name(interface, attribute):
     cpp_class_name = cpp_name(interface)
+    namespace_name = [cpp_class_name]
+    if interface.original_interface:
+        namespace_name.append('Partial')
+    namespace_name.append('V8Internal')

[haraken, 2014/10/16 04:24:08]

Can we move actual_cpp_class and remove this code?

(Probably you need to define a helper method to get actual_cpp_class in
utilities.py.)

[tasak, 2014/10/17 07:38:18]

Done.
Added cpp_name_or_partial().

+
     extended_attributes = attribute.extended_attributes
     if (('Replaceable' in extended_attributes and
          'PutForwards' not in extended_attributes) or
         is_constructor_attribute(attribute)):
-        return '{0}V8Internal::{0}ForceSetAttributeOnThisCallback'.format(cpp_class_name)
+        return '%s::%sForceSetAttributeOnThisCallback' % (
+            ''.join(namespace_name), cpp_class_name)
     if attribute.is_read_only and 'PutForwards' not in extended_attributes:
         return '0'
-    return '%sV8Internal::%sAttributeSetterCallback' % (cpp_class_name, attribute.name)
+    return '%s::%sAttributeSetterCallback' % (''.join(namespace_name), attribute.name)
 
 
 # [DoNotCheckSecurity], [Unforgeable]
 def access_control_list(attribute):
     extended_attributes = attribute.extended_attributes
     access_control = []
     if 'DoNotCheckSecurity' in extended_attributes:
         do_not_check_security = extended_attributes['DoNotCheckSecurity']
         if do_not_check_security == 'Setter':
             access_control.append('v8::ALL_CAN_WRITE')
@@ skipping 43 matching lines @@
     lambda self: strip_suffix(self.base_type, 'Constructor'))
 
 
 def is_constructor_attribute(attribute):
     # FIXME: replace this with [ConstructorAttribute] extended attribute
     return attribute.idl_type.name.endswith('Constructor')
 
 
 def constructor_getter_context(interface, attribute, context):
     context['needs_constructor_getter_callback'] = context['measure_as'] or context['deprecate_as']