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

Issue 2886033002: Really remove override_isolate_target from chromium.gpu.fyi.json (Closed)

Created:
3 years, 7 months ago by ynovikov
Modified:
3 years, 7 months ago
CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org, jbudorick, Dirk Pranke
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Really remove override_isolate_target from chromium.gpu.fyi.json BUG=716117 TBR=kbr@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2886033002 Cr-Commit-Position: refs/heads/master@{#472564} Committed: https://chromium.googlesource.com/chromium/src/+/5b094d1332d6ccf51ce097765dd2a38b2e5116f1

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -6 lines) Patch
M content/test/gpu/generate_buildbot_json.py View 1 chunk +0 lines, -6 lines 0 comments Download

Messages

Total messages: 14 (6 generated)
ynovikov
3 years, 7 months ago (2017-05-16 23:00:19 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2886033002/1
3 years, 7 months ago (2017-05-16 23:03:32 UTC) #4
Ken Russell (switch to Gerrit)
lgtm
3 years, 7 months ago (2017-05-16 23:10:00 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: android_optional_gpu_tests_rel on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_optional_gpu_tests_rel/builds/5427)
3 years, 7 months ago (2017-05-17 00:19:04 UTC) #7
Ken Russell (switch to Gerrit)
On 2017/05/17 00:19:04, commit-bot: I haz the power wrote: > Try jobs failed on following ...
3 years, 7 months ago (2017-05-17 07:19:31 UTC) #8
ynovikov
On 2017/05/17 07:19:31, Ken Russell wrote: > On 2017/05/17 00:19:04, commit-bot: I haz the power ...
3 years, 7 months ago (2017-05-17 19:25:34 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2886033002/1
3 years, 7 months ago (2017-05-17 19:26:28 UTC) #11
commit-bot: I haz the power
3 years, 7 months ago (2017-05-17 21:00:32 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5b094d1332d6ccf51ce097765dd2...

Powered by Google App Engine
This is Rietveld 408576698