Add a NavigationThrottle to the public content/ interface

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

 // 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.
 
 #include "chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h"
 
 #include <string>
 #include <vector>
 
 #include "base/base64.h"
@@ skipping 323 matching lines @@
       request->SetPriority(net::IDLE);
     }
   }
 
   ProfileIOData* io_data = ProfileIOData::FromResourceContext(
       resource_context);
 
 #if defined(OS_ANDROID)
   // TODO(davidben): This is insufficient to integrate with prerender properly.
   // https://crbug.com/370595
-  if (!is_prerendering) {
-    if (resource_type == content::RESOURCE_TYPE_MAIN_FRAME) {
-      throttles->push_back(
-          InterceptNavigationDelegate::CreateThrottleFor(request));
-    } else {
-      InterceptNavigationDelegate::UpdateUserGestureCarryoverInfo(request);
-    }
-  }
-#else
-  if (resource_type == content::RESOURCE_TYPE_MAIN_FRAME) {
-    // Redirect some navigations to apps that have registered matching URL
-    // handlers ('url_handlers' in the manifest).
-    content::ResourceThrottle* url_to_app_throttle =
-        AppUrlRedirector::MaybeCreateThrottleFor(request, io_data);
-    if (url_to_app_throttle)
-      throttles->push_back(url_to_app_throttle);
+  if (!is_prerendering && resource_type != content::RESOURCE_TYPE_MAIN_FRAME) {
+    InterceptNavigationDelegate::UpdateUserGestureCarryoverInfo(request);

[davidben, 2015/09/01 21:55:17]

[This is probably for a separate CL, but... wow, talk about questionable logic.
It just happens to hook into XHR user gestures and not, say, I tapped on a
button.
https://docs.google.com/document/d/1z2O07u5WoEtsLLUkx6RDjbVO5XrNwMJAQIBVvzRpE...
]

On the plus side, I think is_prerendering check and the TODO can go away since
they're not relevant for this call. Dunno if you want to add a TODO to revise
this one though.

[clamy, 2015/09/03 15:30:51]

Done.

   }
 #endif
 
 #if defined(OS_CHROMEOS)
   // Check if we need to add merge session throttle. This throttle will postpone
   // loading of main frames and XHR request.
   if (resource_type == content::RESOURCE_TYPE_MAIN_FRAME ||
       resource_type == content::RESOURCE_TYPE_XHR) {
     // Add interstitial page while merge session process (cookie
     // reconstruction from OAuth2 refresh token in ChromeOS login) is still in
@@ skipping 367 matching lines @@
                                        url_request->GetTotalReceivedBytes()));
   }
 }
 
 // static
 void ChromeResourceDispatcherHostDelegate::
     SetExternalProtocolHandlerDelegateForTesting(
     ExternalProtocolHandler::Delegate* delegate) {
   g_external_protocol_handler_delegate = delegate;
 }