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

Issue 2175053002: [Findit] Fixing debug view UI not to bundle unclassified with reliable failures (Closed)

Created:
4 years, 5 months ago by lijeffrey
Modified:
4 years, 4 months ago
Reviewers:
chanli, Sharu Jiang, stgao
CC:
chromium-reviews, infra-reviews+infra_chromium.org
Base URL:
https://chromium.googlesource.com/infra/infra.git@master
Target Ref:
refs/heads/master
Project:
infra
Visibility:
Public.

Description

[Findit] Fixing debug view UI not to bundle unclassified with reliable failures BUG=627297 Committed: https://chromium.googlesource.com/infra/infra/+/894991b7ef4e30fd4dcf79b17e5a0095fc616fe7

Patch Set 1 #

Patch Set 2 : Viewing diffs #

Patch Set 3 : Fixing unit tests #

Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -40 lines) Patch
M appengine/findit/handlers/build_failure.py View 1 2 chunks +1 line, -2 lines 0 comments Download
M appengine/findit/handlers/handlers_util.py View 1 2 5 chunks +42 lines, -35 lines 0 comments Download
M appengine/findit/handlers/test/handlers_util_test.py View 1 2 4 chunks +7 lines, -3 lines 0 comments Download

Messages

Total messages: 10 (5 generated)
Sharu Jiang
lgtm
4 years, 5 months ago (2016-07-25 23:35:31 UTC) #3
lijeffrey
This change should take care of other failures that the previous iteration didn't, such as ...
4 years, 4 months ago (2016-08-02 03:11:54 UTC) #4
chanli
On 2016/08/02 03:11:54, lijeffrey wrote: > This change should take care of other failures that ...
4 years, 4 months ago (2016-08-02 04:13:58 UTC) #5
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/2175053002/40001
4 years, 4 months ago (2016-08-02 06:41:39 UTC) #8
commit-bot: I haz the power
4 years, 4 months ago (2016-08-02 06:54:59 UTC) #10
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/infra/infra/+/894991b7ef4e30fd4dcf79b17e5a0...

Powered by Google App Engine
This is Rietveld 408576698