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

Unified Diff: tools/telemetry/telemetry/user_story/user_story_runner.py

Issue 962793004: [Telemetry] Make "discard_first_result" apply to user_stories too. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/telemetry/telemetry/user_story/user_story_runner.py
diff --git a/tools/telemetry/telemetry/user_story/user_story_runner.py b/tools/telemetry/telemetry/user_story/user_story_runner.py
index 74452c02c7c371b29d48591c5346606343c6b2a3..544f8bf4d8457e90696323082999c61071cbc61e 100644
--- a/tools/telemetry/telemetry/user_story/user_story_runner.py
+++ b/tools/telemetry/telemetry/user_story/user_story_runner.py
@@ -261,10 +261,8 @@ def Run(test, user_story_set, expectations, finder_options, results,
try:
if state:
_CheckThermalThrottling(state.platform)
- # TODO(slamm): Make discard_first_result part of user_story API.
- # https://crbug.com/440101
discard_current_run = (
- getattr(test, 'discard_first_result', False) and
+ finder_options.discard_first_result and
user_story not in user_story_with_discarded_first_results)
if discard_current_run:
user_story_with_discarded_first_results.add(user_story)

Powered by Google App Engine
This is Rietveld 408576698