[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 b15c7c6bc417783c3da10628c5d7f41ec75f5872
Author: mjs <mjs at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Sun Nov 2 03:53:59 2003 +0000

            Reviewed by Darin.
    
    	- fixed 3469262 - REGRESSION (110-112): logitech page is very slow to load (due to style sheet updates?)
    
            * khtml/xml/dom_nodeimpl.cpp:
            (NodeImpl::dispatchGenericEvent): Don't do default handling for
    	events that don't bubble. Not 100% sure why this caused the big
    	slowdown but it was a somewhat arbitrary change in my earlier
    	patch.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@5354 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index 4822a0e..46f8d8d 100644
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,15 @@
+2003-11-01  Maciej Stachowiak  <mjs at apple.com>
+
+        Reviewed by Darin.
+
+	- fixed 3469262 - REGRESSION (110-112): logitech page is very slow to load (due to style sheet updates?)
+	
+        * khtml/xml/dom_nodeimpl.cpp:
+        (NodeImpl::dispatchGenericEvent): Don't do default handling for
+	events that don't bubble. Not 100% sure why this caused the big
+	slowdown but it was a somewhat arbitrary change in my earlier
+	patch.
+
 2003-11-01  Darin Adler  <darin at apple.com>
 
         Reviewed by John.
diff --git a/WebCore/khtml/xml/dom_nodeimpl.cpp b/WebCore/khtml/xml/dom_nodeimpl.cpp
index b139ad7..50320a7 100644
--- a/WebCore/khtml/xml/dom_nodeimpl.cpp
+++ b/WebCore/khtml/xml/dom_nodeimpl.cpp
@@ -570,12 +570,15 @@ bool NodeImpl::dispatchGenericEvent( EventImpl *evt, int &/*exceptioncode */)
     evt->setEventPhase(0); // I guess this is correct, the spec does not seem to say
                            // anything about the default event handler phase.
 
-    // now we call all default event handlers (this is not part of DOM - it is internal to khtml)
 
-    it.toLast();
-    for (; it.current() && !evt->propagationStopped() && !evt->defaultPrevented() && !evt->defaultHandled(); --it)
-	it.current()->defaultEventHandler(evt);
+    if (evt->bubbles()) {
+	// now we call all default event handlers (this is not part of DOM - it is internal to khtml)
 
+	it.toLast();
+	for (; it.current() && !evt->propagationStopped() && !evt->defaultPrevented() && !evt->defaultHandled(); --it)
+	    it.current()->defaultEventHandler(evt);
+    }
+    
     // In the case of a mouse click, also send a DOMActivate event, which causes things like form submissions
     // to occur. Note that this only happens for _real_ mouse clicks (for which we get a KHTML_CLICK_EVENT or
     // KHTML_DBLCLICK_EVENT), not the standard DOM "click" event that could be sent from js code.

-- 
WebKit Debian packaging