New issue
Advanced search Search tips

Issue 750181 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 749359
Owner: ----
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

14.1%-20.1% regression in blink_perf.events at 488939:489058

Project Member Reported by majidvp@google.com, Jul 28 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jul 28 2017

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=750181

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=c839b0922cc892354a5813a557009b62ef3fc1b51b0287868e42f110828d6a6a


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

chromium-rel-win7-dual
chromium-rel-win7-gpu-intel
chromium-rel-win7-gpu-nvidia
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jul 28 2017

Mergedinto: 749359
Status: Duplicate (was: Untriaged)

=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : Justin Schuh
  Commit : 4a572da99ce704e632221bd05e6d894d49a73dcc
  Date   : Mon Jul 24 14:37:24 2017
  Subject: Replace Saturated*() calls with Clamp*() templates

Bisect Details
  Configuration: winx64intel_perf_bisect
  Benchmark    : blink_perf.events
  Metric       : hit-test-lots-of-layers/hit-test-lots-of-layers
  Change       : 9.54% | 0.850697263947 -> 0.769534089611

Revision             Result                      N
chromium@488964      0.850697 +- 0.00283987      6      good
chromium@488965      0.848857 +- 0.0025474       6      good
chromium@488966      0.772099 +- 0.0101274       6      bad       <--
chromium@488967      0.770972 +- 0.0109101       6      bad
chromium@488970      0.76816 +- 0.0089224        6      bad
chromium@488976      0.772901 +- 0.0102748       6      bad
chromium@488988      0.77834 +- 0.00273552       6      bad
chromium@489011      0.772451 +- 0.00519362      6      bad
chromium@489058      0.769534 +- 0.00308258      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=release_x64 --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests blink_perf.events

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8972812162651876944


For feedback, file a bug with component Speed>Bisection
Status: Duplicate (was: Untriaged)
Fixing duplicate marking from #3.

Sign in to add a comment