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

Issue 828008 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 826829
Owner: ----
Closed: Apr 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

6.8%-116.4% regression in loading.desktop at 507663:546070

Project Member Reported by pmeenan@chromium.org, Apr 2 2018

Issue description

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

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


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

chromium-rel-mac11-air
chromium-rel-mac12
chromium-rel-win7-gpu-ati
chromium-rel-win8-dual
😿 Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/14977b00c40000
Cc: benjhayden@chromium.org sky@chromium.org kenrb@chromium.org oysteine@chromium.org boliu@chromium.org piman@chromium.org afakhry@chromium.org chiniforooshan@chromium.org dproy@chromium.org
Owner: dproy@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/148c306cc40000

tracing: TracingController -> Coordinator by chiniforooshan@chromium.org
https://chromium.googlesource.com/chromium/src/+/c77b44ebb53886e039c8471548c2050cceceda59

Get navigation info without using FrameLoader snapshots by dproy@chromium.org
https://chromium.googlesource.com/catapult/+/5d35a2c7122c1ac4a9a37e7743a19f628f25a38b

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 7 by dproy@chromium.org, Apr 11 2018

Mergedinto: 826829
Status: Duplicate (was: Assigned)

Comment 8 by dproy@chromium.org, Apr 12 2018

Owner: ----
Status: Available (was: Duplicate)
Unduplicating upon further investigation - it doesn't look like the navigation start event for FMP changed so this is not because of my CL. 

Looks like the metric was broken for a quite a few commits in the middle, and the traces before and after look really different. Are we using a different page now? For example, before the regression, the LocalFrameView::performLayout task takes 214ms, but after the regression, it is taking 538ms.

Unassigning myself because I don't have enough context to debug this. 
Project Member

Comment 9 by 42576172...@developer.gserviceaccount.com, Apr 13 2018

Cc: gambard@chromium.org drott@chromium.org bugsnash@chromium.org kojii@chromium.org marq@chromium.org bsheedy@chromium.org lambroslambrou@chromium.org foolip@chromium.org zijiehe@chromium.org bshe@chromium.org e...@chromium.org
Owner: gambard@chromium.org
Status: Assigned (was: Available)
📍 Found significant differences after each of 5 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16b7d59cc40000

[Chromoting Add codec parameter for all payload types by zijiehe@chromium.org
https://chromium.googlesource.com/chromium/src/+/f912ff75b23440a67e3674064db799c0f15a7602

tracing: TracingController -> Coordinator by chiniforooshan@chromium.org
https://chromium.googlesource.com/chromium/src/+/c77b44ebb53886e039c8471548c2050cceceda59

Update VR test APKs to v1.10 by bsheedy@chromium.org
https://chromium.googlesource.com/chromium/src/+/e825ae73219e421007090602d3abb04861a712ba

Text-decoration-skip to text-decoration-skip-ink with initial value auto by drott@chromium.org
https://chromium.googlesource.com/chromium/src/+/cc22f53f9455428f8d6ad59633730e6d4e7b5273

Background color for StackView snapshot animations by gambard@chromium.org
https://chromium.googlesource.com/chromium/src/+/5b35d0e9b9a2f54b58ad2535398c22bb8f473532

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 10 by zijiehe@google.com, Apr 13 2018

My change
[Chromoting Add codec parameter for all payload types by zijiehe@chromium.org
https://chromium.googlesource.com/chromium/src/+/f912ff75b23440a67e3674064db799c0f15a7602
impacts chromoting only.
Owner: ----
Status: Available (was: Assigned)
My change
Background color for StackView snapshot animations by gambard@chromium.org
https://chromium.googlesource.com/chromium/src/+/5b35d0e9b9a2f54b58ad2535398c22bb8f473532
impacts Chrome on iOS only. If I understand correctly, the problem is affecting desktop.

Unassigning myself.
Status: Duplicate (was: Available)

Sign in to add a comment