Update IDL compiler lines in sanitize-win-build-log.sed

We're adding a new idl_compiler.py script and 
renaming the existing generate-bindings.pl,
as part of adding a second IDL compile flow so we can compile
with either Python or Perl.
We thus need to update this sed script to process the new command lines.
(Separate CL b/c build in Blink and this is in Chromium.)

Ok to commit, now that the build CL has landed:
Issue 18190004: Add Python flow to bindings generation, move dummy-generating IDL files over
https://codereview.chromium.org/18190004/

BUG=239771
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=211057

[Nils Barth (inactive), 2013-06-17 04:54:50]

FYI, minor follow-up to build splitting (via JSON),
to commit after that lands.

Don't need to review now, but gives context to comments in GYP file
(there's a non-obvious dependency).

[dominicc (has gone to gerrit), 2013-06-19 02:42:12]

Why wouldn't we do this as part of the change it depends on?

https://codereview.chromium.org/17215002/diff/1/build/sanitize-win-build-log.sed
File build/sanitize-win-build-log.sed (right):

https://codereview.chromium.org/17215002/diff/1/build/sanitize-win-build-log....
build/sanitize-win-build-log.sed:17:
s/^.*"perl".*parse-idl.pl".*\("[^"]\+\.idl"\).*$/  parse-idl \1/
I think that the original pattern may mistakenly do .pl when it means \.pl?

You might consider adding a FIXME with a note about when this can die.

[Nils Barth (inactive), 2013-06-19 02:49:03]

On 2013/06/19 02:42:12, dominicc wrote:
> Why wouldn't we do this as part of the change it depends on?

This is Chromium side; the rest of the change is Blink side.
Is it possible to do both at once?
(My understanding is that one needs to do a multi-sided change for
Chromium+Blink changes.)

[Nils Barth (inactive), 2013-06-19 02:49:35]

Revised per Dominic's comments.
Also question about "when this can die".

https://codereview.chromium.org/17215002/diff/1/build/sanitize-win-build-log.sed
File build/sanitize-win-build-log.sed (right):

https://codereview.chromium.org/17215002/diff/1/build/sanitize-win-build-log....
build/sanitize-win-build-log.sed:17:
s/^.*"perl".*parse-idl.pl".*\("[^"]\+\.idl"\).*$/  parse-idl \1/
On 2013/06/19 02:42:13, dominicc wrote:
> I think that the original pattern may mistakenly do .pl when it means \.pl?

Good point, fixed!

> You might consider adding a FIXME with a note about when this can die.

? What do you mean?
I assume this will continue to exist, since the bindings generation lines will
continue to be long even after we switch to Python.

[haraken, 2013-06-19 03:17:40]

I guess it would be better to get back to this CL after landing the Python
parser.

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

On 2013/06/19 03:17:40, haraken wrote:
> I guess it would be better to get back to this CL after landing the Python
> parser.

Agreed; tabling for now.

[Nils Barth (inactive), 2013-07-10 07:55:29]

Hiyas,
Now that Python flow CL is landing, we should land this one
shortly after, so the logs are still legible.
It's quick (but sed, admittedly); how does it look?

[haraken, 2013-07-11 02:42:55]

LGTM

https://codereview.chromium.org/17215002/diff/11001/build/sanitize-win-build-...
File build/sanitize-win-build-log.sed (right):

https://codereview.chromium.org/17215002/diff/11001/build/sanitize-win-build-...
build/sanitize-win-build-log.sed:18:
s/^.*"perl".*idl_compiler\.py".*\("[^"]\+\.idl"\).*$/  idl_compiler \1/

perl => python

[commit-bot: I haz the power, 2013-07-11 03:32:34]

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

[commit-bot: I haz the power, 2013-07-11 06:41:37]

Retried try job too often on win_rel for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-07-11 06:55:10]

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

[commit-bot: I haz the power, 2013-07-11 10:45:50]

Change committed as 211057