Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(385)

Issue 1386663002: Revert of [swarming] Isolate v8 testing. (Closed)

Created:
5 years, 2 months ago by danno
Modified:
5 years, 2 months ago
CC:
v8-reviews_googlegroups.com, M-A Ruel
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

Revert of [swarming] Isolate v8 testing. (patchset #8 id:140001 of https://codereview.chromium.org/1380593002/ ) Reason for revert: Prime suspect for breaking Linux V8 - no snap Original issue's description: > [swarming] Isolate v8 testing. > > Add gyp support and isolates for default test suites. > Add two default isolates, one (default) for using the > test suite collection we call "default" on the bots. One > (developer_default) for also supporting the way developers > call the driver (i.e. without argument, which includes > the unittests). > > BUG=chromium:535160 > LOG=n > > Committed: https://crrev.com/9bd83f58f29ab0c7c5b71b00bcb1df3a9e641f05 > Cr-Commit-Position: refs/heads/master@{#31081} TBR=tandrii@chromium.org,jochen@chromium.org,maruel@chromium.org,machenbach@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:535160

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -334 lines) Patch
M build/all.gyp View 1 chunk +0 lines, -10 lines 0 comments Download
M test/cctest/cctest.gyp View 1 chunk +0 lines, -19 lines 0 comments Download
D test/cctest/cctest.isolate View 1 chunk +0 lines, -16 lines 0 comments Download
D test/default.gyp View 1 chunk +0 lines, -29 lines 0 comments Download
D test/default.isolate View 1 chunk +0 lines, -12 lines 0 comments Download
D test/developer_default.gyp View 1 chunk +0 lines, -26 lines 0 comments Download
D test/developer_default.isolate View 1 chunk +0 lines, -9 lines 0 comments Download
D test/intl/intl.gyp View 1 chunk +0 lines, -25 lines 0 comments Download
D test/intl/intl.isolate View 1 chunk +0 lines, -14 lines 0 comments Download
D test/message/message.gyp View 1 chunk +0 lines, -25 lines 0 comments Download
D test/message/message.isolate View 1 chunk +0 lines, -14 lines 0 comments Download
D test/mjsunit/mjsunit.gyp View 1 chunk +0 lines, -25 lines 0 comments Download
D test/mjsunit/mjsunit.isolate View 1 chunk +0 lines, -23 lines 0 comments Download
D test/preparser/preparser.gyp View 1 chunk +0 lines, -25 lines 0 comments Download
D test/preparser/preparser.isolate View 1 chunk +0 lines, -14 lines 0 comments Download
M test/unittests/unittests.gyp View 1 chunk +0 lines, -19 lines 0 comments Download
D test/unittests/unittests.isolate View 1 chunk +0 lines, -15 lines 0 comments Download
D tools/testrunner/testrunner.isolate View 1 chunk +0 lines, -14 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
danno
Created Revert of [swarming] Isolate v8 testing.
5 years, 2 months ago (2015-10-02 15:45:59 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1386663002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1386663002/1
5 years, 2 months ago (2015-10-02 15:46:12 UTC) #2
commit-bot: I haz the power
Failed to apply patch for test/cctest/cctest.gyp: While running git apply --index -3 -p1; error: patch ...
5 years, 2 months ago (2015-10-02 15:46:24 UTC) #4
tandrii(chromium)
5 years, 2 months ago (2015-10-02 15:46:26 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698