Bug 610954: Use first continuation frame in new code in nsSVGIntegrationUtils::GetInvalidAreaForChangedSource, to match old code. r=longsonr a=bsmedberg

2010-11-10 13:18:11 -08:00 · 2010-11-10 13:18:11 -08:00 · 0d3e01be28
--- a/layout/svg/base/src/nsSVGIntegrationUtils.cpp
+++ b/layout/svg/base/src/nsSVGIntegrationUtils.cpp
@ -148,7 +148,7 @@ nsSVGIntegrationUtils::GetInvalidAreaForChangedSource(nsIFrame* aFrame,
  if (!effectProperties.mFilter)
    return aInvalidRect;

-  nsSVGFilterProperty *prop = nsSVGEffects::GetFilterProperty(aFrame);
+  nsSVGFilterProperty *prop = nsSVGEffects::GetFilterProperty(firstFrame);
  if (!prop || !prop->IsInObserverList()) {
    return aInvalidRect;
  }
--- a/layout/svg/crashtests/610954-1.html
+++ b/layout/svg/crashtests/610954-1.html
@ -0,0 +1 @@
+<!DOCTYPE html><html><body dir=rtl onload="document.getElementById('g').style.filter = 'url(#filter1)';"><span id="g">&#x200E;---</span></body></html>
--- a/layout/svg/crashtests/crashtests.list
+++ b/layout/svg/crashtests/crashtests.list
@ -96,3 +96,4 @@ load 587336-1.html
 load 590291-1.svg
 load 601999-1.html
 load 605626-1.svg
+load 610954-1.html
				`@ -0,0 +1 @@`
				`<!DOCTYPE html><html><body dir=rtl onload="document.getElementById('g').style.filter = 'url(#filter1)';"><span id="g">‎---</span></body></html>`