Created attachment 3675 [details] patch to fix the leaks I spotted
Comment on attachment 3675 [details] patch to fix the leaks I spotted I have no tests, but still probably should get this reviewed.
Comment on attachment 3675 [details] patch to fix the leaks I spotted Perhaps it would be better to store the CSSValueImpls in a SharedPtr instead of doing the ref()/deref() pair manually.
Created attachment 3710 [details] patch revised to use SharedPtr as per Maciej's suggestion
Comment on attachment 3710 [details] patch revised to use SharedPtr as per Maciej's suggestion r=me