[gn] Fix config for icu data file and swarming

[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true

Committed: https://crrev.com/a774fa5142ae9d8d3539e1f25f6e9f04893a2121
Cr-Commit-Position: refs/heads/master@{#36998}

[Michael Achenbach, 2016-06-15 11:22:16]

Description was changed from

==========
[gn] Fix config for icu data file and swarming

BUG=
==========

to

==========
[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

[Michael Achenbach, 2016-06-15 11:22:16]

machenbach@chromium.org changed reviewers:
+ jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-06-15 11:23:14]

TBR

[vogelheim, 2016-06-15 11:30:24]

lgtm

[Michael Achenbach, 2016-06-15 11:53:12]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-06-15 11:53:18]

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

[commit-bot: I haz the power, 2016-06-15 11:57:17]

Description was changed from

==========
[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

to

==========
[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-06-15 11:57:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-15 11:57:21]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-15 12:00:08]

Description was changed from

==========
[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true
==========

to

==========
[gn] Fix config for icu data file and swarming

Fix after:
https://codereview.chromium.org/2042253002/
https://codereview.chromium.org/2049743002/

Using the data file is the default now and it shouldn't
be overridden in v8's build_overrides. Also the isolate
logic now needs to include the gni file specifying the
flag.

BUG=chromium:616033
TBR=jochen@chromium.org, vogelheim@chromium.org
NOTRY=true

Committed: https://crrev.com/a774fa5142ae9d8d3539e1f25f6e9f04893a2121
Cr-Commit-Position: refs/heads/master@{#36998}
==========

[commit-bot: I haz the power, 2016-06-15 12:00:09]

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