Changeset 176588 in webkit

Timestamp:

Nov 30, 2014, 4:20:34 PM (12 years ago)

Author:

andersca@apple.com

Message:

Fix a test failure.

• History/WebHistory.mm:

(+[WebHistory _setVisitedLinkTrackingEnabled:]):
Call through to the visited link store.

(+[WebHistory _removeAllVisitedLinks]):
Ditto.

• WebCoreSupport/WebVisitedLinkStore.mm:

(WebVisitedLinkStore::isLinkVisited):
Make sure to populate visited links.

Location:

trunk/Source/WebKit/mac

Files:

• ChangeLog (modified) (1 diff)
• History/WebHistory.mm (modified) (2 diffs)
• WebCoreSupport/WebVisitedLinkStore.mm (modified) (1 diff)

trunk/Source/WebKit/mac/ChangeLog

@@ +1 @@
+2014-11-30  Anders Carlsson  <andersca@apple.com>
+
+        Fix a test failure.
+
+        * History/WebHistory.mm:
+        (+[WebHistory _setVisitedLinkTrackingEnabled:]):
+        Call through to the visited link store.
+
+        (+[WebHistory _removeAllVisitedLinks]):
+        Ditto.
+
+        * WebCoreSupport/WebVisitedLinkStore.mm:
+        (WebVisitedLinkStore::isLinkVisited):
+        Make sure to populate visited links.
+
 2014-11-30  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit/mac/History/WebHistory.mm

@@ -922 +922 @@
 {
     PageGroup::setShouldTrackVisitedLinks(visitedLinkTrackingEnabled);
+    WebVisitedLinkStore::setShouldTrackVisitedLinks(visitedLinkTrackingEnabled);
 }
 
@@ -927 +928 @@
 {
     PageGroup::removeAllVisitedLinks();
+    WebVisitedLinkStore::removeAllVisitedLinks();
 }
 

trunk/Source/WebKit/mac/WebCoreSupport/WebVisitedLinkStore.mm

@@ -109 +109 @@
 bool WebVisitedLinkStore::isLinkVisited(Page& page, LinkHash linkHash, const URL& baseURL, const AtomicString& attributeURL)
 {
+    populateVisitedLinksIfNeeded(page);
+
     return m_visitedLinkHashes.contains(linkHash);
 }