Use StringView for String::findIgnoringCase and findIgnoringASCIICase.

Use StringView for String::findIgnoringCase and findIgnoringASCIICase.

StringImpl's version of this already takes a StringView, but the proxy
functions on String took a const String& so callers like
didFireWebGLErrorOrWarning doing message.findIgnoringCase("error") were
allocating temporary strings.

Instead we make the String methods take a StringView too to avoid the
extra allocations. I also added these methods to AtomicString so the API
is the same.

BUG=615174
Committed: https://crrev.com/c34b20c196d32daae5764098479936bc7e10e61f
Cr-Commit-Position: refs/heads/master@{#410973}

[esprehn, 2016-08-09 21:18:08]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-09 21:18:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[esprehn, 2016-08-09 22:38:32]

esprehn@chromium.org changed reviewers:
+ haraken@chromium.org, yutak@chromium.org

[commit-bot: I haz the power, 2016-08-09 23:37:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-09 23:37:21]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[haraken, 2016-08-10 00:58:13]

LGTM

[esprehn, 2016-08-10 01:01:29]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-08-10 01:02:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 05:53:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 05:55:42]

Description was changed from

==========
Use StringView for String::findIgnoringCase and findIgnoringASCIICase.

StringImpl's version of this already takes a StringView, but the proxy
functions on String took a const String& so callers like
didFireWebGLErrorOrWarning doing message.findIgnoringCase("error") were
allocating temporary strings.

Instead we make the String methods take a StringView too to avoid the
extra allocations. I also added these methods to AtomicString so the API
is the same.

BUG=615174
==========

to

==========
Use StringView for String::findIgnoringCase and findIgnoringASCIICase.

StringImpl's version of this already takes a StringView, but the proxy
functions on String took a const String& so callers like
didFireWebGLErrorOrWarning doing message.findIgnoringCase("error") were
allocating temporary strings.

Instead we make the String methods take a StringView too to avoid the
extra allocations. I also added these methods to AtomicString so the API
is the same.

BUG=615174

Committed: https://crrev.com/c34b20c196d32daae5764098479936bc7e10e61f
Cr-Commit-Position: refs/heads/master@{#410973}
==========

[commit-bot: I haz the power, 2016-08-10 05:55:43]

Patchset 1 (id:??) landed as
https://crrev.com/c34b20c196d32daae5764098479936bc7e10e61f
Cr-Commit-Position: refs/heads/master@{#410973}