fix: corrects behavior of style removal

[nicholasrice]

Description

I noticed when using Controller.removeStyles(ElementStyles) that the method was removing all styles except the styles I was trying to remove. This corrects that behavior.

The removal method invokes the Array.prototype.filter method, where a callback returning true keeps the item and returning false removes the item. AdoptedStyleSheetsStyles.removeStylesFrom was returning false for all stylesheets that were not associated to the AdoptedStyleSheetsStyles instance, and true for all that were. This is the opposite behavior that we want.

Motivation & context

Issue type checklist

• Chore: A change that does not impact distributed packages.
• Bug fix: A change that fixes an issue, link to the issue above.
• New feature: A change that adds functionality.

Is this a breaking change?

• This change causes current functionality to break.

Adding or modifying component(s) in @microsoft/fast-components checklist

• I have added a new component
• I have modified an existing component
• I have updated the definition file
• I have updated the configuration file

Process & policy checklist

• I have added tests for my changes.
• I have tested my changes.
• I have updated the project documentation to reflect my changes.
• I have read the CONTRIBUTING documentation and followed the standards for this project.

[EisenbergEffect]

LOL Definitely need to add more unit tests :)

[nicholasrice]

Seeing if I can add one for this real quickly ;)

[EisenbergEffect]

@nicholasrice If you could add a single unit test for this case, I can come in tomorrow and fill in the rest of the cases for the Controller. Would be good to capture this as part of the bug fix though.