Hi, > That's not a merge error. That code is now in AllCSSValuesImpl and is not > needed in SimpleCSSValuesImpl. Please follow the commits.
Consensus was to leave it in SimpleCSSValuesImpl why did you change it without discussion? We have a live application that is using this. AllCSSValuesImpl is not performant and is unlikely to be used in a real application so I don’t see much point in moving it there. Thanks, Justin