Revert of Convert make_qualified_names and make_element_factory to use JSON5.

Revert of Convert make_qualified_names and make_element_factory to use JSON5. (patchset #5 id:80001 of https://codereview.chromium.org/2645283006/ )

Reason for revert:
Unexpected compile failure --
https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/32431

Original issue's description:
> Convert make_qualified_names and make_element_factory to use JSON5.
> 
> Convert make_qualified_names.py, make_element_factory.py,
> make_element_type_helpers.py, and make_element_lookup_trie.py
> to use JSON5 config format.
> 
> Converted these config files accordingly:
>   Source/core/html/HTMLAttributeNames.in
>   Source/core/html/HTMLTagNames.in
>   Source/core/html/parser/MathMLAttributeNames.in
>   Source/core/html/parser/MathMLTagNames.in
>   Source/core/svg/SVGAttributeNames.in
>   Source/core/svg/SVGTagNames.in
>   Source/core/svg/xlinkattrs.in
>   Source/core/xml/xmlattrs.in
>   Source/core/xml/xmlnsattrs.in
> 
> Note that these files are updated together because:
> 1. make_element_factory.py depends on make_qualified_names.py
> 2. HTMLTagNames.in is common input to the scripts make_element_factory,
> make_element_type_helpers and make_element_lookup_trie
> 
> Removed FIXME in make_element_type_helpers.py since that case
> no longer occurs.
> 
> Also fixed bug in json5_generator where parameters should not be
> validated if default_parameters is not provided.
> (caught when I had typo on default_parameters in the script)
> 
> BUG=677884
> 
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Original-Commit-Position: refs/heads/master@{#447450}
> Committed: https://chromium.googlesource.com/chromium/src/+/f5ca5654596eeda38c2a796bbbad299d92b7f008
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Commit-Position: refs/heads/master@{#447891}
> Committed: https://chromium.googlesource.com/chromium/src/+/701ccf95a0ec04657b59ad240e28be4085fe033f

TBR=nainar@chromium.org,sashab@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677884
Review-Url: https://codereview.chromium.org/2677603002
Cr-Commit-Position: refs/heads/master@{#447896}
Committed: https://chromium.googlesource.com/chromium/src/+/d50208a95a17148ec119745d20ceba483da71672

[ktyliu, 2017-02-03 00:29:58]

The CQ bit was checked by ktyliu@chromium.org

[ktyliu, 2017-02-03 00:29:58]

Created Revert of Convert make_qualified_names and make_element_factory to use
JSON5.

[commit-bot: I haz the power, 2017-02-03 00:30:39]

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-03 00:30:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-03 00:30:40]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[ktyliu, 2017-02-03 02:06:09]

On 2017/02/03 at 00:30:40, commit-bot wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

need LGTM from sheriff or Sasha to submit rollback

[nainar, 2017-02-03 02:10:12]

On 2017/02/03 at 02:06:09, ktyliu wrote:
> On 2017/02/03 at 00:30:40, commit-bot wrote:
> > No L-G-T-M from a valid reviewer yet. 
> > CQ run can only be started by full committers or once the patch has
> > received an L-G-T-M from a full committer.
> > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > _not_ a full super star committer.
> > See http://www.chromium.org/getting-involved/become-a-committer
> > Note that this has nothing to do with OWNERS files.
> 
> need LGTM from sheriff or Sasha to submit rollback

If an lgtm from a committer would do - here is mine.

[tapted, 2017-02-03 02:13:09]

tapted@chromium.org changed reviewers:
+ tapted@chromium.org

[tapted, 2017-02-03 02:13:09]

The CQ bit was checked by tapted@chromium.org

[tapted, 2017-02-03 02:13:10]

lgtm

lgtm + cq to revert

[commit-bot: I haz the power, 2017-02-03 02:13:55]

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-03 02:15:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486087989997910, "parent_rev":
"169e1ad7aeb6cf86dda8912fbf7e2ef3e2321344", "commit_rev":
"d50208a95a17148ec119745d20ceba483da71672"}

[commit-bot: I haz the power, 2017-02-03 02:16:27]

Description was changed from

==========
Revert of Convert make_qualified_names and make_element_factory to use JSON5.
(patchset #5 id:80001 of https://codereview.chromium.org/2645283006/ )

Reason for revert:
Unexpected compile failure --
https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/32431

Original issue's description:
> Convert make_qualified_names and make_element_factory to use JSON5.
> 
> Convert make_qualified_names.py, make_element_factory.py,
> make_element_type_helpers.py, and make_element_lookup_trie.py
> to use JSON5 config format.
> 
> Converted these config files accordingly:
>   Source/core/html/HTMLAttributeNames.in
>   Source/core/html/HTMLTagNames.in
>   Source/core/html/parser/MathMLAttributeNames.in
>   Source/core/html/parser/MathMLTagNames.in
>   Source/core/svg/SVGAttributeNames.in
>   Source/core/svg/SVGTagNames.in
>   Source/core/svg/xlinkattrs.in
>   Source/core/xml/xmlattrs.in
>   Source/core/xml/xmlnsattrs.in
> 
> Note that these files are updated together because:
> 1. make_element_factory.py depends on make_qualified_names.py
> 2. HTMLTagNames.in is common input to the scripts make_element_factory,
> make_element_type_helpers and make_element_lookup_trie
> 
> Removed FIXME in make_element_type_helpers.py since that case
> no longer occurs.
> 
> Also fixed bug in json5_generator where parameters should not be
> validated if default_parameters is not provided.
> (caught when I had typo on default_parameters in the script)
> 
> BUG=677884
> 
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Original-Commit-Position: refs/heads/master@{#447450}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f5ca5654596eeda38c2a796bbbad...
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Commit-Position: refs/heads/master@{#447891}
> Committed:
https://chromium.googlesource.com/chromium/src/+/701ccf95a0ec04657b59ad240e28...

TBR=nainar@chromium.org,sashab@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677884
==========

to

==========
Revert of Convert make_qualified_names and make_element_factory to use JSON5.
(patchset #5 id:80001 of https://codereview.chromium.org/2645283006/ )

Reason for revert:
Unexpected compile failure --
https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/32431

Original issue's description:
> Convert make_qualified_names and make_element_factory to use JSON5.
> 
> Convert make_qualified_names.py, make_element_factory.py,
> make_element_type_helpers.py, and make_element_lookup_trie.py
> to use JSON5 config format.
> 
> Converted these config files accordingly:
>   Source/core/html/HTMLAttributeNames.in
>   Source/core/html/HTMLTagNames.in
>   Source/core/html/parser/MathMLAttributeNames.in
>   Source/core/html/parser/MathMLTagNames.in
>   Source/core/svg/SVGAttributeNames.in
>   Source/core/svg/SVGTagNames.in
>   Source/core/svg/xlinkattrs.in
>   Source/core/xml/xmlattrs.in
>   Source/core/xml/xmlnsattrs.in
> 
> Note that these files are updated together because:
> 1. make_element_factory.py depends on make_qualified_names.py
> 2. HTMLTagNames.in is common input to the scripts make_element_factory,
> make_element_type_helpers and make_element_lookup_trie
> 
> Removed FIXME in make_element_type_helpers.py since that case
> no longer occurs.
> 
> Also fixed bug in json5_generator where parameters should not be
> validated if default_parameters is not provided.
> (caught when I had typo on default_parameters in the script)
> 
> BUG=677884
> 
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Original-Commit-Position: refs/heads/master@{#447450}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f5ca5654596eeda38c2a796bbbad...
> Review-Url: https://codereview.chromium.org/2645283006
> Cr-Commit-Position: refs/heads/master@{#447891}
> Committed:
https://chromium.googlesource.com/chromium/src/+/701ccf95a0ec04657b59ad240e28...

TBR=nainar@chromium.org,sashab@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=677884

Review-Url: https://codereview.chromium.org/2677603002
Cr-Commit-Position: refs/heads/master@{#447896}
Committed:
https://chromium.googlesource.com/chromium/src/+/d50208a95a17148ec119745d20ce...
==========

[commit-bot: I haz the power, 2017-02-03 02:16:28]

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