2007-01-29 Ada Chan <adachan@apple.com>
authoradachan <adachan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Jan 2007 00:40:39 +0000 (00:40 +0000)
committeradachan <adachan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Jan 2007 00:40:39 +0000 (00:40 +0000)
        Reviewed by Brady.

        Moved the update of the title of the current entry in the backforward list to WebCore.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::didChangeTitle):
        * WebCoreSupport/WebFrameLoaderClient.mm:
        (WebFrameLoaderClient::setTitle):

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

WebCore/ChangeLog
WebCore/loader/FrameLoader.cpp
WebKit/ChangeLog
WebKit/WebCoreSupport/WebFrameLoaderClient.mm

index 2747fdc..7f996fd 100644 (file)
@@ -1,3 +1,12 @@
+2007-01-29  Ada Chan  <adachan@apple.com>
+
+        Reviewed by Brady.
+        
+        Moved the update of the title of the current entry in the backforward list to WebCore.
+
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::didChangeTitle):
+
 2007-01-29  Adele Peterson  <adele@apple.com>
 
         Reviewed by Darin.
index 519969c..e539ad4 100644 (file)
@@ -4267,6 +4267,8 @@ void FrameLoader::didChangeTitle(DocumentLoader* loader)
     // The title doesn't get communicated to the WebView until we are committed.
     if (loader->isCommitted()) {
         // Must update the entries in the back-forward list too.
+        if (m_currentHistoryItem)
+            m_currentHistoryItem->setTitle(loader->title());
         // This must go through the WebFrame because it has the right notion of the current b/f item.
         m_client->setTitle(loader->title(), loader->urlForHistory());
         m_client->setMainFrameDocumentReady(true); // update observers with new DOMDocument
index 074bdaf..7602be0 100644 (file)
@@ -1,3 +1,12 @@
+2007-01-29  Ada Chan  <adachan@apple.com>
+
+        Reviewed by Brady.
+
+        Moved the update of the title of the current entry in the backforward list to WebCore.
+
+        * WebCoreSupport/WebFrameLoaderClient.mm:
+        (WebFrameLoaderClient::setTitle):
+
 2007-01-29  Adele Peterson  <adele@apple.com>
 
         Reviewed by Darin.
index dc971c7..c5a73ff 100644 (file)
@@ -914,8 +914,6 @@ void WebFrameLoaderClient::setTitle(const String& title, const KURL& URL)
         return;
     NSString *titleNSString = title;
     [[[WebHistory optionalSharedHistory] itemForURL:nsURL] setTitle:titleNSString];
-    if (HistoryItem* item = core(m_webFrame.get())->loader()->currentHistoryItem())
-        item->setTitle(title);
 }
 
 // The following 2 functions are copied from [NSHTTPURLProtocol _cachedResponsePassesValidityChecks] and modified for our needs.