From f8443a8111ed42cd141a0541046ebc1442fcda6c Mon Sep 17 00:00:00 2001
From: Johannes Hofmann <Johannes.Hofmann@gmx.de>
Date: Sun, 17 Oct 2010 18:38:52 +0200
Subject: fix Style leaks in StyleEngine

---
 src/styleengine.cc | 23 ++++++++++++++++++-----
 src/styleengine.hh |  1 +
 2 files changed, 19 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/styleengine.cc b/src/styleengine.cc
index b5553973..e725649d 100644
--- a/src/styleengine.cc
+++ b/src/styleengine.cc
@@ -50,6 +50,7 @@ StyleEngine::StyleEngine (dw::core::Layout *layout) {
 
    n->style = Style::create (layout, &style_attrs);
    n->wordStyle = NULL;
+   n->backgroundStyle = NULL;
    n->styleAttrProperties = NULL;
    n->nonCssProperties = NULL;
    n->inheritBackgroundColor = false;
@@ -76,6 +77,7 @@ void StyleEngine::startElement (int element) {
    n->nonCssProperties = NULL;
    n->style = NULL;
    n->wordStyle = NULL;
+   n->backgroundStyle = NULL;
    n->inheritBackgroundColor = false;
 
    DoctreeNode *dn = doctree->push ();
@@ -210,6 +212,8 @@ void StyleEngine::endElement (int element) {
       n->style->unref ();
    if (n->wordStyle)
       n->wordStyle->unref ();
+   if (n->backgroundStyle)
+      n->backgroundStyle->unref ();
 
    doctree->pop ();
    stack->setSize (stack->size () - 1);
@@ -659,13 +663,17 @@ void StyleEngine::computeBorderWidth (int *dest, CssProperty *p,
  * background. This method ensures that backgroundColor is set.
  */
 Style * StyleEngine::backgroundStyle () {
-   StyleAttrs attrs = *style ();
+   if (!stack->getRef (stack->size () - 1)->backgroundStyle) {
+      StyleAttrs attrs = *style ();
 
-   for (int i = stack->size () - 1; i >= 0 && ! attrs.backgroundColor; i--)
-      attrs.backgroundColor = stack->getRef (i)->style->backgroundColor;
+      for (int i = stack->size () - 1; i >= 0 && ! attrs.backgroundColor; i--)
+         attrs.backgroundColor = stack->getRef (i)->style->backgroundColor;
 
-   assert (attrs.backgroundColor);
-   return Style::create (layout, &attrs);
+      assert (attrs.backgroundColor);
+      stack->getRef (stack->size () - 1)->backgroundStyle =
+         Style::create (layout, &attrs);
+   }
+   return stack->getRef (stack->size () - 1)->backgroundStyle;
 }
 
 /**
@@ -734,6 +742,11 @@ void StyleEngine::restyle () {
          n->style->unref ();
          n->style = NULL;
       }
+      if (n->wordStyle) {
+         n->wordStyle->unref ();
+         n->wordStyle = NULL;
+      }
+
       style0 (i);
    }
 }
diff --git a/src/styleengine.hh b/src/styleengine.hh
index 8c119e26..841d5c06 100644
--- a/src/styleengine.hh
+++ b/src/styleengine.hh
@@ -24,6 +24,7 @@ class StyleEngine {
          CssPropertyList *nonCssProperties;
          dw::core::style::Style *style;
          dw::core::style::Style *wordStyle;
+         dw::core::style::Style *backgroundStyle;
          bool inheritBackgroundColor;
          DoctreeNode *doctreeNode;
       };
-- 
cgit v1.2.3