New issue
Advanced search Search tips

Issue 609082 link

Starred by 4 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

On a page with multiple input fields, text-align kills performance

Reported by richard....@figured.com, May 4 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36

Example URL:

Steps to reproduce the problem:
1. Create a page with a large number (>500) input fields
2. Add text-align:right to the input

What is the expected behavior?
Scrolling remains the same as without text-align

What went wrong?
Scroll performance is very slow

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? Yes 

Chrome version: 50.0.2661.94  Channel: stable
OS Version: OS X 10.11.4
Flash Version: Shockwave Flash 21.0 r0

See this reduced test case:
Slow performance (with text-align) https://jsfiddle.net/h0ork3xc/
Normal (without) https://jsfiddle.net/361nfqsx/
 
Components: -Blink Blink>Scroll
Cc: e...@chromium.org tdres...@chromium.org
Labels: Hotlist-Input-Dev
Status: Available (was: Unconfirmed)
I can confirm that the performance changes significantly when adding 'text-align: right';

Attached is a good, and bad (with text-align:right) trace. We are spending an awful lots of time in compositing (~150ms in PaintLayerCompositor::updateIfNeededRecursive, ) and paint related code (~150 ms in  FrameView::synchronizedPaint). 

This could be either a case of layer explosion.
Or a problem in painting logic, similar to https://bugs.chromium.org/p/chromium/issues/detail?id=588570


Need a bit more investigation to figure out the root cause. Putting this in Hotlist-input-dev for now.
trace_bad.json.gz
2.2 MB Download
trace_good.json.gz
810 KB Download
Components: -Blink>Scroll Blink>Compositing Blink>Paint
Status: Untriaged (was: Available)
The performance issue isn't scrolling related, relabeling.
Status: Available (was: Untriaged)
Project Member

Comment 6 by sheriffbot@chromium.org, May 19 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: schenney@chromium.org
Labels: -Hotlist-Recharge-Cold PaintTeamTriaged-20170519 BugSource-User
Status: Available (was: Untriaged)
This might be anpther one of the bugs related to layer work. I'll be looking at this next week.
Might be hit testing too.
Project Member

Comment 9 by sheriffbot@chromium.org, May 21 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 10 by e...@chromium.org, May 21 2018

Status: Available (was: Untriaged)
This will be fixed in NG where alignment happens during layout.
Components: -Blink>Compositing -Blink>Paint Blink>Layout
Labels: Fixed-In-LayoutNG

Sign in to add a comment