Reviewed by Anders.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 28 Oct 2006 06:18:19 +0000 (06:18 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 28 Oct 2006 06:18:19 +0000 (06:18 +0000)
        - reverted fix for http://bugs.webkit.org/show_bug.cgi?id=11212, later we will address the performance
        impact of fixing it again.

        * bridge/mac/FrameMac.mm:
        (WebCore::FrameMac::clear):

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

WebCore/ChangeLog
WebCore/bridge/mac/FrameMac.mm

index c77fd84c6689e4131241c21c53063fc855b2b8d0..9f76418b0c4fce424103ccb58d0700e02b1c89e3 100644 (file)
@@ -1,3 +1,13 @@
+2006-10-27  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Anders.
+
+        - reverted fix for http://bugs.webkit.org/show_bug.cgi?id=11212, later we will address the performance
+        impact of fixing it again.
+
+        * bridge/mac/FrameMac.mm:
+        (WebCore::FrameMac::clear):
+
 2006-10-27  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by John & Adam.
index 3113971bb072684f8db9787bc70a0cc1c2eb3b4c..726763654690c7c66b0488738533a7212544d9b1 100644 (file)
@@ -2879,7 +2879,9 @@ bool FrameMac::haveToldBridgeAboutLoad(const String& URL)
 
 void FrameMac::clear(bool clearWindowProperties)
 {
-    urlsBridgeKnowsAbout.clear();
+    // FIXME: commenting the below line causes <http://bugs.webkit.org/show_bug.cgi?id=11212>, but putting it
+    // back causes a measurable performance regression which we will need to fix to restore the correct behavior
+    // urlsBridgeKnowsAbout.clear();
     setMarkedTextRange(0, nil, nil);
     Frame::clear();
 }