Remove spaceport benchmark or moving it to tools/perf/contrib/ directory |
|||||
Issue descriptionThis benchmark is not run on any platform due to it being timed out. (see https://cs.chromium.org/chromium/src/tools/perf/benchmarks/spaceport.py?rcl=2b18a942f54f19aff1b52b7aeb3cf897a422b70b&l=105) +Junov: do you want to still keep this benchmarks' code around?
,
May 3 2017
Ping junov@, we are trying to do the benchmark cleanup this quarter.
,
May 5 2017
This benchmark is mostly redundant with others that test sprites. I approve zapping it.
,
May 5 2017
Thanks!
,
May 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/70763ca5a8ee376df302e4a411aa1ab46303f69a commit 70763ca5a8ee376df302e4a411aa1ab46303f69a Author: Nghia Nguyen <NghiaNguyenBH@gmail.com> Date: Fri May 05 12:19:49 2017 Remove spaceport benchmark Bug:717210 Change-Id: Ic87c175732ea7ed5fb38534125f1f64951022ea2 TBR=junov@chromium.org Change-Id: Ic87c175732ea7ed5fb38534125f1f64951022ea2 Reviewed-on: https://chromium-review.googlesource.com/497309 Reviewed-by: Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#469625} [modify] https://crrev.com/70763ca5a8ee376df302e4a411aa1ab46303f69a/testing/buildbot/chromium.perf.fyi.json [modify] https://crrev.com/70763ca5a8ee376df302e4a411aa1ab46303f69a/testing/buildbot/chromium.perf.json [modify] https://crrev.com/70763ca5a8ee376df302e4a411aa1ab46303f69a/tools/perf/benchmark.csv [modify] https://crrev.com/70763ca5a8ee376df302e4a411aa1ab46303f69a/tools/perf/benchmarks/benchmark_smoke_unittest.py [delete] https://crrev.com/4d49107dce95a5491acebfab01023a377ae03b58/tools/perf/benchmarks/spaceport.py
,
May 5 2017
,
May 5 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by nedngu...@google.com
, May 2 2017