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

Issue 794303004: Removed references to win_gpu, which has been removed (Closed)

Created:
6 years ago by Sergiy Byelozyorov
Modified:
5 years, 10 months ago
CC:
chromium-reviews, piman+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@remove_linux_gpu_bots_from_presubmit
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Removed references to win_gpu, which has been removed BUG=426897 R=kbr@chromium.org,phajdan.jr@chromium.org,zmo@chromium.org,darin@chromium.org,brettw@chromium.org,cpu@chromium.org Committed: https://crrev.com/8778ff8184330b24e4196b2a2bcb09e0b1513e83 Cr-Commit-Position: refs/heads/master@{#316870}

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -21 lines) Patch
M PRESUBMIT.py View 1 1 chunk +0 lines, -1 line 0 comments Download
M PRESUBMIT_test.py View 1 1 chunk +0 lines, -4 lines 0 comments Download
D gpu/PRESUBMIT.py View 1 chunk +0 lines, -16 lines 0 comments Download

Messages

Total messages: 21 (7 generated)
Sergiy Byelozyorov
6 years ago (2014-12-12 16:56:29 UTC) #1
Paweł Hajdan Jr.
LGTM
6 years ago (2014-12-12 17:45:47 UTC) #2
Zhenyao Mo
LGTM
6 years ago (2014-12-12 17:50:12 UTC) #3
Ken Russell (switch to Gerrit)
lgtm
6 years ago (2014-12-12 18:43:33 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/794303004/1
5 years, 10 months ago (2015-02-12 22:01:49 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/794303004/20001
5 years, 10 months ago (2015-02-13 01:40:53 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presubmit/builds/42490)
5 years, 10 months ago (2015-02-13 01:47:39 UTC) #11
Sergiy Byelozyorov
jochen@: can you please rubber-stamp this CL?
5 years, 10 months ago (2015-02-13 02:03:15 UTC) #13
Sergiy Byelozyorov
ping
5 years, 10 months ago (2015-02-18 01:23:41 UTC) #14
Sergiy Byelozyorov
removed jochen (he's OOO and can't review gpu/PRESUBMIT.py), added darin@, brettw@, cpu@ - please review ...
5 years, 10 months ago (2015-02-18 01:30:10 UTC) #16
cpu_(ooo_6.6-7.5)
lgtm
5 years, 10 months ago (2015-02-18 16:02:50 UTC) #17
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/794303004/20001
5 years, 10 months ago (2015-02-18 18:53:11 UTC) #19
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 10 months ago (2015-02-18 18:56:45 UTC) #20
commit-bot: I haz the power
5 years, 10 months ago (2015-02-18 18:58:27 UTC) #21
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/8778ff8184330b24e4196b2a2bcb09e0b1513e83
Cr-Commit-Position: refs/heads/master@{#316870}

Powered by Google App Engine
This is Rietveld 408576698