New issue
Advanced search Search tips

Issue 716991 link

Starred by 2 users

Issue metadata

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

Blocking:
issue 709815



Sign in to add a comment

Clean up and make case-insensitive string searches have a consistent interface

Project Member Reported by dcheng@chromium.org, May 1 2017

Issue description

Today, there's Find(), FindIgnoringASCII(), and FindIgnoringCase(). Find() also takes an optional enum argument that controls case-sensitivity of matching.

However, Contains(), StartsWith(), and EndsWith() don't have any equivalents of the IgnoringASCIICase() and IgnoringCase() versions. We should add them for consistency, switch code to use them, and see if we can even remove the enum argument from Find(), so that we can name Find() identically to std::string::find.
 

Sign in to add a comment