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

Issue 651977 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Rename LayoutObject::{container,containingBlock} for clarity.

Project Member Reported by wkorman@chromium.org, Sep 30 2016

Issue description

Breakout from http://crrev.com/2380933004 -- there is a desire to rename LayoutObject::container() and LayoutObject::containingBlock(), and potentially other containerFoo()/containingBar() methods, for clarity.

For background see the comment in Source/core/layout/README.md regarding the difference between container() and containingBlock().

These are heavily referenced methods, and we may want to rename associated comments and local vars, so having its own tracking bug seems warranted.
 
Status: WontFix (was: Assigned)
I no longer think we should do this.

Sign in to add a comment