Persist prompt/block download limiter state on renderer-initiated loads.

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index 6fcdb074b4a0f6badd1ea04256214a7e2223480d..b5061e345425ba82c2cd31bc71b61c4446e11e62 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -175,12 +175,22 @@ void NavigatorImpl::DidStartProvisionalLoad(
         std::unique_ptr<NavigationHandleImpl>());
   }
 
-  NavigationEntry* pending_entry = controller_->GetPendingEntry();
+  // It is safer to assume navigations are renderer-initiated unless shown
+  // otherwise. Browser navigations are generally more powerful, and they should

[clamy, 2016/05/18 13:53:36]

nit: "navigations are more powerful" sounds a bit weird to me. How about "have
more privileges" or "have priority over renderer-initiated navigations"?

[dominickn, 2016/05/19 07:25:45]

Done.

+  // always have a pending NavigationEntry which can be used to differentiate.
+  bool is_renderer_initiated = true;
+  int pending_nav_entry_id = 0;
+  NavigationEntryImpl* pending_entry = controller_->GetPendingEntry();
+  if (pending_entry) {
+    is_renderer_initiated = pending_entry->is_renderer_initiated();
+    pending_nav_entry_id = pending_entry->GetUniqueID();
+  }
   render_frame_host->SetNavigationHandle(NavigationHandleImpl::Create(
       validated_url, render_frame_host->frame_tree_node(),
+      is_renderer_initiated,
       false,             // is_synchronous
       is_iframe_srcdoc,  // is_srcdoc
-      navigation_start, pending_entry ? pending_entry->GetUniqueID() : 0));
+      navigation_start, pending_nav_entry_id));
 }
 
 void NavigatorImpl::DidFailProvisionalLoadWithError(