Corrected specificity for :not for default @namespace.

Corrected specificity for :not for default @namespace.

The argument to :not() consists of a universal, type, or a simple
selector. However, we prepend an implicit universal selector with the
default namespace URI to all compound selectors to correctly match only
elements in the default namespace. That is not necessary for compounds
inside pseudo argument lists, but we do in those cases as well.

When calculating the specificity for :not(), we assume its argument
consists of a single simple selectors, but in the case where we have a
default namespace, a namespaced universal selector is prepended. We
only added the specificity for the universal selector in that case.
Instead, walk all sub-selectors of the :not() compound when calculating
the specificity.

R=meade@chromium.org
BUG=706206
Review-Url: https://codereview.chromium.org/2777063007
Cr-Commit-Position: refs/heads/master@{#460355}
Committed: https://chromium.googlesource.com/chromium/src/+/76f8029bef50611fdddf565925b9420bdac05021

[rune, 2017-03-28 23:59:59]

ptal

[rune, 2017-03-29 00:00:06]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-29 00:01:02]

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

[meade_UTC10, 2017-03-29 00:51:21]

lgtm

[commit-bot: I haz the power, 2017-03-29 01:44:15]

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

[commit-bot: I haz the power, 2017-03-29 01:44:16]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rune, 2017-03-29 09:41:44]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2017-03-29 09:42:12]

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, 2017-03-29 11:42:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490780504877420, "parent_rev":
"f266b4e8d109a11ac0e3fae0b4c8500281e7c457", "commit_rev":
"76f8029bef50611fdddf565925b9420bdac05021"}

[commit-bot: I haz the power, 2017-03-29 11:43:38]

Description was changed from

==========
Corrected specificity for :not for default @namespace.

The argument to :not() consists of a universal, type, or a simple
selector. However, we prepend an implicit universal selector with the
default namespace URI to all compound selectors to correctly match only
elements in the default namespace. That is not necessary for compounds
inside pseudo argument lists, but we do in those cases as well.

When calculating the specificity for :not(), we assume its argument
consists of a single simple selectors, but in the case where we have a
default namespace, a namespaced universal selector is prepended. We
only added the specificity for the universal selector in that case.
Instead, walk all sub-selectors of the :not() compound when calculating
the specificity.

R=meade@chromium.org
BUG=706206
==========

to

==========
Corrected specificity for :not for default @namespace.

The argument to :not() consists of a universal, type, or a simple
selector. However, we prepend an implicit universal selector with the
default namespace URI to all compound selectors to correctly match only
elements in the default namespace. That is not necessary for compounds
inside pseudo argument lists, but we do in those cases as well.

When calculating the specificity for :not(), we assume its argument
consists of a single simple selectors, but in the case where we have a
default namespace, a namespaced universal selector is prepended. We
only added the specificity for the universal selector in that case.
Instead, walk all sub-selectors of the :not() compound when calculating
the specificity.

R=meade@chromium.org
BUG=706206

Review-Url: https://codereview.chromium.org/2777063007
Cr-Commit-Position: refs/heads/master@{#460355}
Committed:
https://chromium.googlesource.com/chromium/src/+/76f8029bef50611fdddf565925b9...
==========

[commit-bot: I haz the power, 2017-03-29 11:43:38]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/76f8029bef50611fdddf565925b9...