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

Side by Side Diff: content/browser/frame_host/navigator_impl.cc

Issue 2884383002: test (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/loader/DocumentLoader.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/navigator_impl.h" 5 #include "content/browser/frame_host/navigator_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/metrics/histogram_macros.h" 10 #include "base/metrics/histogram_macros.h"
(...skipping 969 matching lines...) Expand 10 before | Expand all | Expand 10 after
980 frame_tree_node->navigation_request(); 980 frame_tree_node->navigation_request();
981 981
982 // Client redirects during the initial history navigation of a child frame 982 // Client redirects during the initial history navigation of a child frame
983 // should take precedence over the history navigation (despite being renderer- 983 // should take precedence over the history navigation (despite being renderer-
984 // initiated). See https://crbug.com/348447 and https://crbug.com/691168. 984 // initiated). See https://crbug.com/348447 and https://crbug.com/691168.
985 if (ongoing_navigation_request && 985 if (ongoing_navigation_request &&
986 ongoing_navigation_request->request_params() 986 ongoing_navigation_request->request_params()
987 .is_history_navigation_in_new_child) { 987 .is_history_navigation_in_new_child) {
988 // Preemptively clear this local pointer before deleting the request. 988 // Preemptively clear this local pointer before deleting the request.
989 ongoing_navigation_request = nullptr; 989 ongoing_navigation_request = nullptr;
990 CHECK(false);
990 frame_tree_node->ResetNavigationRequest(false, true); 991 frame_tree_node->ResetNavigationRequest(false, true);
991 } 992 }
992 993
993 // The renderer-initiated navigation request is ignored iff a) there is an 994 // The renderer-initiated navigation request is ignored iff a) there is an
994 // ongoing request b) which is browser or user-initiated and c) the renderer 995 // ongoing request b) which is browser or user-initiated and c) the renderer
995 // request is not user-initiated. 996 // request is not user-initiated.
996 if (ongoing_navigation_request && 997 if (ongoing_navigation_request &&
997 (ongoing_navigation_request->browser_initiated() || 998 (ongoing_navigation_request->browser_initiated() ||
998 ongoing_navigation_request->begin_params().has_user_gesture) && 999 ongoing_navigation_request->begin_params().has_user_gesture) &&
999 !begin_params.has_user_gesture) { 1000 !begin_params.has_user_gesture) {
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
1296 if (navigation_handle) 1297 if (navigation_handle)
1297 navigation_handle->update_entry_id_for_transfer(entry->GetUniqueID()); 1298 navigation_handle->update_entry_id_for_transfer(entry->GetUniqueID());
1298 1299
1299 controller_->SetPendingEntry(std::move(entry)); 1300 controller_->SetPendingEntry(std::move(entry));
1300 if (delegate_) 1301 if (delegate_)
1301 delegate_->NotifyChangedNavigationState(content::INVALIDATE_TYPE_URL); 1302 delegate_->NotifyChangedNavigationState(content::INVALIDATE_TYPE_URL);
1302 } 1303 }
1303 } 1304 }
1304 1305
1305 } // namespace content 1306 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/loader/DocumentLoader.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698