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

Issue 792921 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

59.8% regression in system_health.memory_desktop at 521685:521781

Project Member Reported by npm@chromium.org, Dec 7 2017

Issue description

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

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


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

chromium-rel-mac12
๐Ÿ“ Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/15ec188c040000
Cc: flackr@chromium.org kinuko@chromium.org yigu@chromium.org boliu@chromium.org
Owner: yigu@chromium.org
Status: Assigned (was: Untriaged)
๐Ÿ“ Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/15ec188c040000

Round the scroll offset synced back to main thread instead of flooring
By yigu@chromium.org ยท Tue Dec 05 15:49:54 2017
chromium @ a3ce2d3110a3f9177a41935daadd43ede72db553

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
This may be expected from us compositing more elements on low dpi as that CL also re-enabled compositing opaque fixed position elements. Yi, can you verify this is the case?

Comment 5 by yigu@chromium.org, Dec 7 2017

Investigating the website https://king.com/game/alphabetty

Comment 6 by npm@chromium.org, Dec 7 2017

BTW keep in mind the graph also exhibits a large downwards spike after the regression.

Comment 7 by yigu@chromium.org, Dec 7 2017

There are lots of elements getting composited due to the change which caused the regression. This should be expected.

Comment 8 by npm@chromium.org, Dec 7 2017

So this regression is intended from your change? Please mark as WontFix if that's the case.

Comment 9 by yigu@chromium.org, Dec 7 2017

Status: WontFix (was: Assigned)

Comment 10 by npm@chromium.org, Dec 7 2017

 Issue 792930  has been merged into this issue.

Comment 11 by npm@chromium.org, Dec 7 2017

 Issue 792922  has been merged into this issue.
Status: Assigned (was: WontFix)
This is a pretty significant regression that is wontfixed because of an untested hypothesis. Who is the right person to test this hypothesis?

Comment 13 by yigu@chromium.org, Dec 14 2017

I investigated the test page (https://king.com/play/alphabetty) before I closed the bug. On the page there are some elements that were not but are composited now due to the change we made therefore the memory cost increase is expected.

Comment 14 by yigu@chromium.org, Dec 14 2017

Cc: -yigu@chromium.org benhenry@chromium.org
Owner: erikc...@chromium.org
Erik - should we wontfix this?

Comment 16 by yigu@chromium.org, Jan 10 2018

Cc: yigu@chromium.org

Comment 17 by herb@google.com, Jan 22 2018

Cc: danakj@chromium.org erikc...@chromium.org herb@google.com fmalita@chromium.org
 Issue 792938  has been merged into this issue.
Status: WontFix (was: Assigned)
As per c#13, closing as WontFix.

Sign in to add a comment