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

Issue 775114 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

14.2% regression in thread_times.tough_scrolling_cases at 508720:508852

Project Member Reported by chiniforooshan@chromium.org, Oct 16 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Oct 16 2017

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

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


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

android-nexus5
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Oct 16 2017

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

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

Hi bokan@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 : David Bokan
  Commit : b0d47f5f4d4b9717bfdd43ae6d40257235d0848f
  Date   : Fri Oct 13 16:49:07 2017
  Subject: Fix coordinate space issues in scroll.js

Bisect Details
  Configuration: android_nexus5_perf_bisect
  Benchmark    : thread_times.tough_scrolling_cases
  Metric       : thread_raster_cpu_time_per_frame/text_20000_pixels_per_second
  Change       : 13.44% | 7.21581534861 -> 8.18551747291

Revision                                 Result                   N
chromium@508719                          7.21582 +- 0.230003      6      good
chromium@508786                          7.30855 +- 0.112696      6      good
chromium@508789                          7.30139 +- 0.215748      6      good
chromium@508789,catapult@b0d47f5f4d      8.26186 +- 0.291858      6      bad       <--
chromium@508789,catapult@dd5b39ac0c      8.16784 +- 0.23463       6      bad
chromium@508790                          8.33951 +- 0.263316      6      bad
chromium@508791                          8.31915 +- 0.200547      6      bad
chromium@508795                          8.2648 +- 0.368315       6      bad
chromium@508803                          8.18923 +- 0.122025      6      bad
chromium@508819                          8.28614 +- 0.272462      6      bad
chromium@508852                          8.18552 +- 0.183363      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=text.20000.pixels.per.second thread_times.tough_scrolling_cases

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

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


For feedback, file a bug with component Speed>Bisection

Comment 4 by bokan@chromium.org, Oct 16 2017

Status: WontFix (was: Assigned)
this is undoing an "improvement" that comes from https://chromium-review.googlesource.com/668376. That CL fixes gestures when pinch-zoomed so it changed what gestures are produced and affects performance on various runs (some better, some worse) but it shouldn't have affected this particular metric. The blamed CL fixes the issue so it's back to the original metric.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Oct 18 2017

 Issue 775950  has been merged into this issue.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Oct 20 2017

Cc: pmeenan@chromium.org
 Issue 776752  has been merged into this issue.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Oct 20 2017

 Issue 776753  has been merged into this issue.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Oct 24 2017

 Issue 776750  has been merged into this issue.

Sign in to add a comment