Corrected serialization of ::cue() functions.

Corrected serialization of ::cue() functions.

The selectorList argument of ::cue() functions were just dropped.
Serialization of that list is now done.

The explicit universal tag "*" is not correct per CSSOM spec, but I didn't
address that in this CL. Besides, that also affects a bunch of the custom
pseudo elements. Nothing changed here.

Also removed a bunch of unnecessary CSSSelector:: prefixing.

R=timloh@chromium.org,andersr@opera.com
BUG=428696
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184767

[andersr, 2014-10-31 10:11:29]

lgtm

https://codereview.chromium.org/689933002/diff/1/LayoutTests/fast/css/cue-ser...
File LayoutTests/fast/css/cue-serialize.html (right):

https://codereview.chromium.org/689933002/diff/1/LayoutTests/fast/css/cue-ser...
LayoutTests/fast/css/cue-serialize.html:14: shouldBe("rules[1].selectorText",
"'*::cue(c)'");
Perhaps add a test with multiple sub-selectors?

[rune, 2014-10-31 12:45:59]

On 2014/10/31 at 10:11:29, andersr wrote:
> lgtm
> 
>
https://codereview.chromium.org/689933002/diff/1/LayoutTests/fast/css/cue-ser...
> File LayoutTests/fast/css/cue-serialize.html (right):
> 
>
https://codereview.chromium.org/689933002/diff/1/LayoutTests/fast/css/cue-ser...
> LayoutTests/fast/css/cue-serialize.html:14: shouldBe("rules[1].selectorText",
"'*::cue(c)'");
> Perhaps add a test with multiple sub-selectors?

Done.

[rune, 2014-10-31 12:46:16]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-31 12:47:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/689933002/20001

[commit-bot: I haz the power, 2014-10-31 12:47:28]

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

[commit-bot: I haz the power, 2014-10-31 12:47:29]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[rune, 2014-10-31 13:03:15]

On 2014/10/31 at 12:47:29, commit-bot wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer or
> a provisional committer, _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

Bummer.

[Timothy Loh, 2014-10-31 22:58:54]

On 2014/10/31 13:03:15, rune wrote:
> On 2014/10/31 at 12:47:29, commit-bot wrote:
> > No LGTM from a valid reviewer yet. Only full committers are accepted.
> > Even if an LGTM may have been provided, it was from a non-committer or
> > a provisional committer, _not_ a full super star committer.
> > See http://www.chromium.org/getting-involved/become-a-committer
> > Note that this has nothing to do with OWNERS files.
> 
> Bummer.

lgtm

[rune, 2014-11-01 05:44:48]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-11-01 05:45:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/689933002/20001

[commit-bot: I haz the power, 2014-11-01 06:48:27]

Committed patchset #2 (id:20001) as 184767