Optimize SameDomainOrHost via a fast-fail path

Optimize SameDomainOrHost via a fast-fail path

BUG=678351

[ncarter (slow), 2017-01-06 00:05:55]

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

[commit-bot: I haz the power, 2017-01-06 00:06:32]

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

[ncarter (slow), 2017-01-06 00:07:30]

Description was changed from

==========
Micro-optimize SameDomainOrHost.

BUG=
==========

to

==========
Optimize SameDomainOrHost via a fast-fail path

BUG=678351
==========

[Ryan Sleevi, 2017-01-06 00:38:15]

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

[Ryan Sleevi, 2017-01-06 00:38:16]

https://codereview.chromium.org/2612313003/diff/1/net/base/registry_controlle...
File net/base/registry_controlled_domains/registry_controlled_domain.cc (right):

https://codereview.chromium.org/2612313003/diff/1/net/base/registry_controlle...
net/base/registry_controlled_domains/registry_controlled_domain.cc:336: if
(!base::EndsWith(host2, domain1, base::CompareCase::SENSITIVE))
Unclear if you're doing sensitive as an optimization (to force a memcmp vs a
normalization), but the comparison here would be case-insensitive.

However, both |host1| and |host2| should have already been normalized
(punycoded, lowercased, dot stripped) in order to get the domain, so you should
be fine.

[commit-bot: I haz the power, 2017-01-06 01:09:50]

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

[commit-bot: I haz the power, 2017-01-06 01:09:51]

Dry run: This issue passed the CQ dry run.