Changeset 64535 in webkit

Timestamp:

Aug 3, 2010, 1:32:33 AM (15 years ago)

Author:

ap@apple.com

Message:

Unreviewed (emergency test fix).

REGRESSION (r64518): Most regression tests fail on ASSERT(shouldTrackVisitedLinks)
on Snow Leopard.

Platforms that use strategies now use a different code path than others, and that code path
lacks necessary checks. We don't have a Snow Leopard debug buildbot.

• page/PageGroup.cpp: (WebCore::PageGroup::addVisitedLinkHash): The private version of addVisitedLink() must not be called when we're not tracking visited links.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/PageGroup.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-08-03  Alexey Proskuryakov  <ap@apple.com>
+
+        Unreviewed (emergency test fix).
+
+        REGRESSION (r64518): Most regression tests fail on ASSERT(shouldTrackVisitedLinks)
+        on Snow Leopard.
+
+        Platforms that use strategies now use a different code path than others, and that code path
+        lacks necessary checks. We don't have a Snow Leopard debug buildbot.
+
+        * page/PageGroup.cpp: (WebCore::PageGroup::addVisitedLinkHash): The private version of
+        addVisitedLink() must not be called when we're not tracking visited links.
+
 2010-08-02  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/WebCore/page/PageGroup.cpp

@@ -139 +139 @@
 void PageGroup::addVisitedLinkHash(LinkHash hash)
 {
-    addVisitedLink(hash);
+    if (shouldTrackVisitedLinks)
+        addVisitedLink(hash);
 }