[Telemetry] Split Oopif page_cycler_v2 benchmarks into their own file.

[Telemetry] Split Oopif page_cycler_v2 benchmarks into their own file.

We are currently working on removing the page_cycler_v2 benchmarks, replacing them
with the new loading.desktop benchmark. The Oopif page cycler v2 benchmarks are
not yet ready for this, so we have to split them into their own benchmarks seperate
from page_cycler_v2. The name shall rename the same (including the page_cycler_v2)
in order to keep the data in the same place on the dashboard. Eventually Oopif
benchmarks will be migrated to the new loading benchmark harness as well.

BUG=713346
Review-Url: https://codereview.chromium.org/2884383006
Cr-Commit-Position: refs/heads/master@{#473067}
Committed: https://chromium.googlesource.com/chromium/src/+/5cf77c9e72082bfc4b0a7a9e1481b20569eacac5

[rnephew (Reviews Here), 2017-05-18 23:22:47]

rnephew@chromium.org changed reviewers:
+ charliea@chromium.org, nednguyen@google.com

[rnephew (Reviews Here), 2017-05-18 23:22:48]

Nasko@, from your perspective this CL should be a noop. Everything should go to
the dashboard the same as before. I will reach out to you later when work to
moving Oopif to the loading harness starts.

[nednguyen, 2017-05-18 23:27:28]

nednguyen@google.com changed reviewers:
+ pasko@chromium.org

[nednguyen, 2017-05-18 23:27:28]

lgtm

[nednguyen, 2017-05-18 23:28:21]

https://codereview.chromium.org/2884383006/diff/1/tools/perf/benchmarks/oopif.py
File tools/perf/benchmarks/oopif.py (right):

https://codereview.chromium.org/2884383006/diff/1/tools/perf/benchmarks/oopif...
tools/perf/benchmarks/oopif.py:19: def
AugmentOptionsForLoadingMetrics(tbm_options):
if this is moved to loading.py, can we reuse it instead of copy & paste?

[rnephew (Reviews Here), 2017-05-18 23:28:23]

On 2017/05/18 23:27:28, nednguyen wrote:
> lgtm

I added a simple TODO telling me to eventually migrate this to the loading
harness. Thats the only change after your lgtm, so committing.

[rnephew (Reviews Here), 2017-05-18 23:28:33]

The CQ bit was checked by rnephew@chromium.org

[rnephew (Reviews Here), 2017-05-18 23:28:34]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2884383006/#ps20001
(title: "add a todo")

[commit-bot: I haz the power, 2017-05-18 23:30:49]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-19 03:47:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495150113943830,
"parent_rev": "9c90eb122f3327c8c03bbf2171a97cb157f89459", "commit_rev":
"5cf77c9e72082bfc4b0a7a9e1481b20569eacac5"}

[commit-bot: I haz the power, 2017-05-19 03:48:03]

Description was changed from

==========
[Telemetry] Split Oopif page_cycler_v2 benchmarks into their own file.

We are currently working on removing the page_cycler_v2 benchmarks, replacing
them
with the new loading.desktop benchmark. The Oopif page cycler v2 benchmarks are
not yet ready for this, so we have to split them into their own benchmarks
seperate
from page_cycler_v2. The name shall rename the same (including the
page_cycler_v2)
in order to keep the data in the same place on the dashboard. Eventually Oopif
benchmarks will be migrated to the new loading benchmark harness as well.

BUG=713346
==========

to

==========
[Telemetry] Split Oopif page_cycler_v2 benchmarks into their own file.

We are currently working on removing the page_cycler_v2 benchmarks, replacing
them
with the new loading.desktop benchmark. The Oopif page cycler v2 benchmarks are
not yet ready for this, so we have to split them into their own benchmarks
seperate
from page_cycler_v2. The name shall rename the same (including the
page_cycler_v2)
in order to keep the data in the same place on the dashboard. Eventually Oopif
benchmarks will be migrated to the new loading benchmark harness as well.

BUG=713346

Review-Url: https://codereview.chromium.org/2884383006
Cr-Commit-Position: refs/heads/master@{#473067}
Committed:
https://chromium.googlesource.com/chromium/src/+/5cf77c9e72082bfc4b0a7a9e1481...
==========

[commit-bot: I haz the power, 2017-05-19 03:48:04]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/5cf77c9e72082bfc4b0a7a9e1481...