Move redirect_chain from NavigationEntry to FrameNavigationEntry.

content/public/browser/navigation_entry.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_NAVIGATION_ENTRY_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_ENTRY_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 202 matching lines @@
   //
   //   - this navigation hasn't completed yet;
   //   - a response wasn't received;
   //   - or this navigation was restored and for some reason the
   //     status code wasn't available.
   virtual void SetHttpStatusCode(int http_status_code) = 0;
   virtual int GetHttpStatusCode() const = 0;
 
   // 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;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_ENTRY_H_