summaryrefslogtreecommitdiffstats
path: root/chrome/browser/automation/automation_provider.cc
diff options
context:
space:
mode:
authorericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-10-27 23:01:50 +0000
committerericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-10-27 23:01:50 +0000
commitc058805b99cce98cbc352c499f3e69fd355a84c2 (patch)
tree93fb4ffdf5daf4d8ae0e3291919a074a8b9e58a7 /chrome/browser/automation/automation_provider.cc
parentf07cc2ebf950cb22c5c386f95a1be0292de4db3b (diff)
downloadchromium_src-c058805b99cce98cbc352c499f3e69fd355a84c2.zip
chromium_src-c058805b99cce98cbc352c499f3e69fd355a84c2.tar.gz
chromium_src-c058805b99cce98cbc352c499f3e69fd355a84c2.tar.bz2
Plumb the referrer throughout the OpenURL APIs.
http://code.google.com/p/chromium/issues/detail?id=3224 Caveats: * Did not update TabNavigation yet. Hence session restore will continue to load the tabs with empty referrer. * Did not plumb referrer into incognito url open. (Not sure what the right thing to do is here with respect to privacy vs compatibility.) * Did not plumb referrer throughout the automation controller. No functional impact here, but it makes the code inconsistent with the non-automation version. Review URL: http://codereview.chromium.org/8186 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4036 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/automation/automation_provider.cc')
-rw-r--r--chrome/browser/automation/automation_provider.cc8
1 files changed, 4 insertions, 4 deletions
diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc
index d567a53..7b9214c 100644
--- a/chrome/browser/automation/automation_provider.cc
+++ b/chrome/browser/automation/automation_provider.cc
@@ -826,7 +826,7 @@ void AutomationProvider::AppendTab(const IPC::Message& message,
Browser* browser = browser_tracker_->GetResource(handle);
observer = AddTabStripObserver(browser, message.routing_id());
TabContents* tab_contents =
- browser->AddTabWithURL(url, PageTransition::TYPED, true, NULL);
+ browser->AddTabWithURL(url, GURL(), PageTransition::TYPED, true, NULL);
if (tab_contents) {
append_tab_response =
GetIndexForNavigationController(tab_contents->controller(), browser);
@@ -866,7 +866,7 @@ void AutomationProvider::NavigateToURL(const IPC::Message& message,
new AutomationMsg_NavigateToURLResponse(
message.routing_id(), AUTOMATION_MSG_NAVIGATION_AUTH_NEEDED));
// TODO(darin): avoid conversion to GURL
- browser->OpenURL(url, CURRENT_TAB, PageTransition::TYPED);
+ browser->OpenURL(url, GURL(), CURRENT_TAB, PageTransition::TYPED);
return;
}
}
@@ -888,7 +888,7 @@ void AutomationProvider::NavigationAsync(const IPC::Message& message,
if (browser) {
// Don't add any listener unless a callback mechanism is desired.
// TODO(vibhor): Do this if such a requirement arises in future.
- browser->OpenURL(url, CURRENT_TAB, PageTransition::TYPED);
+ browser->OpenURL(url, GURL(), CURRENT_TAB, PageTransition::TYPED);
status = true;
}
}
@@ -2004,7 +2004,7 @@ void AutomationProvider::NavigateInExternalTab(const IPC::Message& message,
if (tab_tracker_->ContainsHandle(handle)) {
NavigationController* tab = tab_tracker_->GetResource(handle);
- tab->LoadURL(url, PageTransition::TYPED);
+ tab->LoadURL(url, GURL(), PageTransition::TYPED);
status = true;
}