FEImage::m_document is never cleared. Why not?
authorschenney@chromium.org <schenney@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Nov 2012 13:54:49 +0000 (13:54 +0000)
committerschenney@chromium.org <schenney@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Nov 2012 13:54:49 +0000 (13:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=99243

Reviewed by Dirk Schulze.

Adding a comment to explain why the failure to clear m_document is not a problem.

No new tests because no code change at all.

* svg/graphics/filters/SVGFEImage.h:
(FEImage): Add a comment on the lifetime of m_document.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@133158 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/svg/graphics/filters/SVGFEImage.h

index f0a8644..515d2af 100644 (file)
@@ -1,3 +1,17 @@
+2012-11-01  Stephen Chenney  <schenney@chromium.org>
+
+        FEImage::m_document is never cleared. Why not?
+        https://bugs.webkit.org/show_bug.cgi?id=99243
+
+        Reviewed by Dirk Schulze.
+
+        Adding a comment to explain why the failure to clear m_document is not a problem.
+
+        No new tests because no code change at all.
+
+        * svg/graphics/filters/SVGFEImage.h:
+        (FEImage): Add a comment on the lifetime of m_document.
+
 2012-11-01  Eugene Klyuchnikov  <eustas.bug@gmail.com>
 
         Web Inspector: Timeline: make "addRecord" unambiguous
index 5bad950..87e5837 100644 (file)
@@ -54,6 +54,8 @@ private:
     RenderObject* referencedRenderer() const;
 
     RefPtr<Image> m_image;
+
+    // m_document will never be a dangling reference. See https://bugs.webkit.org/show_bug.cgi?id=99243
     Document* m_document;
     String m_href;
     SVGPreserveAspectRatio m_preserveAspectRatio;