WTF: Add comments and tests for Unicode aware case-insensitive string operations.

WTF: Add comments and tests for Unicode aware case-insensitive string operations.

Also, this CL removes unused |equalIgnoringCase(const StringBuilder&, const
StringType&)|.
This CL has no behavior changes.

BUG=627682
Committed: https://crrev.com/9a27ccab310b93cb3a70710833a51eb288ad475f
Cr-Commit-Position: refs/heads/master@{#432962}

[tkent, 2016-11-17 05:07:21]

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

[commit-bot: I haz the power, 2016-11-17 05:08:05]

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

[tkent, 2016-11-17 06:45:48]

tkent@chromium.org changed reviewers:
+ yutak@chromium.org

[tkent, 2016-11-17 06:45:49]

yutak@, would you review this please?

[tkent, 2016-11-17 06:46:51]

Description was changed from

==========
WTF: Add comments and tests for Unicode aware case-insensitive string
operations.

Also, this CL removes unused |equalIgnringCase(const StringBuilder&, const
StringType&)|.
This CL has no behavior changes.

BUG=627682
==========

to

==========
WTF: Add comments and tests for Unicode aware case-insensitive string
operations.

Also, this CL removes unused |equalIgnoringCase(const StringBuilder&, const
StringType&)|.
This CL has no behavior changes.

BUG=627682
==========

[commit-bot: I haz the power, 2016-11-17 07:48:58]

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

[commit-bot: I haz the power, 2016-11-17 07:48:58]

Dry run: This issue passed the CQ dry run.

[Yuta Kitamura, 2016-11-17 11:24:20]

LGTM

I even might append "UnicodeCase" or something like that
to each of those names so these versions are harder to use.
(I don't feel strongly on this)

[tkent, 2016-11-17 21:09:05]

On 2016/11/17 at 11:24:20, yutak wrote:
> LGTM
> 
> I even might append "UnicodeCase" or something like that
> to each of those names so these versions are harder to use.
> (I don't feel strongly on this)

Yeah, I agree on that.  I'll do it separately because renaming patches will be
huge.

[tkent, 2016-11-17 21:09:13]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-11-17 21:10:52]

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-11-17 21:29:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 21:32:49]

Description was changed from

==========
WTF: Add comments and tests for Unicode aware case-insensitive string
operations.

Also, this CL removes unused |equalIgnoringCase(const StringBuilder&, const
StringType&)|.
This CL has no behavior changes.

BUG=627682
==========

to

==========
WTF: Add comments and tests for Unicode aware case-insensitive string
operations.

Also, this CL removes unused |equalIgnoringCase(const StringBuilder&, const
StringType&)|.
This CL has no behavior changes.

BUG=627682

Committed: https://crrev.com/9a27ccab310b93cb3a70710833a51eb288ad475f
Cr-Commit-Position: refs/heads/master@{#432962}
==========

[commit-bot: I haz the power, 2016-11-17 21:32:50]

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