Update equalIgnoreCase to equalIgnoreASCIICase in Source/web

Update equalIgnoreCase to equalIgnoreASCIICase in Source/web

This Cl updates the usage of equalIgnoreCase in Source/web to use
equalsIgnoreASCIICase method instead. This better matches what the spec
requires.

BUG=627682
Committed: https://crrev.com/2d841fa9cb4c1cd9eb3dfb9329cc7b8119ad94ee
Cr-Commit-Position: refs/heads/master@{#411549}

[dsinclair, 2016-08-10 17:28:45]

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

[commit-bot: I haz the power, 2016-08-10 17:29:59]

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

[dsinclair, 2016-08-10 18:59:00]

dsinclair@chromium.org changed reviewers:
+ esprehn@chromium.org, tkent@chromium.org

[dsinclair, 2016-08-10 18:59:01]

PTAL.

These seemed like they were all candidates for conversion. Please let me know if
any of them should have remained.

[esprehn, 2016-08-10 19:00:41]

I'm happy with this if tkent is. :)

lgtm

[dmazzoni, 2016-08-10 19:18:11]

dmazzoni@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2016-08-10 19:18:12]

lgtm for WebAXObject.cpp

I'm assuming you're planning to do the same for the rest of Source?

[dsinclair, 2016-08-10 19:22:19]

On 2016/08/10 19:18:12, dmazzoni wrote:
> lgtm for WebAXObject.cpp
> 
> I'm assuming you're planning to do the same for the rest of Source?


I'm planning on poking away at it as I have time, yes.

[commit-bot: I haz the power, 2016-08-10 20:09:43]

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

[commit-bot: I haz the power, 2016-08-10 20:09:44]

Dry run: This issue passed the CQ dry run.

[dsinclair, 2016-08-11 13:43:55]

tkent@ ping.

[tkent, 2016-08-11 22:53:38]

https://codereview.chromium.org/2236573003/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/WebAXObject.cpp (right):

https://codereview.chromium.org/2236573003/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/WebAXObject.cpp:241: return
equalIgnoringASCIICase(m_private->getAttribute(HTMLNames::aria_readonlyAttr),
"true");
Are ARIA attribute values ascii-case-insensitive?

[esprehn, 2016-08-11 23:10:56]

https://codereview.chromium.org/2236573003/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/WebAXObject.cpp (right):

https://codereview.chromium.org/2236573003/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/WebAXObject.cpp:241: return
equalIgnoringASCIICase(m_private->getAttribute(HTMLNames::aria_readonlyAttr),
"true");
On 2016/08/11 at 22:53:38, tkent wrote:
> Are ARIA attribute values ascii-case-insensitive?

Yeah all attributes value matches in the DOM are.

[tkent, 2016-08-11 23:43:50]

lgtm

[dsinclair, 2016-08-12 02:38:30]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2016-08-12 02:38:42]

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-12 04:11:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-12 04:14:20]

Description was changed from

==========
Update equalIgnoreCase to equalIgnoreASCIICase in Source/web

This Cl updates the usage of equalIgnoreCase in Source/web to use
equalsIgnoreASCIICase method instead. This better matches what the spec
requires.

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

to

==========
Update equalIgnoreCase to equalIgnoreASCIICase in Source/web

This Cl updates the usage of equalIgnoreCase in Source/web to use
equalsIgnoreASCIICase method instead. This better matches what the spec
requires.

BUG=627682

Committed: https://crrev.com/2d841fa9cb4c1cd9eb3dfb9329cc7b8119ad94ee
Cr-Commit-Position: refs/heads/master@{#411549}
==========

[commit-bot: I haz the power, 2016-08-12 04:14:21]

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