IDL parser: implement Stringifier production

tools/idl_parser/idl_parser.py

Index: tools/idl_parser/idl_parser.py
diff --git a/tools/idl_parser/idl_parser.py b/tools/idl_parser/idl_parser.py
index 8a682ea688b6821d8993f8c52c8e9dcf6b78cc44..bba12b1912a4e0ff3169008c74fd7fa710d1f13c 100755
--- a/tools/idl_parser/idl_parser.py
+++ b/tools/idl_parser/idl_parser.py
@@ -437,7 +437,7 @@ class IDLParser(object):
                             | Attribute
                             | Operation"""
     if len(p) > 2:
-      p[0] = p[2]
+      p[0] = self.BuildProduction('Stringifier', p, 1, ListFromConcat(p[2]))

[Jens Widell, 2014/06/11 15:28:51]

Modifying p_AttributeOrOperation() like this is slightly unfortunate, because
Blink's parser overrides it, and thus needs to duplicate this line, or otherwise
handle it itself.

I did it primarily so that I could use the 'stringifier' token's source code
location (the "p, 1," bit.)

[Nils Barth (inactive), 2014/06/12 04:20:27]

This is fine; we only override this b/c we want to support
StaticAttribute,
but that's now part of the reference grammar:
http://heycam.github.io/webidl/#idl-grammar
...so we can move it into the base parser!
(Such cleanup will be *much* easier once repos are merged.)

Specifically:
  AttributeOrOperation
has been replaced by:
  AttributeOrOperationOrIterator
http://heycam.github.io/webidl/#proddef-AttributeOrOperationOrIterator

...which includes
  StaticMember
as a possible expansion.

     else:
       p[0] = p[1]
 
@@ -447,9 +447,9 @@ class IDLParser(object):
                                        | OperationRest
                                        | ';'"""
     if p[1] == ';':
-      p[0] = self.BuildAttribute('STRINGIFIER', Boolean(True))
+      p[0] = None
     else:
-      p[0] = ListFromConcat(self.BuildAttribute('STRINGIFIER', p[1]), p[1])
+      p[0] = p[1]
 
   # [32]
   def p_Attribute(self, p):