New issue
Advanced search Search tips

Issue 891771 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Standardize on "get()" naming in Blink

Project Member Reported by dcheng@chromium.org, Oct 3

Issue description

Another potential yak shave. When we did the rename, we left an exception for STL-like things, so HashMap::find(), HashMap::end(), et cetera retained their lowercase names.

We should probably do the same for Get(): mixing casing of Get() in Blink seems rather confusing

(I'm marking this as an Oilpan bug, since the Oilpan handle classes are the most notable instance of this, though I'm sure there are others)

The flip side of this is that v8 mostly uses Get() as well (though they have a few get() thrown in there). But I think it's probably still worth standardizing on get() in Blink?
 

Sign in to add a comment