New issue
Advanced search Search tips

Issue 836040 link

Starred by 4 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Move RenderedPosition::ComputeCompositedPosition to LocalFrameView

Project Member Reported by xiaoche...@chromium.org, Apr 23 2018

Issue description

With recent refactoring, RenderedPosition::ComputeCompositedPosition():
- doesn't use any RenderedPosition instance
- uses its own static functions that are not used by other RenderedPosition member functions
- has only one call site in LocalFrameView

To improve code health, we should move the function, along with its test cases, to LocalFrameView.
 
Status: Available (was: Assigned)

Sign in to add a comment