String16WildcardMatchInsensitive has its own implementation of pattern matching which looks similar to base::MatchPattern. The big difference is it allows any escape char (should be easy to add to MatchPattern).
However String16WildcardMatchInsensitive includes a cache for performance reasons - we'll need to check if MatchPattern matches the performance without the cache, or else keep the cache layer. Also need to check if we depend on any implementation quirks of String16WildcardMatchInsensitive.
Comment 1 by nparker@chromium.org
, May 4 2018