Add the navigation redirect-chain to Sync sessions proto for offline analysis.

content/browser/frame_host/navigation_entry_impl.cc

Index: content/browser/frame_host/navigation_entry_impl.cc
diff --git a/content/browser/frame_host/navigation_entry_impl.cc b/content/browser/frame_host/navigation_entry_impl.cc
index 03dfa5afb5e5b5d73749907d241ad23b6f5939b2..b021d5eda18cea37a92d27924094752c5b77a5b7 100644
--- a/content/browser/frame_host/navigation_entry_impl.cc
+++ b/content/browser/frame_host/navigation_entry_impl.cc
@@ -289,6 +289,15 @@ int NavigationEntryImpl::GetHttpStatusCode() const {
   return http_status_code_;
 }
 
+void NavigationEntryImpl::SetRedirectChain(
+    const std::vector<GURL>& redirect_chain) {
+  redirect_chain_ = redirect_chain;

[haitaol1, 2014/02/27 22:30:22]

Donn: I found that even an empty GURL object takes almost 100 bytes because of
its internal parsed_url_ have multiple components each taking 8 bytes. Since you
only need URLs, can you just store a vector of URL strings?

[Donn Denman, 2014/03/25 18:21:19]

I don't think this is a problem, and doesn't seem worth changing.

+}
+
+const std::vector<GURL>& NavigationEntryImpl::GetRedirectChain() const {
+  return redirect_chain_;
+}
+
 bool NavigationEntryImpl::IsRestored() const {
   return restore_type_ != RESTORE_NONE;
 }
@@ -335,7 +344,7 @@ void NavigationEntryImpl::ResetForCommit() {
   set_is_renderer_initiated(false);
   set_transferred_global_request_id(GlobalRequestID());
   set_should_replace_entry(false);
-  redirect_chain_.clear();
+
   set_should_clear_history_list(false);
   set_frame_tree_node_id(-1);
 }