New issue
Advanced search Search tips

Issue 671644 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Move scrollbars entirely into CC

Project Member Reported by bokan@chromium.org, Dec 6 2016

Issue description

Currently, if a scroller is composited, its scrollbars will get layers in CC. If it's non-composited, its scrollbars will be painted directly into the scroller's backing canvas (see these slides for a more detailed break down of how scrollbars work today: https://docs.google.com/a/chromium.org/presentation/d/1lHuKG6ROJDCBQnQvPfWPiOZPpTXLZQCARTQgsHPFRHk/edit?usp=sharing).

Much of the pain of this double implementation is well handled, paint code is set in one place and event handling is done entirely on the main thread. We just have some indirection and plumbing to make it work in both places.

However, animations for overlay scrollbars are only available on the CC side. This means non composited scrollers don't get animations for scrollbars (scrollbars in a non-composited scroller on Android will never disappear). Duplicating the animation logic to the main thread would be a maintenance burden and bad for performance.

It seems that with SPv2, since compositing decisions happen in CC, we should be able to create CC scrollbars even for scrollers that need to repaint on each scroll. In that case, we could make sure all scrollbars are always animated. It might even make sense to move the event handling logic from Blink into CC.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Dec 11 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

Comment 2 by bokan@chromium.org, Dec 14 2017

Blockedon: -471333
Status: WontFix (was: Untriaged)
This will need some more thought and isn't as simple as I may have thought in talks with Slimming Paint team. We'll be planning how to implement scrollbars in 2018 Q2 so this will come up at that point.

Sign in to add a comment