Pre-cache PLY yacc parse table and use optimized mode (45% build time improvement)

Pre-cache PLY yacc parse table and use optimized mode (45% build time improvement)

PLY yacc parse table creation is (obviously) expensive.
Pre-caching this in a separate build step substantially improves build time.

Improves build time (user time) by 45% on my Linux box:
1m 50s => 59s

BUG=341748
R=haraken

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168611

[Nils Barth (inactive), 2014-03-05 06:11:50]

Big speedups!

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/scripts/...
File Source/bindings/scripts/idl_reader.py (right):

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/scripts/...
Source/bindings/scripts/idl_reader.py:34: from blink_idl_parser import
BlinkIDLParser
These are just some formatting fixes for consistency and legibility:
we import classes |from| modules (these are obvious as CamelCase).

[haraken, 2014-03-05 06:18:03]

Great improvement. LGTM. (I don't fully understand yacc options, but I believe
you understand what you're doing:-)

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/generate...
File Source/bindings/generated_bindings.gyp (right):

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/generate...
Source/bindings/generated_bindings.gyp:300:
'<(bindings_output_dir)/parsetab.pickle',

parsed_yacc_table.pickle ?

[Nils Barth (inactive), 2014-03-05 06:28:05]

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/generate...
File Source/bindings/generated_bindings.gyp (right):

https://codereview.chromium.org/184233005/diff/40001/Source/bindings/generate...
Source/bindings/generated_bindings.gyp:300:
'<(bindings_output_dir)/parsetab.pickle',
On 2014/03/05 06:18:04, haraken wrote:
> parsed_yacc_table.pickle ?

"parsetab.pickle" is the PLY standard name.
(Similarly, PLY uses "parsetab.py" for the module.)

[Nils Barth (inactive), 2014-03-05 06:29:12]

On 2014/03/05 06:18:03, haraken wrote:
> Great improvement. LGTM.

Thanks!

> (I don't fully understand yacc options, but I believe
> you understand what you're doing:-)

Understood that they're black magic;
I've added detailed comments, and will document at the build design doc
once done:
http://www.chromium.org/developers/design-documents/idl-build

[Nils Barth (inactive), 2014-03-06 08:19:51]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-03-06 08:19:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/184233005/40001

[commit-bot: I haz the power, 2014-03-06 09:37:40]

Change committed as 168611