IDL: [NoImplHeader] should not add '#include "None"'

Source/bindings/scripts/code_generator_v8.py

Index: Source/bindings/scripts/code_generator_v8.py
diff --git a/Source/bindings/scripts/code_generator_v8.py b/Source/bindings/scripts/code_generator_v8.py
index 3b153b5805ccb666149cb75a1060c1e5d248fef0..33f7c41a3f05bc04b3abb44b395c680b95bd7cc7 100644
--- a/Source/bindings/scripts/code_generator_v8.py
+++ b/Source/bindings/scripts/code_generator_v8.py
@@ -192,7 +192,8 @@ class CodeGeneratorV8(CodeGeneratorBase):
         if IdlType(interface_name).is_typed_array:
             template_context['header_includes'].add('core/dom/DOMTypedArray.h')
         else:

[Jens Widell, 2015/01/09 09:09:17]

Use "elif"?

[bashi, 2015/01/09 09:13:21]

Done.

-            template_context['header_includes'].add(interface_info['include_path'])
+            if interface_info['include_path']:
+                template_context['header_includes'].add(interface_info['include_path'])
         header_text, cpp_text = render_template(
             include_paths, header_template, cpp_template, template_context,
             component)
@@ -212,7 +213,8 @@ class CodeGeneratorV8(CodeGeneratorBase):
         interface_info = interfaces_info[dictionary_name]
         include_paths = interface_info.get('dependencies_include_paths')
         # Add the include for interface itself
-        template_context['header_includes'].add(interface_info['include_path'])
+        if interface_info['include_path']:
+            template_context['header_includes'].add(interface_info['include_path'])
         header_text, cpp_text = render_template(
             include_paths, header_template, cpp_template, template_context)
         header_path, cpp_path = self.output_paths(dictionary_name)