isLinkVisited: Don't build complete URL unless we use QWebHistoryInterface.

visitedURL() is non-trivial function with memory allocations,
so we should not call it for nothing.

Change-Id: I72e14fac75640cb6b22dd1444b9a061e8432dd7e
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/Source/WebKit/qt/WebCoreSupport/PlatformStrategiesQt.cpp b/Source/WebKit/qt/WebCoreSupport/PlatformStrategiesQt.cpp
index a8487f2c4..7fcf3de45 100644
--- a/Source/WebKit/qt/WebCoreSupport/PlatformStrategiesQt.cpp
+++ b/Source/WebKit/qt/WebCoreSupport/PlatformStrategiesQt.cpp
@@ -200,14 +200,14 @@ bool PlatformStrategiesQt::isLinkVisited(Page* page, LinkHash hash, const KURL&
 {
     ASSERT(hash);
 
-    Vector<UChar, 512> url;
-    visitedURL(baseURL, attributeURL, url);
-
     // If the Qt4.4 interface for the history is used, we will have to fallback
     // to the old global history.
     QWebHistoryInterface* iface = QWebHistoryInterface::defaultInterface();
-    if (iface)
+    if (iface) {
+        Vector<UChar, 512> url;
+        visitedURL(baseURL, attributeURL, url);
         return iface->historyContains(QString(reinterpret_cast<QChar*>(url.data()), url.size()));
+    }
 
     return page->group().isLinkVisited(hash);
 }