No longer ASSERT for LayerRenderer in VideoLayerChromium destructor
authorhclam@chromium.org <hclam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Oct 2010 22:50:32 +0000 (22:50 +0000)
committerhclam@chromium.org <hclam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Oct 2010 22:50:32 +0000 (22:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=47432

Patch by Victoria Kirst <vrk@google.com> on 2010-10-13
Reviewed by James Robinson.

* platform/graphics/chromium/VideoLayerChromium.cpp:
(WebCore::VideoLayerChromium::cleanupResources):

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

WebCore/ChangeLog
WebCore/platform/graphics/chromium/VideoLayerChromium.cpp

index 7838fb7..bd0088c 100644 (file)
@@ -1,3 +1,13 @@
+2010-10-13  Victoria Kirst  <vrk@google.com>
+
+        Reviewed by James Robinson.
+
+        No longer ASSERT for LayerRenderer in VideoLayerChromium destructor
+        https://bugs.webkit.org/show_bug.cgi?id=47432
+
+        * platform/graphics/chromium/VideoLayerChromium.cpp:
+        (WebCore::VideoLayerChromium::cleanupResources):
+
 2010-10-13  Leandro Pereira  <leandro@profusion.mobi>
 
         [EFL] Unreviewed. Build fix.
index 01b4d6e..46c73a1 100644 (file)
@@ -188,7 +188,9 @@ VideoLayerChromium::~VideoLayerChromium()
 void VideoLayerChromium::cleanupResources()
 {
     releaseCurrentFrame();
-    ASSERT(layerRenderer());
+    if (!layerRenderer())
+        return;
+
     GraphicsContext3D* context = layerRendererContext();
     for (unsigned plane = 0; plane < VideoFrameChromium::maxPlanes; plane++) {
         if (m_textures[plane])