Document selectors with pseudo classes generate incorrect styles

[dfreedm]

Selectors of the form
html:not([foo]) *
will generate
html:not([foo]):not(.style-scope).not([style-scope]) *
instead of
html:not([foo]) *:not(.style-scope):not([style-scope])

[dfreedm]

Looks like _transformDocumentSelector incorrectly assumes the lack of a /deep/ or ::content selector means the whole selector can be treated as a simple selector, whereas the problem case is a complex selector. The simple selector transformer attempts to add the scoping classes before the first pseudo class, which in this case is the :not([foo]).

The easiest approach is to remove this code path and treat all document selectors as complex. This will make a selector that more closely matches the expected output:
html:not(.style-scope):not([style-scope]):not([foo]) *:not(.style-scope):not([style-scope])

[dfreedm]

Fixed in #4886