387835 - Iframe not visible anymore when wrapping svg content around it during page load

Status: VERIFIED FIXED

(Core :: SVG, defect)

Description

[Martijn Wargers (dead)]

Attachment: testcase

See testcase, you should see a green iframe.
That doesn't happen anymore. Only if I remove the svg element and reinsert it again, the iframe gets displayed.

This regressed between 2007-05-21 and 2007-05-22:
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-05-21+04&maxdate=2007-05-22+10&cvsroot=%2Fcvsroot
I guess this is somehow a regression from bug 380590.

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

I don't think that this can be a regression from bug 380590. I put a breakpoint in the newly-added function, and it doesn't get hit. On the other hand, when I load this testcase, I do see:

###!!! ASSERTION: XXX. We shouldn't get here. Viewbox width/height is set to 0. Need to disable display of element as per specs.: 'Error', file /home/mrbkap/src/3mozbuild/mozilla/content/svg/content/src/nsSVGSVGElement.cpp, line 1258

but the content model looks right (as far as I can tell, anyway). Perhaps this is an SVG reflow bug?

Comment 3

[Martijn Wargers (dead)]

Yeah, probably an SVG reflow bug, I guess.
The thing is, I am relying on this to work in a script of mine. I found a workaround by removing and adding the svg element to the document again, but that suddenly seems to have stopped working also in the latest trunk build for me.

Comment 4

[Jonathan Watt [:jwatt]]

This is WFM with my patch for bug 294086.

Comment 5

[Martijn Wargers (dead)]

Great, thanks!
I really would like to see this fixed.

Comment 6

[Jonathan Watt [:jwatt]]

This bug was just fixed by the fix for bug 294086. I'll add a reftest for this shortly.

Comment 7

[Martijn Wargers (dead)]

Yes, thanks for fixing.
Verified fixed, using:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b2pre) Gecko/2007111905 Minefield/3.0b2pre