Remove build_overrides/v8.gni.

Remove build_overrides/v8.gni.

Chrome now uses the default_args system for specifying its own definitions of
these variables so the separate file in build_overrides is no longer necessary.

BUG=684096
Review-Url: https://codereview.chromium.org/2652263003
Cr-Commit-Position: refs/heads/master@{#42712}
Committed: https://chromium.googlesource.com/v8/v8/+/9d9711a8eb6146267f89f889c21d30a6956bb8a4

[brettw, 2017-01-25 21:26:55]

Description was changed from

==========
Remove build_overrides/v8.gni.

Chrome now uses the default_args system for specifying its own definitions of
these variables so the separate file in build_overrides is no longer necessary.

BUG=684096

Version 5.8.33

Performance and stability improvements on all platforms.

Cr-Commit-Position: refs/heads/5.8.33@{#1}
Cr-Branched-From:
1c1742f38d457700ce11674104b4e352187c1b4c-refs/heads/master@{#42658}
==========

to

==========
Remove build_overrides/v8.gni.

Chrome now uses the default_args system for specifying its own definitions of
these variables so the separate file in build_overrides is no longer necessary.

BUG=684096
==========

[brettw, 2017-01-25 21:27:28]

Patchset #2 (id:20001) has been deleted

[brettw, 2017-01-25 21:28:55]

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

[brettw, 2017-01-25 21:28:55]

I will land this after https://codereview.chromium.org/2648233006/ goes in,
which is required to keep Chrome unchanged.

[brettw, 2017-01-25 21:35:24]

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

[commit-bot: I haz the power, 2017-01-25 21:35:28]

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-01-25 21:54:19]

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

[commit-bot: I haz the power, 2017-01-25 21:54:19]

Dry run: Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[brettw, 2017-01-25 22:34:23]

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

[commit-bot: I haz the power, 2017-01-25 22:34:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[brettw, 2017-01-25 22:49:11]

The Chrome change has gone in so this should be ready.

[commit-bot: I haz the power, 2017-01-25 23:05:33]

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

[commit-bot: I haz the power, 2017-01-25 23:05:34]

Dry run: Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[brettw, 2017-01-26 00:10:40]

I *think* the GN error is because Chromium needs to be rolled into V8. Is that a
thing? Does it happen automatically?

[Michael Achenbach, 2017-01-26 07:35:06]

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

[commit-bot: I haz the power, 2017-01-26 07:35:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Michael Achenbach, 2017-01-26 07:35:47]

On 2017/01/26 00:10:40, brettw (ping after 24h) wrote:
> I *think* the GN error is because Chromium needs to be rolled into V8. Is that
a
> thing? Does it happen automatically?

That's automatic every emea morning. Will try again.

[commit-bot: I haz the power, 2017-01-26 07:42:24]

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

[commit-bot: I haz the power, 2017-01-26 07:42:25]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2017-01-26 07:47:01]

lgtm

[brettw, 2017-01-26 20:16:51]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2017-01-26 20:17:03]

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-01-26 20:18:59]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485461811888360,
"parent_rev": "ba1d747d6fb2419b7ac4904dd7f47422471d3b86", "commit_rev":
"9d9711a8eb6146267f89f889c21d30a6956bb8a4"}

[commit-bot: I haz the power, 2017-01-26 20:19:04]

Description was changed from

==========
Remove build_overrides/v8.gni.

Chrome now uses the default_args system for specifying its own definitions of
these variables so the separate file in build_overrides is no longer necessary.

BUG=684096
==========

to

==========
Remove build_overrides/v8.gni.

Chrome now uses the default_args system for specifying its own definitions of
these variables so the separate file in build_overrides is no longer necessary.

BUG=684096

Review-Url: https://codereview.chromium.org/2652263003
Cr-Commit-Position: refs/heads/master@{#42712}
Committed:
https://chromium.googlesource.com/v8/v8/+/9d9711a8eb6146267f89f889c21d30a6956...
==========

[commit-bot: I haz the power, 2017-01-26 20:19:05]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/9d9711a8eb6146267f89f889c21d30a6956...