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

Issue 762247 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

5.3%-10.6% regression in thread_times.key_silk_cases at 499529:499538

Project Member Reported by briander...@chromium.org, Sep 5 2017

Issue description

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

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


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

android-nexus6
Cc: bcwh...@chromium.org
Owner: bcwh...@chromium.org
Status: Assigned (was: Untriaged)

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

Hi bcwhite@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 : Brian White
  Commit : 1278570bd438b22d1a703d67d441d813839d44ac
  Date   : Tue Sep 05 01:25:16 2017
  Subject: Revert "Do histogram validation after every JNI call."

Bisect Details
  Configuration: android_nexus6_perf_bisect
  Benchmark    : thread_times.key_silk_cases
  Metric       : thread_raster_cpu_time_per_frame/infinite_scrolling.html
  Change       : 12.15% | 0.810862234568 -> 0.909353083151

Revision             Result                     N
chromium@499528      0.810862 +- 0.0161175      6      good
chromium@499533      0.807394 +- 0.0166091      6      good
chromium@499536      0.819805 +- 0.0539921      6      good
chromium@499537      0.914369 +- 0.0269363      6      bad       <--
chromium@499538      0.909353 +- 0.01471        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=infinite.scrolling.html thread_times.key_silk_cases

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

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


For feedback, file a bug with component Speed>Bisection
 Issue 762249  has been merged into this issue.
This was expected.  The CL that caused it has already been reverted.

 Issue 762274  has been merged into this issue.
Status: Fixed (was: Assigned)
 Issue 762273  has been merged into this issue.
Cc: rmcilroy@chromium.org
 Issue 763330  has been merged into this issue.
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Sep 14 2017

Cc: toyoshim@chromium.org
 Issue 765119  has been merged into this issue.

Sign in to add a comment