New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 763817 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

3.3%-4.5% regression in speedometer at 500441:500556

Project Member Reported by bmeu...@chromium.org, Sep 11 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

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

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


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

chromium-rel-mac11-air
chromium-rel-win8-dual
win-high-dpi
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

Cc: mgiuca@chromium.org
Owner: mgiuca@chromium.org
Status: Assigned (was: Untriaged)

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

Hi mgiuca@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


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

Suspected Commit
  Author : Matt Giuca
  Commit : 16a22949c41721a9a8b80aa3a0c801dfb54b6bb8
  Date   : Fri Sep 08 05:05:40 2017
  Subject: Revert "BlinkGC: Cleanup heap counters"

Bisect Details
  Configuration: winx64_high_dpi_perf_bisect
  Benchmark    : speedometer
  Metric       : AngularJS-TodoMVC/AngularJS-TodoMVC
  Change       : 3.10% | 670.117285714 -> 649.3255

Revision             Result                  N
chromium@500507      670.117 +- 79.2167      21      good
chromium@500509      676.275 +- 55.2248      6       good
chromium@500510      680.418 +- 41.7372      6       good
chromium@500511      647.562 +- 7.06983      6       bad       <--
chromium@500514      642.273 +- 64.7875      14      bad
chromium@500520      652.099 +- 87.0006      20      bad
chromium@500532      642.142 +- 24.8387      14      bad
chromium@500556      649.326 +- 89.354       20      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 speedometer

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

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


For feedback, file a bug with component Speed>Bisection

Comment 4 by mgiuca@chromium.org, Sep 12 2017

Owner: keishi@chromium.org
This was a revert I made during sheriffing (presumably of a performance improvement).

Assigning to keishi to re-land r500487 if not already done.

Sign in to add a comment