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

Issue 695543 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

battor.power_cases is no longer necessary and should be deleted

Project Member Reported by charliea@chromium.org, Feb 23 2017

Issue description

battor.power_cases was originally written as a simple benchmark to get up on the FYI waterfall to make sure that BattOr benchmarks could run.

Fast forward to now, where it has three stories (about:blank, YouTube, and poster circle), two of which are covered by the system_health.common benchmark we have in place and the third (poster circle) that's already been named as an unrealistic story.

I think the time has come for battor.power_cases to meet its maker.
 
Sniff. Tear.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/798a5b1d03f4760f0c5df2404c3af3c666259b69

commit 798a5b1d03f4760f0c5df2404c3af3c666259b69
Author: charliea <charliea@chromium.org>
Date: Wed Mar 01 17:38:21 2017

Delete battor.power_cases and the power_cases page set

battor.power_cases was originally written as a simple benchmark to get
up on the FYI waterfall to make sure that BattOr benchmarks could run.

Fast forward to now, where it has three stories (about:blank, YouTube,
and poster circle), two of which are covered by the
system_health.common benchmark we have in place and the third (poster
circle) that's already been named as an unrealistic story.

BUG= 695543 

Review-Url: https://codereview.chromium.org/2718493003
Cr-Commit-Position: refs/heads/master@{#453961}

[modify] https://crrev.com/798a5b1d03f4760f0c5df2404c3af3c666259b69/testing/buildbot/chromium.perf.fyi.json
[modify] https://crrev.com/798a5b1d03f4760f0c5df2404c3af3c666259b69/testing/buildbot/chromium.perf.json
[modify] https://crrev.com/798a5b1d03f4760f0c5df2404c3af3c666259b69/tools/perf/benchmarks/battor.py
[delete] https://crrev.com/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7/tools/perf/page_sets/data/power_cases.json
[delete] https://crrev.com/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7/tools/perf/page_sets/data/power_cases_000.wpr.sha1
[delete] https://crrev.com/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7/tools/perf/page_sets/data/power_cases_001.wpr.sha1
[delete] https://crrev.com/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7/tools/perf/page_sets/data/power_cases_002.wpr.sha1
[delete] https://crrev.com/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7/tools/perf/page_sets/power_cases.py

Status: Fixed (was: Assigned)

Sign in to add a comment