2009-01-02 Darin Adler <darin@apple.com>

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::loadItem): One small thing I forgot in my last check-in.
        This cuts down on the number of hash table operations during loading.

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index b47bf54..ac35a9a 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,9 @@
+2009-01-02  Darin Adler  <darin@apple.com>
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::loadItem): One small thing I forgot in my last check-in.
+        This cuts down on the number of hash table operations during loading.
+
 2009-01-02  Dmitry Titov  <dimich@chromium.org>
 
         Reviewed and landed by Darin.

diff --git a/WebCore/loader/FrameLoader.cpp b/WebCore/loader/FrameLoader.cpp
index 54f7acb..8d20edb 100644 (file)
--- a/WebCore/loader/FrameLoader.cpp
+++ b/WebCore/loader/FrameLoader.cpp
@@ -5108,9 +5108,14 @@ void FrameLoader::dispatchAssignIdentifierToInitialRequest(unsigned long identif
 
 void FrameLoader::dispatchWillSendRequest(DocumentLoader* loader, unsigned long identifier, ResourceRequest& request, const ResourceResponse& redirectResponse)
 {
+    StringImpl* oldRequestURL = request.url().string().impl();
     m_documentLoader->didTellClientAboutLoad(request.url());
+
     m_client->dispatchWillSendRequest(loader, identifier, request, redirectResponse);
-    m_documentLoader->didTellClientAboutLoad(request.url());
+
+    // If the URL changed, then we want to put that new URL in the "did tell client" set too.
+    if (oldRequestURL != request.url().string().impl())
+        m_documentLoader->didTellClientAboutLoad(request.url());
 
     if (Page* page = m_frame->page())
         page->inspectorController()->willSendRequest(loader, identifier, request, redirectResponse);