[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

dglazkov at chromium.org dglazkov at chromium.org
Wed Dec 22 12:13:44 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 920ff98c064db4ee20c61b5aa5915088b146a190
Author: dglazkov at chromium.org <dglazkov at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Aug 17 16:15:17 2010 +0000

    2010-08-17  Dimitri Glazkov  <dglazkov at chromium.org>
    
            Reviewed by Eric Seidel.
    
            Eliminate extra traversal of the tree when determining ability to lazy-attach a Node.
            https://bugs.webkit.org/show_bug.cgi?id=43749
    
            Neither SVG nor inputs/counters shadow DOM are traveling down this path (they home-bake their attachment),
            so the check for shadow tree existence isn't needed.
    
            Gives a small, but consisten win in Dromaeo's DOM Modification test.
    
            * dom/Node.cpp: Removed canLazyAttach declaration.
            * dom/Node.h:
            (WebCore::Node::canLazyAttach): Changed to always return true.
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@65508 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index b7ec1a3..4cdff45 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,19 @@
+2010-08-17  Dimitri Glazkov  <dglazkov at chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        Eliminate extra traversal of the tree when determining ability to lazy-attach a Node.
+        https://bugs.webkit.org/show_bug.cgi?id=43749
+
+        Neither SVG nor inputs/counters shadow DOM are traveling down this path (they home-bake their attachment),
+        so the check for shadow tree existence isn't needed.
+
+        Gives a small, but consisten win in Dromaeo's DOM Modification test.
+
+        * dom/Node.cpp: Removed canLazyAttach declaration.
+        * dom/Node.h:
+        (WebCore::Node::canLazyAttach): Changed to always return true.
+
 2010-08-17  Pavel Feldman  <pfeldman at chromium.org>
 
         Reviewed by Yury Semikhatsky.
diff --git a/WebCore/dom/Node.cpp b/WebCore/dom/Node.cpp
index e0f04de..0bc8d9e 100644
--- a/WebCore/dom/Node.cpp
+++ b/WebCore/dom/Node.cpp
@@ -779,11 +779,6 @@ void Node::lazyAttach()
     }
 }
 
-bool Node::canLazyAttach()
-{
-    return shadowAncestorNode() == this;
-}
-    
 void Node::setFocus(bool b)
 { 
     if (b || hasRareData())
diff --git a/WebCore/dom/Node.h b/WebCore/dom/Node.h
index 3f3184d..1e858ba 100644
--- a/WebCore/dom/Node.h
+++ b/WebCore/dom/Node.h
@@ -317,7 +317,7 @@ public:
     void clearIsLink() { clearFlag(IsLinkFlag); }
 
     void lazyAttach();
-    virtual bool canLazyAttach();
+    virtual bool canLazyAttach() { return shadowAncestorNode() == this; }
 
     virtual void setFocus(bool b = true);
     virtual void setActive(bool f = true, bool /*pause*/ = false) { setFlag(f, IsActiveFlag); }

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list