Store original request URL in NavigationEntry

content/browser/tab_contents/navigation_controller_impl.cc

Index: content/browser/tab_contents/navigation_controller_impl.cc
diff --git a/content/browser/tab_contents/navigation_controller_impl.cc b/content/browser/tab_contents/navigation_controller_impl.cc
index dbdc940078290db4681c3b4072cd13bbdad31a7d..898996f31dfb1ea7b6caa0b5a2ffc15bdad1d5ee 100644
--- a/content/browser/tab_contents/navigation_controller_impl.cc
+++ b/content/browser/tab_contents/navigation_controller_impl.cc
@@ -874,6 +874,11 @@ void NavigationControllerImpl::RendererDidNavigateToNewPage(
   new_entry->SetHasPostData(params.is_post);
   new_entry->SetPostID(params.post_id);
 
+  if (params.redirects.size() > 0)
+    new_entry->SetOriginalRequestURL(params.redirects[0]);
+  else
+    new_entry->SetOriginalRequestURL(GURL());

[Charlie Reis, 2012/04/10 01:37:28]

Why not use the actual URL if there weren't any redirects?

[gone, 2012/04/10 17:41:34]

Changed to store the URL.  I was hoping to save some memory by falling back to
using the regular URL field, but I guess there could be situations where this
gets changed.

+
   InsertOrReplaceEntry(new_entry, *did_replace_entry);
 }