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

Unified Diff: chrome/browser/ui/browser_navigator.cc

Issue 10830144: Consolidate all NavigationController::LoadURL and family functions (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebase onto TOT. Created 8 years, 4 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 | « chrome/browser/prerender/prerender_contents.cc ('k') | content/browser/android/content_view_core_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_navigator.cc
diff --git a/chrome/browser/ui/browser_navigator.cc b/chrome/browser/ui/browser_navigator.cc
index f24c2159efaa50127eca680f83a62e8144965ec7..ea1a31edfad7b18c5b10af934e16604c0c9e4664 100644
--- a/chrome/browser/ui/browser_navigator.cc
+++ b/chrome/browser/ui/browser_navigator.cc
@@ -218,25 +218,19 @@ void LoadURLInContents(WebContents* target_contents,
const GURL& url,
chrome::NavigateParams* params,
const std::string& extra_headers) {
+ content::NavigationController::LoadURLParams load_url_params(url);
+ load_url_params.referrer = params->referrer;
+ load_url_params.transition_type = params->transition;
+ load_url_params.extra_headers = extra_headers;
+
if (params->transferred_global_request_id != GlobalRequestID()) {
- target_contents->GetController().TransferURL(
- url,
- params->referrer,
- params->transition, extra_headers,
- params->transferred_global_request_id,
- params->is_renderer_initiated);
+ load_url_params.is_renderer_initiated = params->is_renderer_initiated;
+ load_url_params.transferred_global_request_id =
+ params->transferred_global_request_id;
} else if (params->is_renderer_initiated) {
- target_contents->GetController().LoadURLFromRenderer(
- url,
- params->referrer,
- params->transition, extra_headers);
- } else {
- target_contents->GetController().LoadURL(
- url,
- params->referrer,
- params->transition, extra_headers);
+ load_url_params.is_renderer_initiated = true;
}
-
+ target_contents->GetController().LoadURLWithParams(load_url_params);
}
// This class makes sure the Browser object held in |params| is made visible
« no previous file with comments | « chrome/browser/prerender/prerender_contents.cc ('k') | content/browser/android/content_view_core_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698