Plumb the referrer throughout the OpenURL APIs....

chrome/browser/browser.cc

Index: chrome/browser/browser.cc
===================================================================
--- chrome/browser/browser.cc	(revision 3991)
+++ chrome/browser/browser.cc	(working copy)
@@ -522,7 +522,7 @@
 // TabContentsDelegate
 
 void Browser::OpenURLFromTab(TabContents* source,
-                             const GURL& url,
+                             const GURL& url, const GURL& referrer,
                              WindowOpenDisposition disposition,
                              PageTransition::Type transition) {
   // No code for these yet
@@ -571,7 +571,7 @@
     if (b->tab_count() == 0 && disposition == NEW_BACKGROUND_TAB)
       disposition = NEW_FOREGROUND_TAB;
 
-    b->OpenURL(url, disposition, transition);
+    b->OpenURL(url, referrer, disposition, transition);
     b->Show();
     b->MoveToFront(true);
     return;
@@ -583,7 +583,8 @@
   if (disposition == NEW_WINDOW) {
     Browser* new_browser = new Browser(gfx::Rect(), SW_SHOWNORMAL, profile_,
                                        BrowserType::TABBED_BROWSER, L"");
-    new_contents = new_browser->AddTabWithURL(url, transition, true, instance);
+    new_contents = new_browser->AddTabWithURL(url, referrer, transition, true,
+                                              instance);
     new_browser->Show();
   } else if ((disposition == CURRENT_TAB) && current_tab) {
     if (transition == PageTransition::TYPED ||
@@ -615,7 +616,7 @@
         tabstrip_model_.ForgetGroup(current_tab);
       }
     }
-    current_tab->controller()->LoadURL(url, transition);
+    current_tab->controller()->LoadURL(url, referrer, transition);
     // The TabContents might have changed as part of the navigation (ex: new tab
     // page can become WebContents).
     new_contents = current_tab->controller()->active_contents();
@@ -630,8 +631,8 @@
     return;
   } else if (disposition != SUPPRESS_OPEN) {
     new_contents =
-        AddTabWithURL(url, transition, disposition != NEW_BACKGROUND_TAB,
-                      instance);
+        AddTabWithURL(url, referrer, transition,
+                      disposition != NEW_BACKGROUND_TAB, instance);
   }
 
   if (disposition != NEW_BACKGROUND_TAB && source_tab_was_frontmost) {
@@ -1208,8 +1209,8 @@
 // Tab Creation Functions
 
 TabContents* Browser::AddTabWithURL(
-    const GURL& url, PageTransition::Type transition, bool foreground,
-    SiteInstance* instance) {
+    const GURL& url, const GURL& referrer, PageTransition::Type transition,
+    bool foreground, SiteInstance* instance) {
   if (type_ == BrowserType::APPLICATION && tabstrip_model_.count() == 1) {
     NOTREACHED() << "Cannot add a tab in a mono tab application.";
     return NULL;
@@ -1219,8 +1220,8 @@
   if (url_to_load.is_empty())
     url_to_load = GetHomePage();
   TabContents* contents =
-      CreateTabContentsForURL(url_to_load, profile_, transition, false,
-                              instance);
+      CreateTabContentsForURL(url_to_load, referrer, profile_, transition,
+                              false, instance);
   tabstrip_model_.AddTabContents(contents, -1, transition, foreground);
   // By default, content believes it is not hidden.  When adding contents
   // in the background, tell it that it's hidden.
@@ -1237,14 +1238,14 @@
   // TODO(acw): Do we need an "application launched" transition type?
   // TODO(creis): Should we reuse the current instance (ie. process) here?
   TabContents* contents =
-      CreateTabContentsForURL(web_app->url(), profile, PageTransition::LINK,
-                              lazy, NULL);
+      CreateTabContentsForURL(web_app->url(), GURL(), profile,
+                              PageTransition::LINK, lazy, NULL);
   if (contents->AsWebContents())
     contents->AsWebContents()->SetWebApp(web_app);
 
   if (lazy) {
     contents->controller()->LoadURLLazily(
-        web_app->url(), PageTransition::LINK, web_app->name(), NULL);
+        web_app->url(), GURL(), PageTransition::LINK, web_app->name(), NULL);
   }
   tabstrip_model_.AddTabContents(contents, -1, PageTransition::LINK, !lazy);
   return contents;
@@ -1331,8 +1332,9 @@
 }
 
 TabContents* Browser::CreateTabContentsForURL(
-    const GURL& url, Profile* profile, PageTransition::Type transition,
-    bool defer_load, SiteInstance* instance) const {
+    const GURL& url, const GURL& referrer, Profile* profile,
+    PageTransition::Type transition, bool defer_load,
+    SiteInstance* instance) const {
   // Create an appropriate tab contents.
   GURL real_url = url;
   TabContentsType type = TabContents::TypeForURL(&real_url);
@@ -1345,7 +1347,7 @@
   if (!defer_load) {
     // Load the initial URL before adding the new tab contents to the tab strip
     // so that the tab contents has navigation state.
-    contents->controller()->LoadURL(url, transition);
+    contents->controller()->LoadURL(url, referrer, transition);
   }
 
   return contents;
@@ -1613,7 +1615,7 @@
     }
   }
 
-  TabContents* contents = CreateTabContentsForURL(url, profile_,
+  TabContents* contents = CreateTabContentsForURL(url, GURL(), profile_,
                                                   PageTransition::LINK, false,
                                                   NULL);
   AddNewContents(NULL, contents, NEW_FOREGROUND_TAB, gfx::Rect(), true);
@@ -1637,7 +1639,7 @@
   } else {
     // No navigations. Create a tab with about:blank.
     TabContents* contents =
-        CreateTabContentsForURL(GURL("about:blank"), profile_,
+        CreateTabContentsForURL(GURL("about:blank"), GURL(), profile_,
                                 PageTransition::START_PAGE, false, NULL);
     return new NavigationController(contents, profile_);
   }
@@ -1652,7 +1654,8 @@
     browser = new Browser(gfx::Rect(), SW_SHOWNORMAL, off_the_record_profile,
                           BrowserType::TABBED_BROWSER, L"");
   }
-  browser->AddTabWithURL(url, PageTransition::LINK, true, NULL);
+  // TODO(eroman): should we have referrer here?
+  browser->AddTabWithURL(url, GURL(), PageTransition::LINK, true, NULL);
   browser->Show();
   browser->MoveToFront(true);
 }