Commit 2869b5de authored by Wenbin Zhang's avatar Wenbin Zhang Committed by Commit Bot

[benchmarking] updated instruction in benchmark_ownership.md

The current instruction is a bit confusing and the example 'smoothness'
no longer exists. Updating to make it clear.

Bug: chromium:1058129
Change-Id: Iaa37a77580035493744f2928be26e85ddfd0cf63
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089954
Commit-Queue: Wenbin Zhang <wenbinzhang@google.com>
Reviewed-by: default avatarCaleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747796}
parent 2ba40fac
...@@ -12,7 +12,7 @@ There can be multiple owners of a benchmark, for example if there are multiple t ...@@ -12,7 +12,7 @@ There can be multiple owners of a benchmark, for example if there are multiple t
## How do I update the owner on a benchmark? ## How do I update the owner on a benchmark?
### Telemetry Benchmarks ### Telemetry Benchmarks
1. Open [`src/tools/perf/benchmarks/benchmark_name.py`](https://cs.chromium.org/chromium/src/tools/perf/benchmarks/), where `benchmark_name` is the part of the benchmark before the “.”, like `smoothness` in `smoothness.top_25_smooth`. 1. Open <benchmark_harness>.py under [`src/tools/perf/benchmarks/`](https://cs.chromium.org/chromium/src/tools/perf/benchmarks/). The <benchmark_harness> is the part of the benchmark name before the “.”. For example, when looking for 'blink_perf.paint', you need to open 'blink_perf.py'.
1. Find the class for the benchmark. It has a `Name` method that should match the full name of the benchmark. 1. Find the class for the benchmark. It has a `Name` method that should match the full name of the benchmark.
1. Add a `benchmark.Info` decorator above the class. 1. Add a `benchmark.Info` decorator above the class.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment