Title: [88098] trunk/Source/WebCore
- Revision
- 88098
- Author
- tha...@chromium.org
- Date
- 2011-06-04 02:02:50 -0700 (Sat, 04 Jun 2011)
Log Message
2011-06-04 Nico Weber <tha...@chromium.org>
Reviewed by James Robinson.
[chromium] Make LayerPainterChromium destructor virtual
https://bugs.webkit.org/show_bug.cgi?id=62056
LayerTextureUpdaterCanvas has an OwnPtr<LayerPainterChromium>, which
means that without this patch, the destructor of classes implementing
LayerPainterChromium wasn't called correctly in that case.
* platform/graphics/chromium/LayerPainterChromium.h:
(WebCore::LayerPainterChromium::~LayerPainterChromium):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (88097 => 88098)
--- trunk/Source/WebCore/ChangeLog 2011-06-04 08:08:16 UTC (rev 88097)
+++ trunk/Source/WebCore/ChangeLog 2011-06-04 09:02:50 UTC (rev 88098)
@@ -1,3 +1,17 @@
+2011-06-04 Nico Weber <tha...@chromium.org>
+
+ Reviewed by James Robinson.
+
+ [chromium] Make LayerPainterChromium destructor virtual
+ https://bugs.webkit.org/show_bug.cgi?id=62056
+
+ LayerTextureUpdaterCanvas has an OwnPtr<LayerPainterChromium>, which
+ means that without this patch, the destructor of classes implementing
+ LayerPainterChromium wasn't called correctly in that case.
+
+ * platform/graphics/chromium/LayerPainterChromium.h:
+ (WebCore::LayerPainterChromium::~LayerPainterChromium):
+
2011-06-04 Emil A Eklund <e...@chromium.org>
Reviewed by Andreas Kling.
Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerPainterChromium.h (88097 => 88098)
--- trunk/Source/WebCore/platform/graphics/chromium/LayerPainterChromium.h 2011-06-04 08:08:16 UTC (rev 88097)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerPainterChromium.h 2011-06-04 09:02:50 UTC (rev 88098)
@@ -36,6 +36,7 @@
class LayerPainterChromium {
public:
+ virtual ~LayerPainterChromium() { }
virtual void paint(GraphicsContext&, const IntRect& contentRect) = 0;
};
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes