Jonathan Watt [:jwatt] Reporter
	Description • 11 years ago

The falling snowflakes in the Santa's Workshop demo no longer display. This seems to have regressed sometime between:

2012-03-22-03-12-20
2012-05-16-03-05-15

http://ie.microsoft.com/testdrive/Performance/SantasWorkshop/Default.xhtml

Note than they didn't used to display entirely properly anyway, since they would display black instead of white. Still, that was better than not displaying at all.

	Jonathan Watt [:jwatt] Reporter
	Comment 1 • 11 years ago

This is something to do with IDs. If I move the gradients out of the templates that are cloned (so that they are not cloned, resulting in multiple duplicate gradients with the same ID), then the snowflakes show fine.

	Jonathan Watt [:jwatt] Reporter
	Comment 2 • 11 years ago

So the template markup for the snowflake is in a display:none block, and the template contains the gradient it references. When this template is cloned to create non-display:none snowflakes, the gradient is cloned with it, although that's probably not relevant.

Ultimately the issue is that the gradient references are being resolved to the display:none gradient element in the template, presumably because it comes first in document order, or because it was inserted into the ID table first. At any rate, I think that the reason that we changed from showing black snowflakes to not showing the snowflakes at all is that we used to use black if a gradient reference didn't resolve to a usable gradient frame, but now we paint nothing.

So this is really just us encountering bug 376027.

In this testcase the rect does not display, but if you use the developer tools to remove the display:none <g> then it will.

	Alice0775 White
	Comment 3 • 11 years ago

Pushlog against the rect does not display.
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=c1e2504f48fd&tochange=dc7ccdd73610

This was caused by Bug 746632

	Jonathan Watt [:jwatt] Reporter
	Comment 4 • 11 years ago

Thanks, Alice! That makes sense. Since it was a change of behavior rather than a strict regression, I won't mark this as blocking that bug though.

	Jonathan Watt [:jwatt] Reporter
	Comment 6 • 11 years ago

MS updated their demo.