New issue
Advanced search Search tips

Issue 765572 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Missing Test]: Replace all hardcoded occurrences of ThreatSource with a call to GetThreatSource()

Project Member Reported by pnangunoori@chromium.org, Sep 15 2017

Issue description

Automated tests for the below commit have been missing.Would it be possible to add test coverage to avoid regressions in future?

CL: 
----
https://chromium.googlesource.com/chromium/src.git/+/2f22ffc124d1fbdeb7d420e050ccd55dda2bf95a

Ref Bug: 
---------
https://bugs.chromium.org/p/chromium/issues/detail?id=764965

Thank you.
 

Comment 1 by vakh@chromium.org, Oct 2 2017

Status: WontFix (was: Assigned)
this change was extremely minor and the existing test logic doesn't make it easy to add test for this.

Sign in to add a comment