[Android Downloads] Long-press menu item "Download Link" should delegate job to OfflinePages backen…

chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc

Index: chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc
diff --git a/chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc b/chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc
index a144df7c4cdae464df51155daecf41bc870484f4..ba05d370514c520be686d9e9a85c960245cb1e1e 100644
--- a/chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc
+++ b/chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc
@@ -108,6 +108,10 @@
 #include "components/navigation_interception/intercept_navigation_delegate.h"
 #endif
 
+#if BUILDFLAG(ANDROID_JAVA_UI)
+#include "chrome/browser/android/offline_pages/downloads/resource_throttle.h"
+#endif
+
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/login/signin/merge_session_resource_throttle.h"
 #include "chrome/browser/chromeos/login/signin/merge_session_throttling_utils.h"
@@ -545,6 +549,11 @@ void ChromeResourceDispatcherHostDelegate::DownloadStarting(
                                     resource_context,
                                     content::RESOURCE_TYPE_MAIN_FRAME,
                                     throttles);
+#if BUILDFLAG(ANDROID_JAVA_UI)
+    // On Android, forward text/html downloads to OfflinePages backend.
+    throttles->push_back(

[Pete Williamson, 2016/12/19 21:48:58]

Should the throttle be pushed back *before* we call
AppendStandardResourceThrottles?

[Dmitry Titov, 2016/12/19 22:39:32]

See the earlier comment by asanka on this CL. We need to do it in this order
because SafeBrowsing throttle is added as "Standard" and it should be able to
intercept the download before us.

+        new offline_pages::downloads::ResourceThrottle(request));
+#endif
   }
 }