New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 733140 link

Starred by 1 user
Status: Fixed
Owner:
lijeffrey@chromium.org
Closed: Jun 2017
Cc:
█ robert...@chromium.org
wylieb@chromium.org
chanli@chromium.org
st...@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[Findit] Flake Analyzer - Get rid of template-side validation

Project Member Reported by lijeffrey@chromium.org, Jun 14 2017 
Template-side validation for valid input for manual regression range reun is redundant and already checked on the handler side
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 14 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/fcbbcd78e3fbfbf1544fb14216aee27a67eec0eb

commit fcbbcd78e3fbfbf1544fb14216aee27a67eec0eb
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Wed Jun 14 08:40:01 2017

[Findit] Flake Analyzer - Get rid of input validation on template side

Bug:  733140 
Change-Id: I9c3ec14dc344a978503f75db58532b9061a9b68e
Reviewed-on: https://chromium-review.googlesource.com/535074
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/fcbbcd78e3fbfbf1544fb14216aee27a67eec0eb/appengine/findit/templates/flake/result.html

Comment 2 by lijeffrey@chromium.org, Jun 14 2017

Status: Fixed (was: Started)
►

Sign in to add a comment