New issue
Advanced search Search tips

Issue 754785 link

Starred by 1 user
Status: WontFix
Owner:
skobes@chromium.org
Closed: Aug 2017
Cc:
szager@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 417782



Sign in to add a comment

should LayoutView::ViewRect exclude scrollbars with RLS?

Project Member Reported by skobes@chromium.org, Aug 11 2017 
ViewRect currently excludes FrameView scrollbars, but not root PLSA scrollbars.

At least some callers want to exclude layout viewport scrollbars regardless of RLS setting - for example, ViewPainter in r493525.

Let's try switching it and see what breaks. :)

Comment 1 by skobes@chromium.org, Aug 11 2017 

Related: should the ViewRect location reflect the root PLSA scroll offset?

Comment 2 by skobes@chromium.org, Aug 11 2017

Status: WontFix (was: Started)
Looks like this breaks hit testing (All/ParameterizedWebFrameTest.MouseOverRootScrollbar/1).

Eventually we should get rid of ViewRect and make callers go through the ScrollableArea API, but this is not blocking for RLS.  Let's close this and reevaluate API cleanup after RLS launches.
►

Sign in to add a comment