Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(521)

Unified Diff: webkit/glue/webframe_impl.cc

Issue 19485: Chromium side of merge 40314:40364. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/build/WebCore/WebCore.vcproj ('k') | webkit/glue/webview_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/webframe_impl.cc
===================================================================
--- webkit/glue/webframe_impl.cc (revision 8920)
+++ webkit/glue/webframe_impl.cc (working copy)
@@ -1629,30 +1629,16 @@
// If we're moving in the backforward list, we might want to replace the
// content of this child frame with whatever was there at that point.
- // Reload will maintain the frame contents, LoadSame will not.
if (parent_item && parent_item->children().size() != 0 &&
- (isBackForwardLoadType(load_type) ||
- load_type == WebCore::FrameLoadTypeReloadAllowingStaleData)) {
+ isBackForwardLoadType(load_type)) {
HistoryItem* child_item = parent_item->childItemWithName(request.frameName());
if (child_item) {
// Use the original URL to ensure we get all the side-effects, such as
// onLoad handlers, of any redirects that happened. An example of where
// this is needed is Radar 3213556.
new_url = child_item->originalURL();
-
- // These behaviors implied by these loadTypes should apply to the child
- // frames
child_load_type = load_type;
-
- if (isBackForwardLoadType(load_type)) {
- // For back/forward, remember this item so we can traverse any child
- // items as child frames load.
- child_frame->loader()->setProvisionalHistoryItem(child_item);
- } else {
- // For reload, just reinstall the current item, since a new child frame
- // was created but we won't be creating a new BF item
- child_frame->loader()->setCurrentHistoryItem(child_item);
- }
+ child_frame->loader()->setProvisionalHistoryItem(child_item);
}
}
« no previous file with comments | « webkit/build/WebCore/WebCore.vcproj ('k') | webkit/glue/webview_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698