Split GN target v8_headers for browser non-code dependency

Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG=chromium:716359
Review-Url: https://codereview.chromium.org/2853783002
Cr-Commit-Position: refs/heads/master@{#45009}
Committed: https://chromium.googlesource.com/v8/v8/+/c1e24b60875991f3e7be47e657a2515b29f19040

[wychen, 2017-05-01 05:57:26]

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

[commit-bot: I haz the power, 2017-05-01 05:57:32]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[wychen, 2017-05-01 05:59:27]

Description was changed from

==========
Split GN target v8_headers for browser non-code dependency

BUG=716359
==========

to

==========
Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG=716359
==========

[commit-bot: I haz the power, 2017-05-01 06:00:02]

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

[commit-bot: I haz the power, 2017-05-01 06:00:03]

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

[wychen, 2017-05-01 06:04:31]

Description was changed from

==========
Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG=716359
==========

to

==========
Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG= chromium:716359
==========

[wychen, 2017-05-01 06:05:20]

wychen@chromium.org changed reviewers:
+ yangguo@chromium.org

[wychen, 2017-05-01 06:05:21]

PTAL

[wychen, 2017-05-01 06:05:22]

PTAL

[Yang, 2017-05-01 06:09:34]

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

[Michael Achenbach, 2017-05-02 07:48:30]

lgtm

[wychen, 2017-05-02 08:17:56]

The CQ bit was checked by wychen@chromium.org

[commit-bot: I haz the power, 2017-05-02 08:17:59]

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-02 08:49:38]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493713076917090,
"parent_rev": "c507fdaffc9430f9d254b11b4ef785554c33b9f4", "commit_rev":
"c1e24b60875991f3e7be47e657a2515b29f19040"}

[commit-bot: I haz the power, 2017-05-02 08:49:44]

Description was changed from

==========
Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG= chromium:716359
==========

to

==========
Split GN target v8_headers for browser non-code dependency

This is for https://crrev.com/2851953002/

BUG= chromium:716359

Review-Url: https://codereview.chromium.org/2853783002
Cr-Commit-Position: refs/heads/master@{#45009}
Committed:
https://chromium.googlesource.com/v8/v8/+/c1e24b60875991f3e7be47e657a2515b29f...
==========

[commit-bot: I haz the power, 2017-05-02 08:49:45]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/c1e24b60875991f3e7be47e657a2515b29f...

[Michael Achenbach, 2017-05-02 09:02:15]

Please wait >2 days for landing the dependent Chromium change to guarantee
backwards compatibility to our last canary.

[wychen, 2017-05-04 20:27:01]

Is it OK to land https://codereview.chromium.org/2851953002/ now?

[Michael Achenbach, 2017-05-04 20:47:05]

go ahead. we rolled quite frequently.