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

Issue 2691993002: Enable smoothness.maps benchmark on all platforms. (Closed)

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

Description

Enable smoothness.maps benchmark on all platforms. The benchmark is currently enabled only for android, but there seems to be no reason why it cannot be run on desktop. Locally it runs OK on my Linux machine. The android-only restriction was introduced in this CL: https://codereview.chromium.org/587833003 BUG=691581 Review-Url: https://codereview.chromium.org/2691993002 Cr-Commit-Position: refs/heads/master@{#449985} Committed: https://chromium.googlesource.com/chromium/src/+/c2e7e0a2c9fb07ddc5cb71ef91ac0c157330fb61

Patch Set 1 #

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

Messages

Total messages: 10 (6 generated)
ulan
ptal
3 years, 10 months ago (2017-02-13 15:54:17 UTC) #3
nednguyen
lgtm
3 years, 10 months ago (2017-02-13 15:55:09 UTC) #4
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/2691993002/1
3 years, 10 months ago (2017-02-13 15:56:51 UTC) #6
commit-bot: I haz the power
3 years, 10 months ago (2017-02-13 17:14:12 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c2e7e0a2c9fb07ddc5cb71ef91ac...

Powered by Google App Engine
This is Rietveld 408576698