Reviewed by Trey.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Jul 2004 16:45:13 +0000 (16:45 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Jul 2004 16:45:13 +0000 (16:45 +0000)
        - fixed <rdar://problem/3717147> folder icon used for error page in back/forward
        menu when iTunes is not installed

        * WebView.subproj/WebFrame.m:
        (-[WebFrame _transitionToCommitted:]):
        When displaying the error page for an unreachable URL, leave the requested URL
        in the WebHistoryItem; don't clobber it with a bogus one that represents the
        error page. This not only avoids the wrong-icon problem, but should also help
        in cases where an unreachable URL becomes reachable later on.

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

WebKit/ChangeLog
WebKit/WebView.subproj/WebFrame.m

index 73a84bd043bace686b38b9051e3a6b39d35aa534..e8c3188ada05bfd63ecd2b2459c7ab822e7c8f4a 100644 (file)
@@ -1,3 +1,17 @@
+2004-07-06  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Trey.
+
+        - fixed <rdar://problem/3717147> folder icon used for error page in back/forward 
+        menu when iTunes is not installed
+
+        * WebView.subproj/WebFrame.m:
+        (-[WebFrame _transitionToCommitted:]):
+        When displaying the error page for an unreachable URL, leave the requested URL
+        in the WebHistoryItem; don't clobber it with a bogus one that represents the
+        error page. This not only avoids the wrong-icon problem, but should also help
+        in cases where an unreachable URL becomes reachable later on.
+
 2004-07-06  Trey Matteson  <trey@apple.com>
 
        3716053 - www.theage.com.au has extra back/forward items due to ads
@@ -42,6 +56,7 @@
         (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
        Ask for slideback.
 
+>>>>>>> 1.2612
 2004-07-02  Darin Adler  <darin@apple.com>
 
         Reviewed by Dave.
index dd10937bf1b6dfea8bf2e0c69d9d0b03954763c4..e4c07c10fa8d138200e0e59d3e1bb1d4a3d40b34 100644 (file)
@@ -744,11 +744,18 @@ NSString *WebPageCacheDocumentViewKey = @"WebPageCacheDocumentViewKey";
                     }
 
                 } else {
-                    // update the URL in the BF list that we made before the redirect
-                    [[_private currentItem] setURL:[[ds request] URL]];
-                    // clear out the form data so we don't repost it to the wrong place if we
-                    // ever go back/forward to this item
-                    [[_private currentItem] _setFormInfoFromRequest:[ds request]];
+                    NSURLRequest *request = [ds request];
+                    
+                    // update the URL in the BF list that we made before the redirect, unless
+                    // this is alternate content for an unreachable URL (we want the BF list
+                    // item to remember the unreachable URL in case it becomes reachable later)
+                    if ([request _webDataRequestUnreachableURL] == nil) {
+                        [[_private currentItem] setURL:[request URL]];
+
+                        // clear out the form data so we don't repost it to the wrong place if we
+                        // ever go back/forward to this item
+                        [[_private currentItem] _setFormInfoFromRequest:request];
+                    }
                 }
                 [self _makeDocumentView];
                 break;