New issue
Advanced search Search tips

Issue 721333 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

121.3% regression in page_cycler_v2.typical_25 at 470448:470519

Project Member Reported by hjd@google.com, May 11 2017

Issue description

Suspect commit https://chromium.googlesource.com/chromium/src/+/91088a195969f5af61f404c9f02edc622cbe373a being tracked at crbug/721238. Bisect to confirm.
 

Comment 1 by hjd@google.com, May 11 2017

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

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


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

chromium-rel-win7-gpu-nvidia
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, May 11 2017

Mergedinto: 721238
Status: Duplicate (was: Untriaged)

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

Suspected Commit
  Author : yhirano
  Commit : 91088a195969f5af61f404c9f02edc622cbe373a
  Date   : Wed May 10 05:49:31 2017
  Subject: Add Field Trial Testing Configuration for MojoLoading

Bisect Details
  Configuration: winx64nvidia_perf_bisect
  Benchmark    : page_cycler_v2.typical_25
  Metric       : timeToFirstMeaningfulPaint_avg/pcv1-warm/http___www.nationalgeographic.com_
  Change       : 120.37% | 160.320499999 -> 353.292833333

Revision             Result                  N
chromium@470447      160.32 +- 17.0195       6      good
chromium@470483      162.547 +- 16.0452      6      good
chromium@470488      159.934 +- 7.76556      6      good
chromium@470489      184.486 +- 138.924      6      good
chromium@470490      367.157 +- 49.4864      6      bad       <--
chromium@470492      350.803 +- 32.4487      6      bad
chromium@470501      357.201 +- 17.2686      6      bad
chromium@470519      353.293 +- 9.3098       6      bad

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

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

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


| 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 Speed>Bisection.  Thank you!
Status: WontFix (was: Duplicate)
We decided to not to care this change. See the internal doc:

https://docs.google.com/document/d/1wissv31WUjeHaSaf78uAH0FiFQfSU6sqv14Lcuo1toA/edit
Cc: ksakamoto@chromium.org kouhei@chromium.org
cc-ing metric owners kouhei and ksakamoto as FYI, see document in #4 if you haven't read it already.
This is a false-alert due to the non-elasticity of TTFMP.
WontFix-ing this sgtm.

Sign in to add a comment