Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(99)

Issue 2684633002: Revert of [BotHealth] Disable speedometer benchmarks on Android (Closed)

Created:
3 years, 10 months ago by nednguyen
Modified:
3 years, 9 months ago
CC:
chromium-reviews, telemetry-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of [BotHealth] Disable speedometer benchmarks on Android (patchset #1 id:1 of https://codereview.chromium.org/2671463003/ ) Reason for revert: Root caused should be fixed in crbug.com/687737 Original issue's description: > [BotHealth] Disable speedometer benchmarks on Android > > BUG=687681 > > Review-Url: https://codereview.chromium.org/2671463003 > Cr-Commit-Position: refs/heads/master@{#447609} > Committed: https://chromium.googlesource.com/chromium/src/+/2dfbf027e6a3d4aaee892c9897c28bd18a39dabb TBR=simonhatch@chromium.org,charliea@chromium.org,eakuefner@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=687681

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M tools/perf/benchmarks/speedometer.py View 3 chunks +1 line, -4 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
nednguyen
Created Revert of [BotHealth] Disable speedometer benchmarks on Android
3 years, 10 months ago (2017-02-07 16:26:22 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2684633002/1
3 years, 10 months ago (2017-02-07 16:26:36 UTC) #3
commit-bot: I haz the power
Try jobs failed on following builders: ios-simulator on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/builds/150360)
3 years, 10 months ago (2017-02-07 16:57:54 UTC) #5
charliea (OOO until 10-5)
lgtm
3 years, 10 months ago (2017-02-07 20:13:16 UTC) #6
eakuefner
lgtm
3 years, 10 months ago (2017-02-17 23:43:05 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2684633002/1
3 years, 10 months ago (2017-02-17 23:44:37 UTC) #9
commit-bot: I haz the power
3 years, 10 months ago (2017-02-18 01:48:40 UTC) #11
Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

Powered by Google App Engine
This is Rietveld 408576698