Remove Apple's unused implementation of private browsing from WebCore

Source/WebCore/loader/HistoryController.cpp

Index: Source/WebCore/loader/HistoryController.cpp
diff --git a/Source/WebCore/loader/HistoryController.cpp b/Source/WebCore/loader/HistoryController.cpp
index 7d155a81590e044de2a38b4b2c39954ad6ba3675..d0f921f6b4363e8f08a62682ac3b375d3616673c 100644
--- a/Source/WebCore/loader/HistoryController.cpp
+++ b/Source/WebCore/loader/HistoryController.cpp
@@ -352,8 +352,6 @@ void HistoryController::updateForStandardLoad(HistoryUpdateType updateType)
 
     FrameLoader* frameLoader = m_frame->loader();
 
-    Settings* settings = m_frame->settings();
-    bool needPrivacy = !settings || settings->privateBrowsingEnabled();
     const KURL& historyURL = frameLoader->documentLoader()->urlForHistory();
 
     if (!frameLoader->documentLoader()->isClientRedirect()) {
@@ -366,7 +364,7 @@ void HistoryController::updateForStandardLoad(HistoryUpdateType updateType)
         updateCurrentItem();
     }
 
-    if (!historyURL.isEmpty() && !needPrivacy) {
+    if (!historyURL.isEmpty()) {
         if (Page* page = m_frame->page())
             addVisitedLink(page, historyURL);
     }
@@ -379,8 +377,6 @@ void HistoryController::updateForRedirectWithLockedBackForwardList()
         LOG(History, "WebCoreHistory: Updating History for redirect load in frame %s", m_frame->loader()->documentLoader()->title().string().utf8().data());
 #endif
     
-    Settings* settings = m_frame->settings();
-    bool needPrivacy = !settings || settings->privateBrowsingEnabled();
     const KURL& historyURL = m_frame->loader()->documentLoader()->urlForHistory();
 
     if (m_frame->loader()->documentLoader()->isClientRedirect()) {
@@ -396,7 +392,7 @@ void HistoryController::updateForRedirectWithLockedBackForwardList()
             parentFrame->loader()->history()->m_currentItem->setChildItem(createItem());
     }
 
-    if (!historyURL.isEmpty() && !needPrivacy) {
+    if (!historyURL.isEmpty()) {
         if (Page* page = m_frame->page())
             addVisitedLink(page, historyURL);
     }
@@ -416,11 +412,9 @@ void HistoryController::updateForClientRedirect()
         m_currentItem->clearScrollPoint();
     }
 
-    Settings* settings = m_frame->settings();
-    bool needPrivacy = !settings || settings->privateBrowsingEnabled();
     const KURL& historyURL = m_frame->loader()->documentLoader()->urlForHistory();
 
-    if (!historyURL.isEmpty() && !needPrivacy) {
+    if (!historyURL.isEmpty()) {
         if (Page* page = m_frame->page())
             addVisitedLink(page, historyURL);
     }
@@ -509,10 +503,6 @@ void HistoryController::updateForSameDocumentNavigation()
     if (m_frame->document()->url().isEmpty())
         return;
 
-    Settings* settings = m_frame->settings();
-    if (!settings || settings->privateBrowsingEnabled())
-        return;
-
     Page* page = m_frame->page();
     if (!page)
         return;
@@ -833,10 +823,6 @@ void HistoryController::pushState(PassRefPtr<SerializedScriptValue> stateObject,
 
     page->backForward()->addItem(topItem.release());
 
-    Settings* settings = m_frame->settings();
-    if (!settings || settings->privateBrowsingEnabled())
-        return;
-
     addVisitedLink(page, KURL(ParsedURLString, urlString));
 }
 
@@ -852,10 +838,6 @@ void HistoryController::replaceState(PassRefPtr<SerializedScriptValue> stateObje
     m_currentItem->setFormData(0);
     m_currentItem->setFormContentType(String());
 
-    Settings* settings = m_frame->settings();
-    if (!settings || settings->privateBrowsingEnabled())
-        return;
-
     ASSERT(m_frame->page());
     addVisitedLink(m_frame->page(), KURL(ParsedURLString, urlString));
 }