Created attachment 423739 [details] testcase ###!!! ASSERTION: New (SVG > 1.1) SVG viewport establishing element?: 'Error', file /Users/jruderman/mozilla-central/layout/svg/base/src/nsSVGUtils.cpp, line 533
Created attachment 594620 [details] [diff] [review] Don't treat <svg:image> as a viewport establishing element. I think it makes sense just to have <image> not be considered to be a viewport establishing element. It does establish a viewport but for the referenced document, not child content in this document.
Autoland Patchset: Patches: 594620 Branch: mozilla-central => try Destination: http://hg.mozilla.org/try/rev/70fc825a5a84 Try run started, revision 70fc825a5a84. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=70fc825a5a84
Try run for 70fc825a5a84 is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=70fc825a5a84 Results (out of 208 total builds): success: 188 warnings: 20 Builds (or logs if builds failed) available at: http://email@example.com
Comment on attachment 594620 [details] [diff] [review] Don't treat <svg:image> as a viewport establishing element. Yeah, agreed. Can you add an explanatory comment to nsSVGUtils::EstablishesViewport noting this though? (Maybe with a pointer to this bug.)