Changeset 84763 in webkit

Timestamp:

Apr 24, 2011 4:09:39 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-04-24 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r84759.
​http://trac.webkit.org/changeset/84759
​https://bugs.webkit.org/show_bug.cgi?id=59306

Caused gc-shadow.html to start crashing in V8. (Requested by
dglazkov|away on #webkit).

• fast/dom/shadow/append-child-text-expected.txt: Removed.
• fast/dom/shadow/append-child-text.html: Removed.

2011-04-24 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r84759.
​http://trac.webkit.org/changeset/84759
​https://bugs.webkit.org/show_bug.cgi?id=59306

Caused gc-shadow.html to start crashing in V8. (Requested by
dglazkov|away on #webkit).

• dom/Node.cpp: (WebCore::NodeRendererFactory::createRendererAndStyle): (WebCore::Node::styleForRenderer):

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/fast/dom/shadow/append-child-text-expected.txt (deleted)
• LayoutTests/fast/dom/shadow/append-child-text.html (deleted)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/dom/Node.cpp (modified) (4 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2011-04-24  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r84759.
+        http://trac.webkit.org/changeset/84759
+        https://bugs.webkit.org/show_bug.cgi?id=59306
+
+        Caused gc-shadow.html to start crashing in V8. (Requested by
+        dglazkov|away on #webkit).
+
+        * fast/dom/shadow/append-child-text-expected.txt: Removed.
+        * fast/dom/shadow/append-child-text.html: Removed.
+
 2011-04-24  Maciej Stachowiak  <mjs@apple.com>
 

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-24  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r84759.
+        http://trac.webkit.org/changeset/84759
+        https://bugs.webkit.org/show_bug.cgi?id=59306
+
+        Caused gc-shadow.html to start crashing in V8. (Requested by
+        dglazkov|away on #webkit).
+
+        * dom/Node.cpp:
+        (WebCore::NodeRendererFactory::createRendererAndStyle):
+        (WebCore::Node::styleForRenderer):
+
 2011-04-24  Maciej Stachowiak  <mjs@apple.com>
 

trunk/Source/WebCore/dom/Node.cpp

@@ -1484 +1484 @@
     ContainerNode* parentNodeForRenderingAndStyle() const { return m_parentNodeForRenderingAndStyle; }
     void createRendererIfNeeded();
-    PassRefPtr<RenderStyle> styleForRenderer();
 
 private:
@@ -1498 +1497 @@
     ShadowRoot* m_visualParentShadowRoot;
 };
-
-PassRefPtr<RenderStyle> NodeRendererFactory::styleForRenderer()
-{
-    if (m_node->isElementNode()) {
-        bool allowSharing = true;
-#if ENABLE(XHTMLMP)
-        // noscript needs the display property protected - it's a special case
-        allowSharing = m_node->localName() != HTMLNames::noscriptTag.localName();
-#endif
-        return m_node->document()->styleSelector()->styleForElement(static_cast<Element*>(m_node), 0, allowSharing);
-    }
-    ContainerNode* parentNodeForStyle = parentNodeForRenderingAndStyle();
-    return parentNodeForStyle && parentNodeForStyle->renderer() ? parentNodeForStyle->renderer()->style() : 0;
-}
 
 ContainerNode* NodeRendererFactory::findVisualParent()
@@ -1572 +1557 @@
         return 0;
 
-    RefPtr<RenderStyle> style = styleForRenderer();
+    RefPtr<RenderStyle> style = m_node->styleForRenderer();
     if (!m_node->rendererIsNeeded(style.get()))
         return 0;
@@ -1637 +1622 @@
 PassRefPtr<RenderStyle> Node::styleForRenderer()
 {
-    return NodeRendererFactory(this).styleForRenderer();
+    if (isElementNode()) {
+        bool allowSharing = true;
+#if ENABLE(XHTMLMP)
+        // noscript needs the display property protected - it's a special case
+        allowSharing = localName() != HTMLNames::noscriptTag.localName();
+#endif
+        return document()->styleSelector()->styleForElement(static_cast<Element*>(this), 0, allowSharing);
+    }
+    return parentNode() && parentNode()->renderer() ? parentNode()->renderer()->style() : 0;
 }