Use the new mojom parser from check_mojom_dart.py.

Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree. Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py directly invokes v1 of the parser in order to extract the module name and the 'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we also:
- Create a new script in pylib called parser_runner.py that encapsulates invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke parser_runner.py instead of having the logic directly embbedded in them.

R=azani@chromium.org, johnmccutchan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/63a720b40073fcb15966102a7a45166c47193cf1

[rudominer, 2016-02-17 04:54:44]

Patchset #1 (id:1) has been deleted

[rudominer, 2016-02-17 04:54:52]

Patchset #1 (id:20001) has been deleted

[rudominer, 2016-02-17 04:54:58]

Patchset #1 (id:40001) has been deleted

[rudominer, 2016-02-17 04:55:06]

Patchset #1 (id:60001) has been deleted

[rudominer, 2016-02-17 05:01:21]

Description was changed from

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=
==========

to

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree.
Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py
directly invokes v1 of the parser in order to exract the module name and the
'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we
also:
- Create a new script in pylib called parser_runner.py that encapsulates
invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke
parser_runner.py instead of having the logic directly embbedded in them.
==========

[rudominer, 2016-02-17 05:01:59]

Description was changed from

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree.
Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py
directly invokes v1 of the parser in order to exract the module name and the
'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we
also:
- Create a new script in pylib called parser_runner.py that encapsulates
invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke
parser_runner.py instead of having the logic directly embbedded in them.
==========

to

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree.
Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py
directly invokes v1 of the parser in order to extract the module name and the
'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we
also:
- Create a new script in pylib called parser_runner.py that encapsulates
invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke
parser_runner.py instead of having the logic directly embbedded in them.
==========

[rudominer, 2016-02-17 05:09:35]

rudominer@chromium.org changed reviewers:
+ azani@chromium.org, johnmccutchan@google.com

[rudominer, 2016-02-17 05:09:35]

Hi John and Alex, ptal.

thanks,
Mitch

[rudominer, 2016-02-17 05:10:58]

Patchset #1 (id:80001) has been deleted

[Cutch, 2016-02-17 14:58:39]

lgtm

[azani, 2016-02-17 17:14:48]

lgtm

[rudominer, 2016-02-17 17:15:54]

Description was changed from

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree.
Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py
directly invokes v1 of the parser in order to extract the module name and the
'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we
also:
- Create a new script in pylib called parser_runner.py that encapsulates
invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke
parser_runner.py instead of having the logic directly embbedded in them.
==========

to

==========
Use the new mojom parser from check_mojom_dart.py.

BUG=#671

This is part of my work to delete the old Mojom parser from the source tree.
Currently the pre-submit script mojo/dart/tools/presubmit/check_mojom_dart.py
directly invokes v1 of the parser in order to extract the module name and the
'DartPackage' attribute of a .mojom file.

This CL changes check_mojom_dart.py to use the new parser. In support of that we
also:
- Create a new script in pylib called parser_runner.py that encapsulates
invoking the parser and the translator.
- Invoke this script from check_mojom_dart.py
- Refactor run_code_generators.py and mojom_bindings_generator_v2.py to invoke
parser_runner.py instead of having the logic directly embbedded in them.

R=azani@chromium.org, johnmccutchan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/63a720b40073fcb15966102a7a4...
==========

[rudominer, 2016-02-17 17:15:55]

Committed patchset #1 (id:100001) manually as
63a720b40073fcb15966102a7a45166c47193cf1 (presubmit successful).