Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2203593002:
Re-enable some PCv1 benchmarks for CrOS (Closed)
|
Created:
4 years, 4 months ago by kouhei (in TOK) Modified:
4 years, 4 months ago Reviewers:
nednguyen CC:
chromium-reviews, telemetry-reviews_chromium.org Base URL:
https://chromium.googlesource.com/chromium/src.git@master Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionRe-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815, 611329
Committed: https://crrev.com/f00d65e14a65a308a1f7b9e07c96859fff8b5280
Cr-Commit-Position: refs/heads/master@{#409108}
Patch Set 1 #
Messages
Total messages: 15 (9 generated)
Description was changed from
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
BUG=632815,611329
==========
to
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
BUG=632815,611329
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========
Description was changed from
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
BUG=632815,611329
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========
to
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========
kouhei@chromium.org changed reviewers: + nednguyen@google.com
Ned, sorry for tbr-ing. This is basically a revert & cmtice@ wants this asap
The CQ bit was checked by kouhei@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Description was changed from
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========
to
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
==========
lgtm I removed the extra try bot since that takes forever to run ~_~
The CQ bit was unchecked by nednguyen@google.com
The CQ bit was checked by nednguyen@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
==========
to
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
==========
to
==========
Re-enable some PCv1 benchmarks for CrOS
This CL re-enables page_cycler.intl_{ja_zh,es_fr_pt-BR,ar_fa_he} PCv1 benchmarks
for ChromeOS
This is a temporary solution until crbug.com/632815 is resolved
TBR=nednguyen@google.com
BUG=632815,611329
Committed: https://crrev.com/f00d65e14a65a308a1f7b9e07c96859fff8b5280
Cr-Commit-Position: refs/heads/master@{#409108}
==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/f00d65e14a65a308a1f7b9e07c96859fff8b5280 Cr-Commit-Position: refs/heads/master@{#409108} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
