See upcoming testcase. This testcase gives an assertion for me, when loading the page: ###!!! ASSERTION: Not a URL value!: 'eCSSUnit_URL == mUnit', file c:/mozilla/moz illa/layout/style/nsCSSValue.cpp, line 353
Created attachment 203624 [details] [diff] [review] Fix
12 years ago
How does ParseCursor create an array value with a 0th item being something that's not a URL?
It doesn't, but the next time we hit this code (say the next time we're resolving style for this node) the unit is eCSSUnit_Image, since we've already started the load. In that case we don't really want to StartImageLoad again, which is what we're doing as the code stands.
Comment on attachment 203624 [details] [diff] [review] Fix OK, r+sr=dbaron if you add the comment "don't try to restart loads we've already started".
Fixed, with that comment added.