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

Reply via email to