Wrappers for inline style declarations disappeared on GC. Fix the

issue by adding the style declaration wrappers to the object groups of
the elements they belong to.
Review URL: http://codereview.chromium.org/100204

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15037 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 14 insertions, 1 deletions

diff --git a/webkit/port/bindings/v8/v8_proxy.cpp b/webkit/port/bindings/v8/v8_proxy.cpp
index 07da396..9180dc9 100644
--- a/webkit/port/bindings/v8/v8_proxy.cpp
+++ b/webkit/port/bindings/v8/v8_proxy.cpp
@@ -456,10 +456,23 @@ ACTIVE_DOM_OBJECT_TYPES(MAKE_CASE)
     Vector<v8::Persistent<v8::Value> > group;
     group.reserveCapacity(next_key_index - i);
     for (; i < next_key_index; ++i) {
+      Node* node = grouper[i].second;
       v8::Persistent<v8::Value> wrapper =
-          getDOMNodeMap().get(grouper[i].second);
+          getDOMNodeMap().get(node);
       if (!wrapper.IsEmpty())
         group.append(wrapper);
+      // If the node is styled and there is a wrapper for the inline
+      // style declaration, we need to keep that style declaration
+      // wrapper alive as well, so we add it to the object group.
+      if (node->isStyledElement()) {
+        StyledElement* element = reinterpret_cast<StyledElement*>(node);
+        CSSStyleDeclaration* style = element->inlineStyleDecl();
+        if (style != NULL) {
+          wrapper = getDOMObjectMap().get(style);
+          if (!wrapper.IsEmpty())
+            group.append(wrapper);
+        }
+      }
     }
 
     if (group.size() > 1)