Move redirect_chain from NavigationEntry to FrameNavigationEntry.

content/public/browser/navigation_entry.h

Index: content/public/browser/navigation_entry.h
diff --git a/content/public/browser/navigation_entry.h b/content/public/browser/navigation_entry.h
index 87c17cd29e18c2dd566e6435cc46dbb6716c75d5..38e8eb1a388c3830e3618afcae5fe4f5e297eb8c 100644
--- a/content/public/browser/navigation_entry.h
+++ b/content/public/browser/navigation_entry.h
@@ -220,8 +220,9 @@ class NavigationEntry {
 
   // The redirect chain traversed during this navigation, from the initial
   // redirecting URL to the final non-redirecting current URL.
-  virtual void SetRedirectChain(const std::vector<GURL>& redirects) = 0;
-  virtual const std::vector<GURL>& GetRedirectChain() const = 0;
+  virtual void SetMainFrameRedirectChain(

[Charlie Reis, 2016/09/28 04:13:17]

I appreciate getting to easily see all the call sites that are affected due to
the rename, but I think we should put it back to SetRedirectChain.  We generally
haven't been updating other method names (e.g., GetURL, etc), and the subframe
tracking is mostly an internal implementation detail.

[arthursonzogni, 2016/09/28 16:36:14]

Yes, that's what I thought. I will make it come back to the original names.

+      const std::vector<GURL>& redirects) = 0;
+  virtual const std::vector<GURL>& GetMainFrameRedirectChain() const = 0;
 
   // True if this entry is restored and hasn't been loaded.
   virtual bool IsRestored() const = 0;