Add a ContentSettingsPattern test for "*" vs. a site-scoped pattern.

Add a ContentSettingsPattern test for "*" vs. a site-scoped pattern.

We have tests for "*".Compare(Wildcard()) == IDENTITY and for
Wildcard().Compare("[*.]google.com") == SUCCESSOR. Assuming the comparison
is transitive, it should hold that "*".Compare("[*.]google.com") == SUCCESSOR.

This CL adds an explicit test case, so that we know that the comparison is,
in this case, in fact transitive.

BUG=602470
Committed: https://crrev.com/69745e4858c876f6c2329266c28f01a35a41ea6d
Cr-Commit-Position: refs/heads/master@{#386936}

[msramek, 2016-04-12 09:19:42]

msramek@chromium.org changed reviewers:
+ dmurph@chromium.org

[msramek, 2016-04-12 09:19:43]

Hi Dan,

please have a look! I cannot reproduce your bug, and I'm adding a few tests to
prove it :)

Thanks,
Martin

[dmurph, 2016-04-12 18:13:15]

On 2016/04/12 at 09:19:43, msramek wrote:
> Hi Dan,
> 
> please have a look! I cannot reproduce your bug, and I'm adding a few tests to
prove it :)
> 
> Thanks,
> Martin

Thanks! lgtm.

I made a mistake with my debugging, sorry for the extra work. But it did reveal
a question about how I'm matching contentsettingspatterns, which I left for you
on the other CL. Thanks for all your help!

[msramek, 2016-04-13 08:13:50]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2016-04-13 08:14:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1880043002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1880043002/1

[commit-bot: I haz the power, 2016-04-13 08:19:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-13 08:20:22]

Description was changed from

==========
Add a ContentSettingsPattern test for "*" vs. a site-scoped pattern.

We have tests for "*".Compare(Wildcard()) == IDENTITY and for
Wildcard().Compare("[*.]google.com") == SUCCESSOR. Assuming the comparison
is transitive, it should hold that "*".Compare("[*.]google.com") == SUCCESSOR.

This CL adds an explicit test case, so that we know that the comparison is,
in this case, in fact transitive.

BUG=602470
==========

to

==========
Add a ContentSettingsPattern test for "*" vs. a site-scoped pattern.

We have tests for "*".Compare(Wildcard()) == IDENTITY and for
Wildcard().Compare("[*.]google.com") == SUCCESSOR. Assuming the comparison
is transitive, it should hold that "*".Compare("[*.]google.com") == SUCCESSOR.

This CL adds an explicit test case, so that we know that the comparison is,
in this case, in fact transitive.

BUG=602470

Committed: https://crrev.com/69745e4858c876f6c2329266c28f01a35a41ea6d
Cr-Commit-Position: refs/heads/master@{#386936}
==========

[commit-bot: I haz the power, 2016-04-13 08:20:23]

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