Use GN response files for devtools .grd

Use GN response files for devtools .grd

Eliminates the write_file call for making the debug devtools grd files by using a GN response file.

This was made more complicated by the GYP and GN response files having different formats, so it now supports two different arguments. Further, in the normal (non debugging devtools) case, the response file is empty which GN interprets as "don't use a response file" and things get confused. As a result, the file list argument to the script is now optional.

The script takes multiple categories of variable length files. This means that the normal OptionParser doesn't work well and I had to enhance the already complicated parsing.

Fixes a typo in a GN error message I noticed.

Committed: https://crrev.com/7cf73b7282dad450109b3eab2db79825cf03a4e0
Cr-Commit-Position: refs/heads/master@{#359634}

[brettw, 2015-11-13 18:24:18]

Description was changed from

==========
Use GN response files for devtools .grd

Eliminates the write_file call for making the debug devtools grd files by using
a GN response file.

This was made more complicated by the GYP and GN response files having different
formats, so it now supports two different arguments. Further, in the normal (non
debugging devtools) case, the response file is empty which GN interprets as
"don't use a response file" and things get confused. As a result, the file list
argument to the script is now optional.

The script takes mutliple categories of variable length files. This means that
the normal OptionParser doesn't work well and I had to enhance the already
complicated parsing.
==========

to

==========
Use GN response files for devtools .grd

Eliminates the write_file call for making the debug devtools grd files by using
a GN response file.

This was made more complicated by the GYP and GN response files having different
formats, so it now supports two different arguments. Further, in the normal (non
debugging devtools) case, the response file is empty which GN interprets as
"don't use a response file" and things get confused. As a result, the file list
argument to the script is now optional.

The script takes mutliple categories of variable length files. This means that
the normal OptionParser doesn't work well and I had to enhance the already
complicated parsing.

Fixes a typo in a GN error message I noticed.
==========

[brettw, 2015-11-13 18:24:26]

brettw@chromium.org changed reviewers:
+ dpranke@chromium.org

[brettw, 2015-11-13 18:24:31]

The CQ bit was checked by brettw@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-11-13 18:24:56]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1442013004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1442013004/1

[commit-bot: I haz the power, 2015-11-13 20:12:01]

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

[commit-bot: I haz the power, 2015-11-13 20:12:02]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2015-11-13 20:32:08]

Description was changed from

==========
Use GN response files for devtools .grd

Eliminates the write_file call for making the debug devtools grd files by using
a GN response file.

This was made more complicated by the GYP and GN response files having different
formats, so it now supports two different arguments. Further, in the normal (non
debugging devtools) case, the response file is empty which GN interprets as
"don't use a response file" and things get confused. As a result, the file list
argument to the script is now optional.

The script takes mutliple categories of variable length files. This means that
the normal OptionParser doesn't work well and I had to enhance the already
complicated parsing.

Fixes a typo in a GN error message I noticed.
==========

to

==========
Use GN response files for devtools .grd

Eliminates the write_file call for making the debug devtools grd files by using
a GN response file.

This was made more complicated by the GYP and GN response files having different
formats, so it now supports two different arguments. Further, in the normal (non
debugging devtools) case, the response file is empty which GN interprets as
"don't use a response file" and things get confused. As a result, the file list
argument to the script is now optional.

The script takes multiple categories of variable length files. This means that
the normal OptionParser doesn't work well and I had to enhance the already
complicated parsing.

Fixes a typo in a GN error message I noticed.
==========

[Dirk Pranke, 2015-11-13 20:34:11]

lgtm

[brettw, 2015-11-13 20:46:24]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-11-13 20:47:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1442013004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1442013004/1

[commit-bot: I haz the power, 2015-11-13 21:07:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-13 21:08:05]

Patchset 1 (id:??) landed as
https://crrev.com/7cf73b7282dad450109b3eab2db79825cf03a4e0
Cr-Commit-Position: refs/heads/master@{#359634}