Use the v8 shell instead of d8 to compile jstests

Use the v8 shell instead of d8 to compile jstests

BUG=609107
R=machenbach@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/11513aa0e64b7be852f123c7f3dedc9be50b78ff
Cr-Commit-Position: refs/heads/master@{#392891}

[jochen (gone - plz use gerrit), 2016-05-06 06:39:39]

Description was changed from

==========
Use the v8 shell instead of d8 to compile jstests

BUG=609107
R=machenbach@chromium.org
==========

to

==========
Use the v8 shell instead of d8 to compile jstests

BUG=609107
R=machenbach@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[jochen (gone - plz use gerrit), 2016-05-06 07:45:38]

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

[jochen (gone - plz use gerrit), 2016-05-06 07:45:38]

ptal

depends on https://crrev.com/609b5e5e30b719805dfe31f2f44f61d1293e83e8

Locally, it works :)

[Michael Achenbach, 2016-05-06 08:15:14]

lgtm. What has been done looks right. Can't judge the completeness. Not all the
gyp targets have correspondents in gn?

[jochen (gone - plz use gerrit), 2016-05-06 09:31:58]

On 2016/05/06 at 08:15:14, machenbach wrote:
> lgtm. What has been done looks right. Can't judge the completeness. Not all
the gyp targets have correspondents in gn?

the GN files use js2gtest instead of copying the action all over the place as
gyp does

[Michael Achenbach, 2016-05-06 10:22:30]

On 2016/05/06 09:31:58, jochen wrote:
> On 2016/05/06 at 08:15:14, machenbach wrote:
> > lgtm. What has been done looks right. Can't judge the completeness. Not all
> the gyp targets have correspondents in gn?
> 
> the GN files use js2gtest instead of copying the action all over the place as
> gyp does

Ack

[Dirk Pranke, 2016-05-06 22:37:23]

lgtm.

[jochen (gone - plz use gerrit), 2016-05-09 06:50:45]

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

[commit-bot: I haz the power, 2016-05-09 06:50:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1953233002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1953233002/1

[commit-bot: I haz the power, 2016-05-09 07:06:03]

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

[commit-bot: I haz the power, 2016-05-09 07:06:04]

Dry run: Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[jochen (gone - plz use gerrit), 2016-05-11 08:45:48]

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

[commit-bot: I haz the power, 2016-05-11 08:46:01]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1953233002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1953233002/20001

[commit-bot: I haz the power, 2016-05-11 09:50:24]

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

[commit-bot: I haz the power, 2016-05-11 09:50:25]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-05-11 09:56:17]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-05-11 09:56:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
dpranke@chromium.org, machenbach@chromium.org
Link to the patchset: https://codereview.chromium.org/1953233002/#ps20001
(title: "rename to v8_shell")

[commit-bot: I haz the power, 2016-05-11 09:56:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1953233002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1953233002/20001

[commit-bot: I haz the power, 2016-05-11 10:00:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-11 10:01:27]

Description was changed from

==========
Use the v8 shell instead of d8 to compile jstests

BUG=609107
R=machenbach@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Use the v8 shell instead of d8 to compile jstests

BUG=609107
R=machenbach@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/11513aa0e64b7be852f123c7f3dedc9be50b78ff
Cr-Commit-Position: refs/heads/master@{#392891}
==========

[commit-bot: I haz the power, 2016-05-11 10:01:28]

Patchset 2 (id:??) landed as
https://crrev.com/11513aa0e64b7be852f123c7f3dedc9be50b78ff
Cr-Commit-Position: refs/heads/master@{#392891}