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):

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index c77fd84c6689e4131241c21c53063fc855b2b8d0..9f76418b0c4fce424103ccb58d0700e02b1c89e3 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -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.

diff --git a/WebCore/bridge/mac/FrameMac.mm b/WebCore/bridge/mac/FrameMac.mm
index 3113971bb072684f8db9787bc70a0cc1c2eb3b4c..726763654690c7c66b0488738533a7212544d9b1 100644 (file)
--- a/WebCore/bridge/mac/FrameMac.mm
+++ b/WebCore/bridge/mac/FrameMac.mm
@@ -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();
 }