Convert make_qualified_names and make_element_factory to use JSON5.

third_party/WebKit/Source/build/scripts/make_qualified_names.py

Index: third_party/WebKit/Source/build/scripts/make_qualified_names.py
diff --git a/third_party/WebKit/Source/build/scripts/make_qualified_names.py b/third_party/WebKit/Source/build/scripts/make_qualified_names.py
index a7d9d1a7b0881beffab00e359d5fe3c993f474aa..b9b5ff5b292e4717fcd5c3da1a460df91c5ff6b4 100755
--- a/third_party/WebKit/Source/build/scripts/make_qualified_names.py
+++ b/third_party/WebKit/Source/build/scripts/make_qualified_names.py
@@ -30,24 +30,20 @@
 import sys
 
 import hasher
-import in_generator
+import json5_generator
 import name_utilities
 import template_expander
 
-from in_file import InFile
+from json5_generator import Json5File
 
 
 def _symbol(entry):
-    # FIXME: Remove this special case for the ugly x-webkit-foo attributes.
-    if entry['name'].startswith('x-webkit-'):
-        return entry['name'].replace('-', '')[1:]
     return entry['name'].replace('-', '_')
 
 
-class MakeQualifiedNamesWriter(in_generator.Writer):
-    defaults = {
-    }
-    default_parameters = {
+class MakeQualifiedNamesWriter(json5_generator.Writer):
+    default_parameters = {}
+    default_metadata = {
         'attrsNullNamespace': None,
         'export': '',
         'namespace': '',
@@ -60,43 +56,44 @@ class MakeQualifiedNamesWriter(in_generator.Writer):
         'to_macro_style': name_utilities.to_macro_style,
     }
 
-    def __init__(self, in_file_paths):
+    def __init__(self, json5_file_paths):
         super(MakeQualifiedNamesWriter, self).__init__(None)
-        assert len(in_file_paths) <= 2, 'MakeQualifiedNamesWriter requires at most 2 in files, got %d.' % len(in_file_paths)
+        assert len(json5_file_paths) <= 2, 'MakeQualifiedNamesWriter requires at most 2 in files, got %d.' % len(json5_file_paths)
 
-        if len(in_file_paths) == 2:
-            self.tags_in_file = InFile.load_from_files([in_file_paths.pop(0)], self.defaults, self.valid_values, self.default_parameters)
+        if len(json5_file_paths) == 2:
+            self.tags_json5_file = Json5File.load_from_files(
+                [json5_file_paths.pop(0)], self.default_metadata, self.default_parameters)
         else:
-            self.tags_in_file = None
+            self.tags_json5_file = None
 
-        self.attrs_in_file = InFile.load_from_files([in_file_paths.pop()], self.defaults, self.valid_values, self.default_parameters)
+        self.attrs_json5_file = Json5File.load_from_files([json5_file_paths.pop()], self.default_metadata, self.default_parameters)
 
-        self.namespace = self._parameter('namespace')
+        self.namespace = self._metadata('namespace')
 
-        namespace_prefix = self._parameter('namespacePrefix') or self.namespace.lower()
-        namespace_uri = self._parameter('namespaceURI')
+        namespace_prefix = self._metadata('namespacePrefix') or self.namespace.lower()
+        namespace_uri = self._metadata('namespaceURI')
 
-        use_namespace_for_attrs = self.attrs_in_file.parameters['attrsNullNamespace'] is None
+        use_namespace_for_attrs = self.attrs_json5_file.metadata['attrsNullNamespace'] is None
 
         self._outputs = {
             (self.namespace + "Names.h"): self.generate_header,
             (self.namespace + "Names.cpp"): self.generate_implementation,
         }
         self._template_context = {
-            'attrs': self.attrs_in_file.name_dictionaries,
-            'export': self._parameter('export'),
+            'attrs': self.attrs_json5_file.name_dictionaries,
+            'export': self._metadata('export'),
             'namespace': self.namespace,
             'namespace_prefix': namespace_prefix,
             'namespace_uri': namespace_uri,
-            'tags': self.tags_in_file.name_dictionaries if self.tags_in_file else [],
+            'tags': self.tags_json5_file.name_dictionaries if self.tags_json5_file else [],
             'use_namespace_for_attrs': use_namespace_for_attrs,
         }
 
-    def _parameter(self, name):
-        parameter = self.attrs_in_file.parameters[name].strip('"')
-        if self.tags_in_file:
-            assert parameter == self.tags_in_file.parameters[name].strip('"'), 'Both in files must have the same %s.' % name
-        return parameter
+    def _metadata(self, name):
+        metadata = self.attrs_json5_file.metadata[name].strip('"')
+        if self.tags_json5_file:
+            assert metadata == self.tags_json5_file.metadata[name].strip('"'), 'Both files must have the same %s.' % name
+        return metadata
 
     @template_expander.use_jinja('MakeQualifiedNames.h.tmpl', filters=filters)
     def generate_header(self):
@@ -107,5 +104,5 @@ class MakeQualifiedNamesWriter(in_generator.Writer):
         return self._template_context
 
 
-if __name__ == "__main__":
-    in_generator.Maker(MakeQualifiedNamesWriter).main(sys.argv)
+if __name__ == "__majson5__":
+    json5_generator.Maker(MakeQualifiedNamesWriter).main()