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

Issue 681106 link

Starred by 1 user

Issue metadata

Status: Fixed
Merged: issue 680776
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

7.9% regression in page_cycler_v2.typical_25 at 442968:443155

Project Member Reported by nzolghadr@chromium.org, Jan 13 2017

Issue description

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

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


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

chromium-rel-mac-retina
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jan 13 2017

Mergedinto: 680776
Status: Duplicate (was: Untriaged)

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

Suspected Commit
  Author : zakerinasab
  Commit : 7fb9535883e50ab0d7262ea8a0e8470934e57a9c
  Date   : Wed Jan 11 19:09:13 2017
  Subject: Implement color management for ImageData

Bisect Details
  Configuration: mac_retina_perf_bisect
  Benchmark    : page_cycler_v2.typical_25
  Metric       : timeToFirstMeaningfulPaint_avg/pcv1-warm/http___www.html5rocks.com_en_
  Change       : 8.15% | 65.9559999804 -> 71.3308333357

Revision             Result                  N
chromium@442968      65.956 +- 3.45702       6      good
chromium@442972      65.5917 +- 1.21183      6      good
chromium@442973      71.2077 +- 3.1499       6      bad       <--
chromium@442974      70.4113 +- 1.98108      6      bad
chromium@442975      70.2417 +- 1.9968       6      bad
chromium@442981      71.3818 +- 3.53766      6      bad
chromium@442993      72.2773 +- 4.36405      6      bad
chromium@443015      70.8427 +- 2.34032      6      bad
chromium@443062      69.8733 +- 2.09298      6      bad
chromium@443155      71.3308 +- 1.91498      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=release --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=http...www.html5rocks.com.en. page_cycler_v2.typical_25

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8990556374084655104

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5834753137180672


| 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!
Owner: zakerinasab@chromium.org
Status: Assigned (was: Duplicate)
Un-duping since this bug is a performance regression and the other bug is a test breakage. zakerinasab: probably want to run this benchmark on the mac retina perf trybot before relanding.
Status: Fixed (was: Assigned)
The issue has been fixed and the CL is landed again at commit position #443939.(https://codereview.chromium.org/2555213002). Here is the link to the Chrome perf graph showing that the regression is fixed:

https://chromeperf.appspot.com/report?sid=a4d3fe175b3e112e74d6ce1f4d1eb51156def8b95f4b0a93936b31086af00bca&start_rev=442200&end_rev=444200
Labels: Performance-Loading

Sign in to add a comment