[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.15-1-40151-g37bb677

[mjs]

The following commit has been merged in the debian/unstable branch:
commit c4b4828d94f6f895e9eba68eaa781847bb4fad77
Author: mjs <mjs at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Wed Nov 27 22:12:23 2002 +0000

    	- avoid doing any work when setting the document's link colors if
    	there is no change. This avoids superfluous style updates and
    	results in a 20% improvment on the JavaScript iBench.
    
            * khtml/ecma/kjs_html.cpp:
            (KJS::HTMLDocument::putValue):
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2896 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog-2002-12-03 b/WebCore/ChangeLog-2002-12-03
index 956ec19..d43214b 100644
--- a/WebCore/ChangeLog-2002-12-03
+++ b/WebCore/ChangeLog-2002-12-03
@@ -1,3 +1,12 @@
+2002-11-27  Maciej Stachowiak  <mjs at apple.com>
+
+	- avoid doing any work when setting the document's link colors if
+	there is no change. This avoids superfluous style updates and
+	results in a 20% improvment on the JavaScript iBench.
+	
+        * khtml/ecma/kjs_html.cpp:
+        (KJS::HTMLDocument::putValue):
+
 2002-11-27  David Hyatt  <hyatt at apple.com>
 
 	Fix <br clear=""> and <br clear> to just behave like <br>
diff --git a/WebCore/ChangeLog-2003-10-25 b/WebCore/ChangeLog-2003-10-25
index 956ec19..d43214b 100644
--- a/WebCore/ChangeLog-2003-10-25
+++ b/WebCore/ChangeLog-2003-10-25
@@ -1,3 +1,12 @@
+2002-11-27  Maciej Stachowiak  <mjs at apple.com>
+
+	- avoid doing any work when setting the document's link colors if
+	there is no change. This avoids superfluous style updates and
+	results in a 20% improvment on the JavaScript iBench.
+	
+        * khtml/ecma/kjs_html.cpp:
+        (KJS::HTMLDocument::putValue):
+
 2002-11-27  David Hyatt  <hyatt at apple.com>
 
 	Fix <br clear=""> and <br clear> to just behave like <br>
diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index 956ec19..d43214b 100644
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,12 @@
+2002-11-27  Maciej Stachowiak  <mjs at apple.com>
+
+	- avoid doing any work when setting the document's link colors if
+	there is no change. This avoids superfluous style updates and
+	results in a 20% improvment on the JavaScript iBench.
+	
+        * khtml/ecma/kjs_html.cpp:
+        (KJS::HTMLDocument::putValue):
+
 2002-11-27  David Hyatt  <hyatt at apple.com>
 
 	Fix <br clear=""> and <br clear> to just behave like <br>
diff --git a/WebCore/khtml/ecma/kjs_html.cpp b/WebCore/khtml/ecma/kjs_html.cpp
index f321e9c..89565dc 100644
--- a/WebCore/khtml/ecma/kjs_html.cpp
+++ b/WebCore/khtml/ecma/kjs_html.cpp
@@ -336,13 +336,37 @@ void KJS::HTMLDocument::putValue(ExecState *exec, int token, const Value& value,
     body.setText(value.toString(exec).string());
     break;
   case AlinkColor:
-    body.setALink(value.toString(exec).string());
+    // this check is a bit silly, but some benchmarks like to set the
+    // document's link colors over and over to the same value and we
+    // don't want to incur a style update each time.
+    {
+      DOM::DOMString newColor = value.toString(exec).string();
+      if (body.aLink() != newColor) {
+	body.setALink(newColor);
+      }
+    }
     break;
   case LinkColor:
-    body.setLink(value.toString(exec).string());
+    // this check is a bit silly, but some benchmarks like to set the
+    // document's link colors over and over to the same value and we
+    // don't want to incur a style update each time.
+    {
+      DOM::DOMString newColor = value.toString(exec).string();
+      if (body.link() != newColor) {
+	body.setLink(newColor);
+      }
+    }
     break;
   case VlinkColor:
-    body.setVLink(value.toString(exec).string());
+    // this check is a bit silly, but some benchmarks like to set the
+    // document's link colors over and over to the same value and we
+    // don't want to incur a style update each time.
+    {
+      DOM::DOMString newColor = value.toString(exec).string();
+      if (body.vLink() != newColor) {
+	body.setVLink(newColor);
+      }
+    }
     break;
   case Dir:
     body.setDir(value.toString(exec).string());

-- 
WebKit Debian packaging