New issue
Advanced search Search tips

Issue 801543 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

7.5% improvement in speedometer at 518746:518863

Project Member Reported by cbruni@google.com, Jan 12 2018

Issue description

Documenting improvement.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jan 12 2018

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

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


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

android-nexus5
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jan 12 2018

Cc: lfg@chromium.org
Owner: lfg@chromium.org

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

Hi lfg@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 : Lucas Furukawa Gadani
  Commit : b9fddd03657a4b15506833bf7678a51785094dc7
  Date   : Wed Nov 22 22:55:17 2017
  Subject: Optimize HTML parser for speed instead of size on Android.

Bisect Details
  Configuration: android_nexus5_perf_bisect
  Benchmark    : speedometer
  Metric       : jQuery-TodoMVC/jQuery-TodoMVC
  Change       : 6.74% | 10571.3505556 -> 9858.54333333

Revision             Result                  N
chromium@518745      10571.4 +- 132.138      6      good
chromium@518772      10486.5 +- 38.375       6      good
chromium@518779      10466.5 +- 111.868      6      good
chromium@518780      10524.8 +- 218.338      6      good
chromium@518781      9918.52 +- 71.0468      6      bad       <--
chromium@518782      9939.27 +- 171.47       6      bad
chromium@518785      9924.35 +- 179.241      6      bad
chromium@518798      9809.1 +- 162.847       6      bad
chromium@518863      9858.54 +- 172.553      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 speedometer

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

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


For feedback, file a bug with component Speed>Bisection

Comment 4 by lfg@chromium.org, Jan 12 2018

Cc: jbroman@chromium.org adithyas@chromium.org
Components: Blink>HTML>Parser
Status: Fixed (was: Assigned)
Marking as fixed.

Sign in to add a comment