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

Issue 1949843003: Disable smoothness.scrolling_tough_ad_cases on android. (Closed)

Created:
4 years, 7 months ago by qyearsley
Modified:
4 years, 7 months ago
Reviewers:
dtu, shatch
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.

Description

Disable smoothness.scrolling_tough_ad_cases on android. This benchmark is failing most of the time on ref build and ToT for Android Nexus 7v2 and Nexus 9. BUG=607616 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Patch Set 1 #

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

Messages

Total messages: 5 (2 generated)
qyearsley
In this case, should I find a way to disable it for just Android 7v2 ...
4 years, 7 months ago (2016-05-04 17:02:08 UTC) #3
shatch
On 2016/05/04 17:02:08, qyearsley wrote: > In this case, should I find a way to ...
4 years, 7 months ago (2016-05-04 17:52:20 UTC) #4
qyearsley
4 years, 7 months ago (2016-05-04 17:57:46 UTC) #5
On 2016/05/04 at 17:52:20, simonhatch wrote:
> On 2016/05/04 17:02:08, qyearsley wrote:
> > In this case, should I find a way to disable it for just Android 7v2 and 9?
> 
> I have a CL out to do that, https://codereview.chromium.org/1936293002/,
didn't realize it had failed to land.

Ah, right, that makes sense. Closing this CL in favor of that one.

Powered by Google App Engine
This is Rietveld 408576698