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

Issue 750227 link

Starred by 7 users

Issue metadata

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



Sign in to add a comment

85.6% regression in page_cycler_v2_site_isolation.basic_oopif at 489656:489840

Project Member Reported by majidvp@google.com, Jul 28 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jul 28 2017

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

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


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

linux-release
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jul 28 2017

Cc: csharrison@chromium.org
Owner: csharrison@chromium.org

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

Hi csharrison@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 : Charles Harrison
  Commit : 7847afc23c816960a019476e995c696143365cac
  Date   : Thu Jul 27 01:15:34 2017
  Subject: Remove the DocumentWriteEvaluator

Bisect Details
  Configuration: linux_perf_bisect
  Benchmark    : page_cycler_v2_site_isolation.basic_oopif
  Metric       : timeToFirstContentfulPaint_avg/pcv1-warm/http___www.fifa.com_
  Change       : 44.43% | 70.0029999996 -> 101.105666667

Revision             Result                  N
chromium@489655      70.003 +- 1.62286       6      good
chromium@489748      71.5963 +- 2.00195      6      good
chromium@489794      75.0007 +- 30.5858      9      good
chromium@489817      73.3155 +- 4.6223       6      good
chromium@489823      73.4784 +- 8.54781      9      good
chromium@489824      70.5685 +- 2.16422      6      good
chromium@489825      83.6666 +- 44.4148      9      bad       <--
chromium@489826      89.5475 +- 36.5552      6      bad
chromium@489829      87.6074 +- 56.1445      9      bad
chromium@489840      101.106 +- 58.9688      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.fifa.com. page_cycler_v2_site_isolation.basic_oopif

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

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


For feedback, file a bug with component Speed>Bisection
Cc: kouhei@chromium.org
Status: WontFix (was: Untriaged)
Yep, I'm sad we're losing this optimization too! Unfortunately it didn't impact user perceived metrics in Stable channel, and it hasn't been enabled in a while.

If anyone is interested in reviving the experiment, please contact me.

(BTW I confirmed that the fifa.com hosted in telemetry does receive this optimization). 
Cc: sullivan@google.com
 Issue 751155  has been merged into this issue.
Cc: sullivan@chromium.org mtklein@chromium.org
 Issue 751632  has been merged into this issue.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Aug 10 2017

Issue 753794 has been merged into this issue.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Aug 21 2017

 Issue 750820  has been merged into this issue.

Sign in to add a comment