idl_parser: Add support for the record<K, V> WebIDL type.

idl_parser: Add support for the record<K, V> WebIDL type.

This CL is the first step towards fully supporting it in the bindings code.
It merely makes the IDL parser aware of the new type (namely, the
"RecordType" symbol) by updating the grammar to match the current WebIDL
version.

The devil is in the details though: the IDL parser was also unaware of the
"StringType" symbol that comprises ByteStrings, DOMStrings and
USVStrings (the only allowed key types for records), so most of the patch is
just updating existing tests to consider those 3 string types StringTypes,
not PrimitiveTypes.

BUG=685754
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org

Review-Url: https://codereview.chromium.org/2708173002
Cr-Commit-Position: refs/heads/master@{#451970}
Committed: https://chromium.googlesource.com/chromium/src/+/4bec0d7dc53aa20a83fbd1b4e6bf097b5ff4dded

[Raphael Kubo da Costa (rakuco), 2017-02-21 19:37:47]

PTAL. This is part 1 of ? for the aforementioned bug; this bit only makes the
lexer/parser aware of the new symbols.

[haraken, 2017-02-21 23:36:09]

LGTM

https://codereview.chromium.org/2708173002/diff/1/tools/idl_parser/idl_lexer.py
File tools/idl_parser/idl_lexer.py (right):

https://codereview.chromium.org/2708173002/diff/1/tools/idl_parser/idl_lexer....
tools/idl_parser/idl_lexer.py:107: 'USVString' : 'USVSTRING',

alphabetical order?

[Yuki, 2017-02-22 07:44:57]

LGTM.

https://codereview.chromium.org/2708173002/diff/1/tools/idl_parser/idl_lexer.py
File tools/idl_parser/idl_lexer.py (right):

https://codereview.chromium.org/2708173002/diff/1/tools/idl_parser/idl_lexer....
tools/idl_parser/idl_lexer.py:107: 'USVString' : 'USVSTRING',
On 2017/02/21 23:36:09, haraken wrote:
> 
> alphabetical order?

It's already case-insensitive alphabetical order, I think.

[Raphael Kubo da Costa (rakuco), 2017-02-22 08:11:04]

The CQ bit was checked by raphael.kubo.da.costa@intel.com

[commit-bot: I haz the power, 2017-02-22 08:11:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Raphael Kubo da Costa (rakuco), 2017-02-22 09:50:21]

The CQ bit was unchecked by raphael.kubo.da.costa@intel.com

[Raphael Kubo da Costa (rakuco), 2017-02-22 09:50:30]

The CQ bit was checked by raphael.kubo.da.costa@intel.com

[commit-bot: I haz the power, 2017-02-22 09:50:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 10:12:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487757030419950, "parent_rev":
"5f64972b263657c4f1b524f1ee544e571758b9dc", "commit_rev":
"4bec0d7dc53aa20a83fbd1b4e6bf097b5ff4dded"}

[commit-bot: I haz the power, 2017-02-22 10:13:27]

Description was changed from

==========
idl_parser: Add support for the record<K, V> WebIDL type.

This CL is the first step towards fully supporting it in the bindings code.
It merely makes the IDL parser aware of the new type (namely, the
"RecordType" symbol) by updating the grammar to match the current WebIDL
version.

The devil is in the details though: the IDL parser was also unaware of the
"StringType" symbol that comprises ByteStrings, DOMStrings and
USVStrings (the only allowed key types for records), so most of the patch is
just updating existing tests to consider those 3 string types StringTypes,
not PrimitiveTypes.

BUG=685754
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org
==========

to

==========
idl_parser: Add support for the record<K, V> WebIDL type.

This CL is the first step towards fully supporting it in the bindings code.
It merely makes the IDL parser aware of the new type (namely, the
"RecordType" symbol) by updating the grammar to match the current WebIDL
version.

The devil is in the details though: the IDL parser was also unaware of the
"StringType" symbol that comprises ByteStrings, DOMStrings and
USVStrings (the only allowed key types for records), so most of the patch is
just updating existing tests to consider those 3 string types StringTypes,
not PrimitiveTypes.

BUG=685754
R=bashi@chromium.org,haraken@chromium.org,yukishiino@chromium.org

Review-Url: https://codereview.chromium.org/2708173002
Cr-Commit-Position: refs/heads/master@{#451970}
Committed:
https://chromium.googlesource.com/chromium/src/+/4bec0d7dc53aa20a83fbd1b4e6bf...
==========

[commit-bot: I haz the power, 2017-02-22 10:13:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4bec0d7dc53aa20a83fbd1b4e6bf...