Fix extending inner of wrapped selectors (unwrap)

[mgreter]

Fixes #2031

sample

:not(.asd, .qwe) {
  content: list;
}
:not(.foo) {
  content: test;
}
.bar, .baz {
  @extend .foo;
}
test {
  content: selector-extend(":not():not(.foo)", ".foo", ".bar");
}

now yields:

:not(.asd):not(.qwe) {
  content: list; }
:not(.foo):not(.bar):not(.baz) {
  content: test; }
test {
  content: :not:not(.foo):not(.bar); }

ruby sass:

:not(.asd, .qwe) {
  content: list; }
:not(.foo):not(.bar):not(.baz) {
  content: test; }
test {
  content: :not:not(.foo):not(.bar); }

So there is one breaking change from :not(.asd, .qwe) to :not(.asd):not(.qwe). The first is invalid css so I do not mind shipping this in 3.3.6. I think ruby sass should do it either the same way or produce an error as it does in other situations when it tries to print invalid css syntax.

[coveralls]

Coverage increased (+0.009%) to 78.807% when pulling 6172695 on mgreter:bugfix/issue_2031 into 7889f8c on sass:master.

[nicovalencia]

Thanks! 👍

[bschaf]

👍 Thank you!

[coveralls]

Coverage decreased (-0.006%) to 79.236% when pulling 4796e00 on mgreter:bugfix/issue_2031 into 57fbfb9 on sass:master.

[xzyfer]

@mgreter any objections to releasing 3.3.6 after this is merged?

[mgreter]

@xzyfer yes one, #2001 should also be ready, there was only one issue with MSVC that I already fixed locally. Waiting for CI to catch up.

[xzyfer]

@mgreter once these land I'll make sure major projects like foundation, and sassdash still compile as expected.

[mgreter]

@xzyfer just to make sure, you are aware of the "breaking" change here? sass/sass#2036. Reminds me I need to split the spec test I created.

[xzyfer]

I wasn't aware, thanks for the link.