New issue
Advanced search Search tips

Issue 648909 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

2.7% regression in smoothness.sync_scroll.key_mobile_sites_smooth at 419745:419777

Project Member Reported by toyoshim@chromium.org, Sep 21 2016

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=648909

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICghaKtswsM


Bot(s) for this bug's original alert(s):

android-nexus5
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Sep 21 2016

Cc: mdjones@chromium.org
Owner: mdjones@chromium.org

=== Auto-CCing suspected CL author mdjones@chromium.org ===

Hi mdjones@chromium.org, the bisect results pointed to your CL below as possibly
causing a regression. Please have a look at this info and see whether
your CL be related.


===== BISECT JOB RESULTS =====
Status: completed


===== SUSPECTED CL(s) =====
Subject : Update testing config to use smooth-indeterminate animation
Author  : mdjones
Commit description:
  
Use smooth-indeterminate animation for the progress bar instead of
only smooth.

BUG=

Review-Url: https://codereview.chromium.org/2353553002
Cr-Commit-Position: refs/heads/master@{#419774}
Commit  : c23774e0a7fb2071658a7dec45bc0da7f068824e
Date    : Tue Sep 20 15:46:42 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev    N  Good?
chromium@419744  17.0927  0.573936   8  good
chromium@419761  16.9244  0.109838   5  good
chromium@419769  16.8981  0.0773199  5  good
chromium@419773  16.9377  0.0836449  5  good
chromium@419774  17.3924  0.150438   5  bad    <--
chromium@419775  17.3375  0.0857482  5  bad
chromium@419777  17.3797  0.141271   8  bad

Bisect job ran on: android_nexus5_perf_bisect
Bug ID: 648909

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests smoothness.sync_scroll.key_mobile_sites_smooth
Test Metric: frame_times/http___nytimes.com_
Relative Change: 0.62%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/4137
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9000920608085801840


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5209179013251072

| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Tests>AutoBisect.  Thank you!
Perf sheriff ping
Hi mdjones@, does it look like the above CL (Update testing config to use smooth-indeterminate animation) could have affected the performance test results for the android-nexus5 perf bot? If so, do you think is this an actual regression?
Friendly perf sheriff ping. mdjones@ can you please respond to #5. If this is an actual regression we may want decide to revert the change or look into potential mitigations.

This is probably a real regression but it is expected. There is ongoing discussion about the feature this config is for: https://crbug.com/464377
Status: WontFix (was: Assigned)

Sign in to add a comment