Chromium Code Reviews| Index: appengine/findit/common/waterfall/failure_type.py |
| diff --git a/appengine/findit/common/waterfall/failure_type.py b/appengine/findit/common/waterfall/failure_type.py |
| index c95c0174bc26be716c5f1a745c3bd81e2e804378..58c51dc2d9a3f28032094ce2f6e0fc69109449f4 100644 |
| --- a/appengine/findit/common/waterfall/failure_type.py |
| +++ b/appengine/findit/common/waterfall/failure_type.py |
| @@ -6,6 +6,7 @@ UNKNOWN = 0x00 |
| INFRA = 0x01 |
| COMPILE = 0x08 |
| TEST = 0x10 |
| +BOTH = 0x18 |
|
stgao
2016/09/13 17:23:27
What does 'both' mean here? Both compile and test
chanli
2016/09/13 18:06:30
This type should be really rare. But I remembered
|
| def GetDescriptionForFailureType(failure_type): # pragma: no cover. |
| @@ -14,5 +15,6 @@ def GetDescriptionForFailureType(failure_type): # pragma: no cover. |
| INFRA: 'infra', |
| COMPILE: 'compile', |
| TEST: 'test', |
| + BOTH: 'both' |
| } |
| return description.get(failure_type, 'No description for %s' % failure_type) |