| Index: tools/telemetry/telemetry/value/merge_values.py
|
| diff --git a/tools/telemetry/telemetry/value/merge_values.py b/tools/telemetry/telemetry/value/merge_values.py
|
| index 89d187824e19fd7e5bb7a45083cae1f6e45f3e0d..6d24686db3ea061d314364de2e81ae9fe619ddf9 100644
|
| --- a/tools/telemetry/telemetry/value/merge_values.py
|
| +++ b/tools/telemetry/telemetry/value/merge_values.py
|
| @@ -3,6 +3,7 @@
|
| # found in the LICENSE file.
|
|
|
| from telemetry.value import failure
|
| +from telemetry.value import skip
|
|
|
| def MergeLikeValuesFromSamePage(all_values):
|
| """Merges values that measure the same thing on the same page.
|
| @@ -112,9 +113,12 @@ def GroupStably(all_values, key_func):
|
| merge_groups_in_creation_order = []
|
| for value in all_values:
|
| # TODO(chrishenry): This is temporary. When we figure out the
|
| - # right summarization strategy for page runs with failures, we
|
| + # right summarization strategy for page runs with failures/skips, we
|
| # should use that instead.
|
| - if isinstance(value, failure.FailureValue):
|
| + should_skip_value = (isinstance(value, failure.FailureValue) or
|
| + isinstance(value, skip.SkipValue))
|
| +
|
| + if should_skip_value:
|
| continue
|
|
|
| key = key_func(value)
|
|
|