[Bindings] Pass files to compute_interfaces_info_individual.py in one way

Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.

BUG=617899
Committed: https://crrev.com/4ca6e02d38f91102e6da13149743b523b4d20a63
Cr-Commit-Position: refs/heads/master@{#420945}

[peria, 2016-09-26 05:35:49]

Description was changed from

==========
Refactoring around IDL compiler

BUG=617899
==========

to

==========
Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.


BUG=617899
==========

[peria, 2016-09-26 05:48:02]

Patchset #1 (id:1) has been deleted

[peria, 2016-09-26 06:00:57]

Patchset #1 (id:20001) has been deleted

[peria, 2016-09-26 06:18:31]

peria@chromium.org changed reviewers:
+ bashi@chromium.org, yukishiino@chromium.org

[peria, 2016-09-26 06:18:31]

PTL

[bashi, 2016-09-26 06:26:26]

lgtm with one question

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/scripts.gni (right):

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/scripts.gni:77:
response_file_contents = rebase_path(invoker.sources, root_build_dir)
Help me understand: what does 'response' mean here?

[haraken, 2016-09-26 06:30:40]

LGTM

[Yuki, 2016-09-26 06:44:33]

lgtm

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/scripts.gni (right):

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/scripts.gni:77:
response_file_contents = rebase_path(invoker.sources, root_build_dir)
On 2016/09/26 06:26:26, bashi1 wrote:
> Help me understand: what does 'response' mean here?

It seems we don't need to struggle with naming if we inline |rebase_path(...)|.

[peria, 2016-09-26 06:47:30]

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/scripts/scripts.gni (right):

https://codereview.chromium.org/2372463002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/scripts/scripts.gni:77:
response_file_contents = rebase_path(invoker.sources, root_build_dir)
On 2016/09/26 06:26:26, bashi1 wrote:
> Help me understand: what does 'response' mean here?

This is a meta variable feature of GN to store a GN list into a temporary file
to avoid blowing out command line length limit.
See "gn help response_file_contents" in your command line, or
https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/referen...

[peria, 2016-09-26 06:51:40]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-09-26 06:51:46]

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, 2016-09-26 09:31:17]

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

[commit-bot: I haz the power, 2016-09-26 09:31:18]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[peria, 2016-09-26 09:34:19]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-09-26 09:34:38]

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, 2016-09-26 12:05:17]

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

[commit-bot: I haz the power, 2016-09-26 12:05:18]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[peria, 2016-09-26 17:06:30]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-09-26 17:07:19]

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, 2016-09-26 18:42:02]

Description was changed from

==========
Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.


BUG=617899
==========

to

==========
Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.


BUG=617899
==========

[commit-bot: I haz the power, 2016-09-26 18:42:06]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-09-26 18:44:18]

Description was changed from

==========
Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.


BUG=617899
==========

to

==========
Pass file names list to compute_interfaces_info_individual.py in one way

This is a refactoring change around IDL compiler.

BUG=617899

Committed: https://crrev.com/4ca6e02d38f91102e6da13149743b523b4d20a63
Cr-Commit-Position: refs/heads/master@{#420945}
==========

[commit-bot: I haz the power, 2016-09-26 18:44:19]

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