Back out one of the previous changes, it causes a crash. Whoops!
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Apr 2004 08:43:51 +0000 (08:43 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Apr 2004 08:43:51 +0000 (08:43 +0000)
        * kwq/KWQScrollView.mm:
        (QScrollView::addChild):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@6461 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQScrollView.mm

index 45373a914a3cee6b6e7af000836c9673c7e20f69..0c88e87b4ee828bc85b18baf2e669c3e0809d95a 100644 (file)
@@ -1,3 +1,25 @@
+2004-04-23  Maciej Stachowiak  <mjs@apple.com>
+
+        Back out one of the previous changes, it causes a crash. Whoops!
+        
+        * kwq/KWQScrollView.mm:
+        (QScrollView::addChild):
+
+2004-04-23  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Dave.
+
+       These two tiny changes (made w/ Dave's help) should get us about
+       1% or so combined on the HTML iBench.
+
+        * khtml/rendering/render_br.cpp:
+        (RenderBR::setStyle): Don't call superclass setStyle, since
+       according to Dave we don't need any of the work done in there.
+        * kwq/KWQScrollView.mm:
+        (QScrollView::addChild): Don't actually move the widget if the x
+       coordinate is -500000, since we don't need the initial offscreen
+       positioning that KHTML does and it is costly to do.
+
 2004-04-22  Ken Kocienda  <kocienda@apple.com>
 
         Reviewed by Hyatt
index e231ac0a6af945332d65d22058f675fe20ec8d1b..9a80c29fd33221b0a7e05bf0abeda5bb6c4030d1 100644 (file)
@@ -296,7 +296,9 @@ void QScrollView::addChild(QWidget* child, int x, int y)
 {
     ASSERT(child != this);
     
-    child->move(x, y);
+    // we don't need to do the offscreen position initialization that KDE needs
+    if (x != -500000)
+       child->move(x, y);
 
     KWQ_BLOCK_EXCEPTIONS;