Modify DEPS file to recurse into V8's DEPS.chromium.

Modify DEPS file to recurse into V8's DEPS.chromium.

This picks up the simulator test files for ARM64.

BUG=718439

[cavalcantii1, 2017-05-16 16:58:15]

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

[commit-bot: I haz the power, 2017-05-16 16:58:56]

Dry run: 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, 2017-05-16 22:14:14]

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

[commit-bot: I haz the power, 2017-05-16 22:14:14]

Dry run: This issue passed the CQ dry run.

[cavalcantii1, 2017-05-17 00:21:45]

Description was changed from

==========
Modify DEPS file to recurse into V8's DEPS.chromium.

This picks up the simulator test files for ARM64.

BUG=718439
==========

to

==========
Modify DEPS file to recurse into V8's DEPS.chromium.

This picks up the simulator test files for ARM64.

BUG=718439
==========

[stephen.kyle, 2017-05-17 12:15:45]

stephen.kyle@arm.com changed reviewers:
+ martyn.capewell@arm.com

[stephen.kyle, 2017-05-17 12:16:10]

stephen.kyle@arm.com changed reviewers:
+ thakis@chromium.org

[Nico, 2017-05-17 15:55:46]

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

[Nico, 2017-05-17 15:55:46]

On the bug: "ARM64 NEON support in V8 depends on an external git repository for
its simulator tests, as the files are large and infrequently changed."

It sounds like this change will make it so that these large files would be
downloaded in chromium checkouts by default, is that correct? If so, I'm pretty
sure that we don't want to do that.

[martyn.capewell, 2017-05-17 16:10:17]

Large is 72MB total, which is too large to go through codereview.chromium.org,
but I don't think they're large compared to the rest of the code base.

The repository contains a set of header files with reference outputs, used to
test the ARM64 simulator matches the hardware.  The largest file is 1.2MB, and
there are about 1400 files in the repository.

[Nico, 2017-05-17 18:37:53]

On 2017/05/17 16:10:17, martyn.capewell wrote:
> Large is 72MB total, which is too large to go through
http://codereview.chromium.org,
> but I don't think they're large compared to the rest of the code base.
> 
> The repository contains a set of header files with reference outputs, used to
> test the ARM64 simulator matches the hardware.  The largest file is 1.2MB, and
> there are about 1400 files in the repository.

Yeah, we don't want everyone on linux download 72MB of stuff most people won't
use. (Similar recent discussion on
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/WB9FME7IVZM
for example)